[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Regression] efi: Don't display a uefi-firmware entry if it's not su
From: |
Javier Martinez Canillas |
Subject: |
Re: [Regression] efi: Don't display a uefi-firmware entry if it's not supported |
Date: |
Tue, 30 Aug 2022 23:34:10 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 |
On 8/30/22 23:14, Mike Gilbert wrote:> On Tue, Aug 30, 2022 at 4:16 PM Robbie
Harwood <rharwood@redhat.com> wrote:
>> Why doesn't grub on the MBR get updated when you install a new grub
>> package? That seems like the real issue here - what am I missing?
>
> I think the grub project tries to maintain config file compatibility
> with older versions so that users can use new config files even if an
> older grub version is still installed in /boot/grub.
>
In my opinion the grub configuration scheme is already too complex (a script
that executes a set of scripts that generate sections of a script executed
on boot) to also throw backward compatibility in the mix.
I agree with Robbie here, either don't re-generate your GRUB config file in
a installed system or update the GRUB core image in the embedding gap too.
Anything but that feels like an uphill battle to me and would just add much
more complexity.
>> Regardless, if we can't count on fwsetup being updated, then I think we
>> need to go back to the original version of my patch which doesn't have
>> --is-supported.
>
Indeed. The original patch didn't use that option, it was added to address
some feedback that Robbie had on an earlier version.
> You could add a feature flag, which causes grub-core to set an
> environment variable when a new feature is supported. See the features
> array in grub-core/normal/main.c.
>
> You would then check for this feature flag in the grub.d snippet
> before calling fwsetup --is-supported.
>
Please don't. As mentioned, I think we should aim to simplify the grub.cfg
instead of making it more complicated.
--
Best regards,
Javier Martinez Canillas
Core Platforms
Red Hat
Re: [Regression] efi: Don't display a uefi-firmware entry if it's not supported, Mihai Moldovan, 2022/08/31
Re: [Regression] efi: Don't display a uefi-firmware entry if it's not supported, Dimitri John Ledkov, 2022/08/31