Re: Running bootstrap script fails on Debian unstable

2023-02-20 Thread Glenn Washburn
On Mon, 20 Feb 2023 18:50:05 +0100 John Paul Adrian Glaubitz wrote: > Hi Glenn! > > On Mon, 2023-02-20 at 11:34 -0600, Glenn Washburn wrote: > > Does the 2.06 release build with the new autoconf? It looks like > > sid is using 2.06, I don't see any special patches for > > configure.ac, and I

Re: [PATCH v6 09/14] gdb: Add more support for debugging on EFI platforms

2023-02-20 Thread Glenn Washburn
On Mon, 20 Feb 2023 15:06:46 -0500 Robbie Harwood wrote: > Glenn Washburn writes: > > > If the configure option --enable-efi-debug is given, then enable the > > printing early in EFI startup of the command needed to load symbols > > for the GRUB EFI kernel. This is needed because EFI firmware

[PATCH v3] loader/i386/linux.c Correct wrong initrd address for debug

2023-02-20 Thread Jeremy Szu
The 'addr' is used to request the memory with specific ranges but the real loadable address come from the relocator. Thus, print the final retrieved addresses (virtual and physical) for initrd. v2: append SOB v3: fix wrong code base Signed-off-by: Jeremy Szu --- grub-core/loader/i386/linux.c

Re: [PATCH v6 09/14] gdb: Add more support for debugging on EFI platforms

2023-02-20 Thread Robbie Harwood
Glenn Washburn writes: > If the configure option --enable-efi-debug is given, then enable the > printing early in EFI startup of the command needed to load symbols for > the GRUB EFI kernel. This is needed because EFI firmware determines where > to load the GRUB EFI at runtime, and so the

[PATCH v2 4/7] util/grub.d: activate bli module on EFI

2023-02-20 Thread Oliver Steffen
Add a new configuration drop-in file that loads the boot-loader-interface (bli) module and runs the command in case we are booting on the EFI platform. Signed-off-by: Oliver Steffen --- util/grub.d/25_boot_loader_interface.in | 34 + 1 file changed, 34 insertions(+)

[PATCH v2 1/7] efi: add grub_efi_set_variable_with_attributes

2023-02-20 Thread Oliver Steffen
Add a function to the EFI module that allows setting EFI variables with specific attributes. This is useful for marking variables as volatile, for example. Signed-off-by: Oliver Steffen --- grub-core/kern/efi/efi.c | 19 +-- include/grub/efi/efi.h | 6 ++ 2 files

[PATCH v2 2/7] efi: check for integer overflow in string conversion

2023-02-20 Thread Oliver Steffen
Check for integer overflow when converting the name of the EFI variable to UTF16 in grub_efi_set_variable_with_attributes(). Signed-off-by: Oliver Steffen --- grub-core/kern/efi/efi.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/grub-core/kern/efi/efi.c b/grub-core/kern/efi/efi.c

[PATCH v2 6/7] commands/probe: use grub_gpt_part_guid_snprint

2023-02-20 Thread Oliver Steffen
Use the new function for printing the partition guid. Signed-off-by: Oliver Steffen --- grub-core/commands/probe.c | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/grub-core/commands/probe.c b/grub-core/commands/probe.c index 9a80ea54f..3fe50d11e 100644 ---

[PATCH v2 5/7] partmap/gpt: add print function for guids

2023-02-20 Thread Oliver Steffen
Code to print gpt patition guids has been duplicated in multiple places. Add a common function for that. Signed-off-by: Oliver Steffen --- grub-core/partmap/gpt.c | 13 + include/grub/gpt_partition.h | 6 ++ 2 files changed, 19 insertions(+) diff --git

[PATCH v2 0/7] Add basic Boot Loader Interface support

2023-02-20 Thread Oliver Steffen
This is a step towards supporting unified kernel images (UKI) in Grub. Add a new module named bli, which provides a command with the same name. It implements a small but quite useful part of the Boot Loader Interface [0]. This interface uses EFI variables for communication between the boot

[PATCH v2 3/7] Add a module for the Boot Loader Interface

2023-02-20 Thread Oliver Steffen
Add a new module named boot_loader_interface, which provides a command with the same name. It implements a small but quite useful part of the Boot Loader Interface [0]. This interface uses EFI variables for communication between the boot loader and the operating system. This module sets two EFI

[PATCH v2 7/7] commands/bli: use grub_gpt_part_guid_snprint

2023-02-20 Thread Oliver Steffen
Use the new function for printing the partition guid. Signed-off-by: Oliver Steffen --- grub-core/commands/bli.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/grub-core/commands/bli.c b/grub-core/commands/bli.c index 10993222d..160cbdb34 100644 ---

Re: Running bootstrap script fails on Debian unstable

2023-02-20 Thread John Paul Adrian Glaubitz
Hi Glenn! On Mon, 2023-02-20 at 11:34 -0600, Glenn Washburn wrote: > Does the 2.06 release build with the new autoconf? It looks like sid is > using 2.06, I don't see any special patches for configure.ac, and I > presume that the sid grub binary is built with the new autoconf. If > this all is

Re: Running bootstrap script fails on Debian unstable

2023-02-20 Thread Glenn Washburn
Hi Adrian, On Mon, 20 Feb 2023 15:26:35 +0100 John Paul Adrian Glaubitz wrote: > Hi Mihai! > > On Mon, 2023-02-20 at 15:01 +0100, Mihai Moldovan wrote: > > * On 2/20/23 10:28, John Paul Adrian Glaubitz wrote: > > > I wanted to start the current GRUB master branch on various > > > architectures

Re: [PATCH v1 2/2] Add a module for the Boot Loader Interface

2023-02-20 Thread Oliver Steffen
Thank you for the comments, Daniel. Quoting Daniel Kiper (2023-02-15 19:27:03) > On Mon, Jan 16, 2023 at 12:40:53PM +0100, Oliver Steffen wrote: > > Add a new module named boot_loader_interface, which provides a command > > I would prefer something shorter than boot_loader_interface. bli? Then

Re: Running bootstrap script fails on Debian unstable

2023-02-20 Thread Mihai Moldovan
* On 2/20/23 15:26, John Paul Adrian Glaubitz wrote: > [...] >> >> These are standard macros, so if autoconf cannot find them, something really >> weird must be going on. You'll probably have to strace autoreconf to figure >> out >> what it's reading. Unfortunately, debugging autotools is a

Re: Running bootstrap script fails on Debian unstable

2023-02-20 Thread John Paul Adrian Glaubitz
Hi Mihai! On Mon, 2023-02-20 at 15:01 +0100, Mihai Moldovan wrote: > * On 2/20/23 10:28, John Paul Adrian Glaubitz wrote: > > I wanted to start the current GRUB master branch on various architectures > > and it > > turns out that the bootstrap script currently fails on Debian unstable due > >

Re: Running bootstrap script fails on Debian unstable

2023-02-20 Thread Mihai Moldovan
* On 2/20/23 10:28, John Paul Adrian Glaubitz wrote: > I wanted to start the current GRUB master branch on various architectures and > it > turns out that the bootstrap script currently fails on Debian unstable due to > the autoconf syntax changes: > > running: AUTOPOINT=true LIBTOOLIZE=true

Running bootstrap script fails on Debian unstable

2023-02-20 Thread John Paul Adrian Glaubitz
Hello! I wanted to start the current GRUB master branch on various architectures and it turns out that the bootstrap script currently fails on Debian unstable due to the autoconf syntax changes: running: AUTOPOINT=true LIBTOOLIZE=true autoreconf --verbose --install --force -I m4 --no-recursive