[PATCH v2 RESEND] util/grub.d/30_os-prober.in: Conditionally show or hide chain and efi menu entries

2025-02-24 Thread Pascal Hambourg
legacy BIOS boot, in order to generate a config file which can be used with either BIOS or EFI boot. Signed-off-by: Pascal Hambourg --- Note: I did not change indentation to keep the patch minimal. v2: - show 'chain' menu entries if platform=pc instead of platform!=efi - add explanatory co

Re: [SECURITY PATCH 49/73] fs: Disable many filesystems under lockdown

2025-02-21 Thread Pascal Hambourg
On 20/02/2025 at 17:43, Daniel Kiper wrote: > On Wed, Feb 19, 2025 at 09:15:50AM +0100, Petr Řehák wrote: >> >> why is there a lockdown for the NTFS file system, please? Is it vulnerable >> as well when no CVE exists for it? We are developers of computer-aided >> assistive technology for blind and

[PATCH v2 RESEND] util/grub.d/30_os-prober.in: Fix GRUB_OS_PROBER_SKIP_LIST for non-EFI

2025-02-21 Thread Pascal Hambourg
comma and semicolon as separator Fixes: 55e706c9 (Add GRUB_OS_PROBER_SKIP_LIST to selectively skipping systems) Signed-off-by: Pascal Hambourg --- Changes in v2: - enforce UUID boundaries with correct separators - warn if @/dev/* is found in GRUB_OS_PROBER_SKIP_LIST - update documentation -

Re: [PATCH RESEND] grub-install: force journal draining to ensure data integrity

2025-01-20 Thread Pascal Hambourg
Le 21/01/2025 à 07:25, Michael Chang via Grub-devel a écrit : In XFS, the system would end up in unbootable state if an abrupt power off after grub-install is occuring. It can be easily reproduced with. grub-install /dev/vda; reboot -f The grub error would show many different kinds of corrup

Re: [PATCH v2] util/grub.d/30_os-prober.in: Fix GRUB_OS_PROBER_SKIP_LIST for non-EFI

2024-09-17 Thread Pascal Hambourg
On 08/05/2024 at 01:03, Pascal Hambourg wrote: GRUB documentation states: ‘GRUB_OS_PROBER_SKIP_LIST’ List of space-separated FS UUIDs of filesystems to be ignored from os-prober output. For efi chainloaders it’s @ But the actual behaviour does not match this description

Re: [PATCH] Change "efi" to "EFI" in grub-mkrescue for secure boot

2024-09-08 Thread Pascal Hambourg
On 07/09/2024 at 10:30, Thomas Schmitt via Grub-devel wrote: Pascal Hambourg wrote: When booting from a EFI partition on a regular disk, then $root is set to the EFI partition (hdX,Y). When booting from a EFI El Torito image on a CD, then $root is set to the whole CD (cdX), and the El Torito

Re: [PATCH] Change "efi" to "EFI" in grub-mkrescue for secure boot

2024-09-06 Thread Pascal Hambourg
On 06/09/2024 at 15:15, Askar Safin wrote: While testing this script I noticed this: if I boot from disk (i. e. using "-drive" Qemu option), then "grub.cfg" is read from ESP, i. e. FAT. But if I boot from CD (i. e. using "-cdrom" Qemu option), then "grub.cfg" is read from main .iso partition, i.

EFI El Torito image on ISO9660 (was: [PATCH] bli: Fix crash in get_part_uuid)

2024-07-16 Thread Pascal Hambourg
On 16/07/2024 at 08:55, Michael Chang via Grub-devel wrote: 4. When booting from an ISO, the ESP is treated as an El Torito image in the boot catalog. It is therefore presented by firmware and interpreted by grub as a disk (cd0). In my limited experience, (cd0) is the whole CD/DVD/BD a

[PATCH v2][Bugfix] util/grub.d/00_header.in: quote background image pathname in output

2024-05-19 Thread Pascal Hambourg
This is required if the pathname contains spaces or grub shell metacharacters, else the generated config file check will fail. Signed-off-by: Pascal Hambourg --- v2: Correct subject util/grub.d/00_header.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/grub.d

[PATCH][Bugfix] debian/grub.d/05_debian_theme: quote background image pathname in output

2024-05-19 Thread Pascal Hambourg
This is required if the pathname contains spaces or grub shell metacharacters, else the generated config file check will fail. Signed-off-by: Pascal Hambourg --- util/grub.d/00_header.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/grub.d/00_header.in b/util/grub.d

[PATCH v2] util/grub.d/30_os-prober.in: Fix GRUB_OS_PROBER_SKIP_LIST for non-EFI

2024-05-08 Thread Pascal Hambourg
comma and semicolon as separator Fixes: 55e706c9 (Add GRUB_OS_PROBER_SKIP_LIST to selectively skipping systems) Signed-off-by: Pascal Hambourg --- Changes in v2: - enforce UUID boundaries with correct separators - warn if @/dev/* is found in GRUB_OS_PROBER_SKIP_LIST - update documentation -

Re: [PATCH v2] util/grub.d/30_os-prober.in: Conditionally show or hide chain and efi menu entries

2024-05-01 Thread Pascal Hambourg
Any feedback on this ? On 19/01/2024 at 16:25, Pascal Hambourg wrote: On systems which support multiple boot platforms such as BIOS or EFI, it makes no sense to show menu entries which are not supported by the current boot platform. Menu entries generated from os-prober 'chain' boo

Re: [PATCH] util/grub.d/30_os-prober.in: Fix GRUB_OS_PROBER_SKIP_LIST for non-EFI

2024-05-01 Thread Pascal Hambourg
On 27/04/2024 at 08:32, Glenn Washburn wrote: On Fri, 19 Jan 2024 11:28:55 +0100 Pascal Hambourg wrote: GRUB documentation states: 'GRUB_OS_PROBER_SKIP_LIST' List of space-separated FS UUIDs of filesystems to be ignored from os-prober output. For efi chainloaders it's @

Re: [PATCH] util/grub.d/30_os-prober.in: Fix GRUB_OS_PROBER_SKIP_LIST for non-EFI

2024-04-11 Thread Pascal Hambourg
Friendly ping. Any comments about this ? On 19/01/2024 at 11:28, Pascal Hambourg wrote: GRUB documentation states: 'GRUB_OS_PROBER_SKIP_LIST' List of space-separated FS UUIDs of filesystems to be ignored from os-prober output. For efi chainloaders it's @ But the actual

Re: Idea: Improve ext2 to allow for embedding

2024-03-13 Thread Pascal Hambourg
On 13/03/2024 at 16:18, Dr. Tilmann Bubeck wrote: Yes, thats exactly the use case: I always have multiple linux installations (ubuntu and fedora) in two seperate partitions. To stay compatible with their individual kernel and grub updates, I always install a second GRUB into the root parition o

Re: Idea: Improve ext2 to allow for embedding

2024-03-13 Thread Pascal Hambourg
On 13/03/2024 at 11:25, Mate Kukri wrote: Do you have a proposed use-case for this in mind? On MBR disks there is usually enough space for core.img before the first partition. On GPT you can simply create a so-called "BIOS boot partition" to store core.img. A use case could be when you do not

[PATCH v2] util/grub.d/30_os-prober.in: Conditionally show or hide chain and efi menu entries

2024-01-19 Thread Pascal Hambourg
gacy BIOS boot. Signed-off-by: Pascal Hambourg --- Note: I did not change indentation to keep the patch minimal. v2: - show 'chain' menu entries if platform=pc instead of platform!=efi - add explanatory comments in generated config util/grub.d/30_os-prober.in | 6 ++ 1 file cha

[PATCH v2] util/grub.d/30_os-prober.in: Skip drivemap for Windows 8 to 19, Server 2012 and later

2024-01-19 Thread Pascal Hambourg
version 20. Signed-off-by: Pascal Hambourg --- v2: - Harden the version check to exclude Windows 1.* (even though os-prober does not detect it) util/grub.d/30_os-prober.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/grub.d/30_os-prober.in b/util/grub.d/30_os-prober.in

[PATCH] util/grub.d/30_os-prober.in: Fix GRUB_OS_PROBER_SKIP_LIST for non-EFI

2024-01-19 Thread Pascal Hambourg
GRUB_OS_PROBER_SKIP_LIST="@" will ignore non-EFI boot loaders detected on filesystem even though the goal is to ignore a given EFI boot loader only. However I think this is desirable to preserve the behaviour of existing setups which use GRUB_OS_PROBER_SKIP_LIST="@". Signe

Re: [PATCH] util/grub.d/30_os-prober.in: Conditionally show or hide chain and efi menu entries

2024-01-17 Thread Pascal Hambourg
On 17/01/2024 at 17:48, Vladimir 'phcoder' Serbinenko wrote: Le mer. 17 janv. 2024, 18:10, Pascal Hambourg a écrit : On 17/01/2024 at 04:15, Vladimir 'phcoder' Serbinenko wrote: +if [ "\$grub_platform" != "efi" ]; then This is not the right check.

Re: [PATCH] util/grub.d/30_os-prober.in: Conditionally show or hide chain and efi menu entries

2024-01-17 Thread Pascal Hambourg
On 17/01/2024 at 04:15, Vladimir 'phcoder' Serbinenko wrote: onstr="$(gettext_printf "(on %s)" "${DEVICE}")" cat << EOF +if [ "\$grub_platform" != "efi" ]; then This is not the right check. Only "pc" platform supports chainloadin boot sector. All other: coreboot, qemu, emu,

Re: [PATCH] util/grub.d/30_os-prober.in: Skip drivemap for Windows 8 to 19, Server 2012 and later

2024-01-17 Thread Pascal Hambourg
On 17/01/2024 à 04:17, Vladimir 'phcoder' Serbinenko wrote: On Sat, Jan 6, 2024 at 6:38 PM Pascal Hambourg wrote: If Windows Vista, Seven and Server 2008 do not need drivemap, then later versions using bootmgr too should not need it either. (...) case ${LONGNAME} in -

[PATCH] util/grub.d/30_os-prober.in: Conditionally show or hide chain and efi menu entries

2024-01-06 Thread Pascal Hambourg
'efi' menu entries only if booted in EFI mode. This will allow os-prober to report both EFI and BIOS legacy boot loaders regardless of the current boot mode on systems which support both EFI and BIOS/legacy boot. Note: I did not change indentation to keep the patch minimal. Signed-off

[PATCH] util/grub.d/30_os-prober.in: Skip drivemap for Windows 8 to 19, Server 2012 and later

2024-01-06 Thread Pascal Hambourg
before version 20. Signed-off-by: Pascal Hambourg --- util/grub.d/30_os-prober.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/grub.d/30_os-prober.in b/util/grub.d/30_os-prober.in index 656301eaf..8e5cbb50e 100644 --- a/util/grub.d/30_os-prober.in +++ b/util/grub.d