Re: [PATCH v7] efi: new 'connectefi' command

2025-06-16 Thread Renaud Métrich via Grub-devel
Please see inline, thanks for reviewing. Le 6/10/25 à 7:36 PM, sudhakar a écrit : On 2025-06-10 15:24, Renaud Métrich via Grub-devel wrote: When network booting is used, trying to chainload to the local disk (which is used in deployment tools such as Red Hat Satellite) may fail when searching

[PATCH v7] efi: new 'connectefi' command

2025-06-10 Thread Renaud Métrich via Grub-devel
x" ]; then chainloader ($prefix)/EFI/redhat/shimx64.efi ... 8< 8< 8< ---- The code is easily extensible to handle other arguments in the future if needed. Signed-off-by: Renaud Métrich --- docs/grub.texi

[PATCH v1] lsefi: fixed memory leaks

2025-06-10 Thread Renaud Métrich via Grub-devel
Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/grub-core/commands/efi/lsefi.c b/grub-core/commands/efi/lsefi.c index 7b8316d41..a70f243d0 100644 --- a/grub-core/commands/efi/lsefi.c +++ b/grub-core

Re: [PATCH v6 2/2] efi: new 'connectefi' command

2025-05-31 Thread Renaud Métrich via Grub-devel
Hi Daniel, Sorry I missed your email, please see inline. Le 3/21/25 à 3:51 PM, Daniel Kiper a écrit : On Wed, Mar 19, 2025 at 01:47:57PM +0100, Renaud Métrich via Grub-devel wrote: When network booting is used, trying to chainload to the local disk (which is used in deployment tools such as

Re: [PATCH v6 2/2] efi: new 'connectefi' command

2025-05-31 Thread Renaud Métrich via Grub-devel
erstand or am wrong. Thank you! Andrew On Wed, Mar 19, 2025 at 7:53 AM Renaud Métrich via Grub-devel wrote: When network booting is used, trying to chainload to the local disk (which is used in deployment tools such as Red Hat Satellite) may fail when searching for the boot loader, e.g. /EFI/

[PATCH v2] grub-probe: detect DDF container similar to IMSM

2025-04-24 Thread Renaud Métrich via Grub-devel
https://issues.redhat.com/browse/RHEL-44336 Signed-off-by: Renaud Métrich --- grub-core/osdep/linux/getroot.c | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/grub-core/osdep/linux/getroot.c b/grub-core/osdep/linux/getroot.c index 7dd775d2a..0c6c12945 100644

Re: [PATCH v1] grub-probe: detect DDF container similar to IMSM

2025-04-24 Thread Renaud Métrich via Grub-devel
Yes exactly, there are BIOS or UEFI implementations that support DDF containers natively. I've sent the new version of the patch right now. Le 4/14/25 à 11:47 AM, Vladimir 'phcoder' Serbinenko a écrit : Le ven. 21 juin 2024, 11:21, Renaud Métrich a écrit : DDF an

[PATCH v6 1/2] lsefi: fixed memory leaks

2025-03-19 Thread Renaud Métrich via Grub-devel
Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 4 1 file changed, 4 insertions(+) diff --git a/grub-core/commands/efi/lsefi.c b/grub-core/commands/efi/lsefi.c index 7b8316d41..bda25a3a9 100644 --- a/grub-core/commands/efi/lsefi.c +++ b/grub-core/commands/efi/lsefi.c

[PATCH v6 2/2] efi: new 'connectefi' command

2025-03-19 Thread Renaud Métrich via Grub-devel
x" ]; then chainloader ($prefix)/EFI/redhat/shimx64.efi ... 8< 8< 8< ---- The code is easily extensible to handle other arguments in the future if needed. Signed-off-by: Renaud Métrich --- NEWS

Re: [PATCH v5 2/2] efi: new 'connectefi' command

2024-07-16 Thread Renaud Métrich
Serbinenko a écrit : The need to connect looks like an internal implementation detail. Can we treat it as such and connect when needed automatically rather than having an extra configuration knob? On Fri, Jun 28, 2024 at 2:14 PM Renaud Métrich wrote: When efi.quickboot is enabled on VMWare (whi

[PATCH v5 1/2] lsefi: fixed memory leaks

2024-06-28 Thread Renaud Métrich
Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 4 1 file changed, 4 insertions(+) diff --git a/grub-core/commands/efi/lsefi.c b/grub-core/commands/efi/lsefi.c index 7b8316d41..bda25a3a9 100644 --- a/grub-core/commands/efi/lsefi.c +++ b/grub-core/commands/efi/lsefi.c

[PATCH v5 2/2] efi: new 'connectefi' command

2024-06-28 Thread Renaud Métrich
uot;$prefix" ]; then chainloader ($prefix)/EFI/redhat/grubx64/efi ... 8< 8< ---- 8< ---- The code is easily extensible to handle other arguments in the future if needed. Signed-off-by: Renaud Métrich --- NEWS

[PATCH v1] grub-probe: detect DDF container similar to IMSM

2024-06-21 Thread Renaud Métrich
DDF and IMSM are very similar in handling, especially these should not be considered as RAID abstraction. This fixes the requirement of having a device map when probing DDF containers. Fixes https://issues.redhat.com/browse/RHEL-44336 Signed-off-by: Renaud Métrich --- grub-core/osdep/linux

[PATCH v2 3/3] dns: fix lookup error when no IPv6 is returned

2023-05-03 Thread Renaud Métrich
- 8< The root cause is the code exiting prematurely when the data->addresses buffer has been allocated in recv_hook(), even if there was no address returned last time recv_hook() executed. Signed-off-by: Renaud Métrich --- grub-core/net/dns.c | 22 ++

[PATCH v2 2/3] dns: add debugging messages in recv_hook() function

2023-05-03 Thread Renaud Métrich
Signed-off-by: Renaud Métrich --- grub-core/net/dns.c | 5 + 1 file changed, 5 insertions(+) diff --git a/grub-core/net/dns.c b/grub-core/net/dns.c index f28d1099f..6993a4aba 100644 --- a/grub-core/net/dns.c +++ b/grub-core/net/dns.c @@ -355,6 +355,7 @@ recv_hook (grub_net_udp_socket_t sock

[PATCH v2 1/3] dns: simplify error handling of recv_hook() function

2023-05-03 Thread Renaud Métrich
Signed-off-by: Renaud Métrich --- grub-core/net/dns.c | 48 ++--- 1 file changed, 15 insertions(+), 33 deletions(-) diff --git a/grub-core/net/dns.c b/grub-core/net/dns.c index afa389494..f28d1099f 100644 --- a/grub-core/net/dns.c +++ b/grub-core/net

[PATCH v1] dns: fixed lookup error when no IPv6 is returned

2023-04-28 Thread Renaud Métrich
- 8< The root cause is the code exiting prematurely when the data->addresses buffer has been allocated in recv_hook(), even if there was no address returned last time recv_hook() executed. This patch additionally simplifies the error handling

[PATCH v1] dns: fixed removal of DNS server

2023-04-28 Thread Renaud Métrich
t; 8< 8< This happens because the implementation is broken, it does a "add" internally. Signed-off-by: Renaud Métrich --- grub-core/net/dns.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/grub-core/net/dns.c b/grub-core

Re: efi: new 'connectefi' command

2022-09-01 Thread Renaud Métrich
t' command" Renaud. Le 8/28/22 à 21:51, Glenn Washburn a écrit : On Tue, 15 Feb 2022 14:05:20 +0100 Renaud Métrich wrote: This set of patches fixes a memory leak in 'lsefi' command and introduces a new 'connectefi pciroot|scsi' command which recursively connects

[PATCH v4 2/2] efi: new 'eficonnect' command

2022-09-01 Thread Renaud Métrich
uot;$prefix" ]; then chainloader ($prefix)/EFI/redhat/grubx64/efi ... 8< 8< ---- 8< ---- The code is easily extensible to handle other arguments in the future if needed. Signed-off-by: Renaud Métrich --- NEWS

[PATCH v4 1/2] lsefi: fixed memory leaks

2022-09-01 Thread Renaud Métrich
Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 4 1 file changed, 4 insertions(+) diff --git a/grub-core/commands/efi/lsefi.c b/grub-core/commands/efi/lsefi.c index f46ba3b49..241be79f9 100644 --- a/grub-core/commands/efi/lsefi.c +++ b/grub-core/commands/efi/lsefi.c

[PATCH v5 3/3] search: new --efidisk-only option on EFI systems

2022-03-07 Thread Renaud Métrich
s root envvar to 'md/boot_efi' which cannot be used for chainloading since there is no effective EFI device behind. Signed-off-by: Renaud Métrich --- docs/grub.texi | 5 - grub-core/commands/search.c | 19 +++ grub-core/commands/search_wrap.c |

[PATCH v5 0/3] search: new --efidisk-only option on EFI systems

2022-03-07 Thread Renaud Métrich
When using 'search' on EFI systems, we sometimes want to exclude devices that are not EFI disks (e.g. md, lvm). This is typically used when wanting to chainload when having a software raid (md) for EFI partition: with no option, 'search --file /EFI/redhat/shimx64.efi' sets root envvar to 'md/boot_e

[PATCH v5 1/3] search: fixed bug stopping iteration when --no-floppy is used

2022-03-07 Thread Renaud Métrich
When using --no-floppy and a floppy was encountered, iterate_device() was returning 1, causing the iteration to stop instead of continuing. Signed-off-by: Renaud Métrich --- grub-core/commands/search.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/grub-core/commands

[PATCH v5 2/3] search: refactor --no-floppy option to have something generic

2022-03-07 Thread Renaud Métrich
Signed-off-by: Renaud Métrich --- grub-core/commands/search.c | 8 grub-core/commands/search_wrap.c | 13 +++-- include/grub/search.h| 14 +++--- 3 files changed, 22 insertions(+), 13 deletions(-) diff --git a/grub-core/commands/search.c b/grub-core

Re: [PATCH v3 1/2] lsefi: fixed memory leaks

2022-02-25 Thread Renaud Métrich
.     Signed-off-by: Peter Jones Le 2/24/22 à 20:39, Daniel Kiper a écrit : On Tue, Feb 15, 2022 at 02:05:21PM +0100, Renaud Métrich wrote: Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 4 1 file changed, 4 insertions(+) diff --git a/grub-core/commands/efi/lsefi.c b/grub

[PATCH v4 3/3] search: new --efidisk-only option on EFI systems

2022-02-25 Thread Renaud Métrich
s root envvar to 'md/boot_efi' which cannot be used for chainloading since there is no effective EFI device behind. Signed-off-by: Renaud Métrich --- docs/grub.texi | 5 - grub-core/commands/search.c | 19 +++ grub-core/commands/search_wrap.c |

[PATCH v4 1/3] search: fixed bug stopping iteration when --no-floppy is used

2022-02-25 Thread Renaud Métrich
When using --no-floppy and a floppy was encountered, iterate_device() was returning 1, causing the iteration to stop instead of continuing. Signed-off-by: Renaud Métrich --- grub-core/commands/search.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/grub-core/commands

[PATCH v4 2/3] search: refactor --no-floppy option to have something generic

2022-02-25 Thread Renaud Métrich
Signed-off-by: Renaud Métrich --- grub-core/commands/search.c | 8 grub-core/commands/search_wrap.c | 13 +++-- include/grub/search.h| 14 +++--- 3 files changed, 22 insertions(+), 13 deletions(-) diff --git a/grub-core/commands/search.c b/grub-core

[PATCH v4 0/3] search: new --efidisk-only option on EFI systems

2022-02-25 Thread Renaud Métrich
When using 'search' on EFI systems, we sometimes want to exclude devices that are not EFI disks (e.g. md, lvm). This is typically used when wanting to chainload when having a software raid (md) for EFI partition: with no option, 'search --file /EFI/redhat/shimx64.efi' sets root envvar to 'md/boot_e

Re: [PATCH v2 2/2] lsefi: fixed memory leaks

2022-02-15 Thread Renaud Métrich
Please ignore, deprecated by "efi: new 'connectefi' command" (v3). Sorry for the mess. Le 2/14/22 à 14:25, Renaud Métrich a écrit : Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 4 1 file changed, 4 insertions(+) diff --git a/grub-core/command

Re: [PATCH v2 1/2] efi: new 'connectefi' command

2022-02-15 Thread Renaud Métrich
Please ignore, deprecated by "efi: new 'connectefi' command" (v3). Sorry for the mess. Le 2/14/22 à 14:23, Renaud Métrich a écrit : When efi.quickboot is enabled on VMWare (which is the default for hardware release 16 and later), it may happen that not all EFI devices a

[PATCH v3 2/2] efi: new 'connectefi' command

2022-02-15 Thread Renaud Métrich
lt; 8< 8< connectefi scsi unset prefix search --file --set=prefix /EFI/redhat/grubx64.efi if [ -n "$prefix" ]; then chainloader ($prefix)/EFI/redhat/grubx64/efi ... 8< 8< 8< ---- The c

[PATCH v3 1/2] lsefi: fixed memory leaks

2022-02-15 Thread Renaud Métrich
Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 4 1 file changed, 4 insertions(+) diff --git a/grub-core/commands/efi/lsefi.c b/grub-core/commands/efi/lsefi.c index d1ce99af4..b1f2d4695 100644 --- a/grub-core/commands/efi/lsefi.c +++ b/grub-core/commands/efi/lsefi.c

efi: new 'connectefi' command

2022-02-15 Thread Renaud Métrich
This set of patches fixes a memory leak in 'lsefi' command and introduces a new 'connectefi pciroot|scsi' command which recursively connects the corresponding EFI handles. This is required on VMWare with efi.quickBoot enabled when chainloading to grub on the harddisk from a network boot, otherwise

[PATCH v2 2/2] lsefi: fixed memory leaks

2022-02-14 Thread Renaud Métrich
Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 4 1 file changed, 4 insertions(+) diff --git a/grub-core/commands/efi/lsefi.c b/grub-core/commands/efi/lsefi.c index 7acba3b39..f0b9201f1 100644 --- a/grub-core/commands/efi/lsefi.c +++ b/grub-core/commands/efi/lsefi.c

[PATCH v2 1/2] efi: new 'connectefi' command

2022-02-14 Thread Renaud Métrich
In the future, if really needed, the code may be enhanced to avoid the recursion and only connect PCI devices block devices. Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 52 ++ grub-core/disk/efi/efidisk.c | 13 + grub-core/kern/ef

[PATCH v2 2/2] lsefi: fixed memory leaks

2022-02-14 Thread Renaud Métrich
From: Fedora Ninjas Signed-off-by: Renaud Métrich --- grub-core/commands/efi/lsefi.c | 4 1 file changed, 4 insertions(+) diff --git a/grub-core/commands/efi/lsefi.c b/grub-core/commands/efi/lsefi.c index 7acba3b39..f0b9201f1 100644 --- a/grub-core/commands/efi/lsefi.c +++ b/grub-core

Re: [PATCH v1] efi: make sure EFI disk controllers are connected when discovering devices

2022-02-11 Thread Renaud Métrich
Le 2/10/22 à 15:53, Ard Biesheuvel a écrit : Hello all, Please don't top post. I am not going to rearrange the text below and just reply to each chunk if it seems relevant. On Thu, 10 Feb 2022 at 14:02, Alexander Graf wrote: On 09.02.22 18:20, Renaud Métrich wrote: Hi, I observ

Re: [PATCH v1] efi: make sure EFI disk controllers are connected when discovering devices

2022-02-09 Thread Renaud Métrich
, Renaud Métrich wrote: When efi.quickboot is enabled on VMWare (which is the default for hardware release 16 and later), it may happen that not all EFI devices are connected. Due to this, browsing the devices in make_devices() just fails to find devices, in particular partitions for a given disk. This

[PATCH v1] efi: make sure EFI disk controllers are connected when discovering devices

2022-02-08 Thread Renaud Métrich
result, which is 0 (EFI_SUCCESS) or 14 (EFI_NOT_FOUND) when the handle is not a controller. Signed-off-by: Renaud Métrich --- grub-core/disk/efi/efidisk.c | 10 ++ grub-core/kern/efi/efi.c | 13 + include/grub/efi/efi.h | 5 + 3 files changed, 28 insertions

Re: [PATCH] efi: make sure EFI disk controllers are connected when, discovering devices

2022-02-08 Thread Renaud Métrich
Please ignore, resent properly using a git send-email command. Le 2/1/22 à 11:38, Renaud Métrich a écrit : When efi.quickboot is enabled on VMWare (which is the default for hardware release 16 and later), it may happen that not all EFI devices are connected. Due to this, browsing the

Re: [PATCH] search: new --efidisk-only option on EFI systems

2022-02-08 Thread Renaud Métrich
So Glenn, I'm new to Grub developer, in the past I was relying on Javier Martinez Canillas ... Just sent v3. Le 2/7/22 à 22:32, Glenn Washburn a écrit : On Mon, 7 Feb 2022 12:12:14 +0100 Renaud Métrich wrote: Please find inline the new patch integrating Glenn's comments (new &quo

[PATCH v3 2/2] search: new --efidisk-only option on EFI systems

2022-02-08 Thread Renaud Métrich
s root envvar to 'md/boot_efi' which cannot be used for chainloading since there is no effective EFI device behind. This commit also refactors handling of --no-floppy option. Signed-off-by: Renaud Métrich --- grub-core/commands/search.c | 27 +++ grub-co

[PATCH v3 1/2] search: fixed bug stopping iteration when --no-floppy is used

2022-02-08 Thread Renaud Métrich
When using --no-floppy and a floppy was encountered, iterate_device() was returning 1, causing the iteration to stop instead of continuing. Signed-off-by: Renaud Métrich --- grub-core/commands/search.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/grub-core/commands

Re: [PATCH] search: new --efidisk-only option on EFI systems

2022-02-07 Thread Renaud Métrich
typically used when wanting to chainload when having a software raid (md) for EFI partition: with no option, 'search --file /EFI/redhat/shimx64.efi' sets root envvar to 'md/boot_efi' which cannot be used for chainloading since there is no effective EFI device behind. Signed-off-by:

Re: [PATCH] search: new --efidisk-only option on EFI systems

2022-02-07 Thread Renaud Métrich
See inline, sorry for the format. Le 2/4/22 à 23:28, Glenn Washburn a écrit : On Tue, 1 Feb 2022 11:36:01 +0100 Renaud Métrich wrote: When using 'search' on EFI systems, we sometimes want to exclude devices that are not EFI disks (e.g. md, lvm). This is typically used when

[PATCH] search: new --efidisk-only option on EFI systems

2022-02-01 Thread Renaud Métrich
boot_efi' which cannot be used for chainloading since there is no effective EFI device behind. Signed-off-by: Renaud Métrich diff --git a/grub-core/commands/search.c b/grub-core/commands/search.c index ed090b3af..5e1e88643 100644 --- a/grub-core/commands/search.c +++ b/grub-core/commands/sea

[PATCH] efi: make sure EFI disk controllers are connected when, discovering devices

2022-02-01 Thread Renaud Métrich
(EFI_NOT_FOUND) when the handle is not a controller. Signed-off-by: Renaud Métrich diff --git a/grub-core/disk/efi/efidisk.c b/grub-core/disk/efi/efidisk.c index f077b5f55..0b16c3868 100644 --- a/grub-core/disk/efi/efidisk.c +++ b/grub-core/disk/efi/efidisk.c @@ -387,6 +387,16 @@ static void e

Re: [PATCH] at_keyboard: Fix keyboards that report IBM PC AT scan codes

2021-02-24 Thread Renaud Métrich
these above, the fallback mechanism is not used at all, set is detected correctly. However the mechanism is necessary for HP DL380p Gen 8 systems. Renaud. On 1/8/21 3:00 PM, Renaud Métrich wrote: Hello, It appears that the proposal works fine on all systems I could test except one: a HP DL380p

Re: [PATCH] at_keyboard: Fix keyboards that report IBM PC AT scan codes

2021-01-08 Thread Renaud Métrich
_keyboard_fallback_set" environment variable in grub.cfg that would end using a specific set if query_mode() fails. See proposed patch in attachment. Renaud. On 12/14/20 5:47 PM, Renaud Métrich wrote: Hi Vladimir, Thanks for the hint, this was obvious now. Please find attached the new pat

Re: [PATCH] at_keyboard: Fix keyboards that report IBM PC AT scan codes

2020-12-14 Thread Renaud Métrich
, Vladimir 'phcoder' Serbinenko wrote: On Fri, Dec 4, 2020 at 5:53 AM Renaud Métrich wrote: Hi, Testing the proposed patch on my old Asus N53SN in Legacy failed: as soon as at_keyboard is selected, the keys are corrupted and it's impossible to do anything. Digging into this,

Re: [PATCH] at_keyboard: Fix keyboards that report IBM PC AT scan codes

2020-12-04 Thread Renaud Métrich
uot;): commit 5e8d6963c50cd727bfb5614448f9c80743a59de1 (HEAD -> master) Author: Renaud Métrich Date: Thu Dec 3 09:43:21 2020 +0100 Force using set_scancodes() code querying the driver. This solves issues when query_mode() returns 2 but the driver continues to send keycodes suitable for set 1 (se

Non-functional at_keyboard

2020-11-02 Thread Renaud Métrich
Hi there, A french Red Hat customer reported that in Legacy mode (the code isn't used with UEFI), immediately after switching |terminal_input| to |at_keyboard|, the keyboard was just not usable due to invalid key mappings. This happens on various laptop models from various manufacturers with F