Re: [PATCH 1/3] util/grub.d/00_header.in: Disable loading all_video for EFI

On Wed, Nov 05, 2025 at 08:47:54PM -0600, Andrew Hamilton wrote:
> Loading all_video for EFI can cause video issues in some cases
> since bochs / cirrus may conflict with them. Change default

s#bochs / cirrus may conflict with them#GRUB Bochs/Cirrus drivers may conflict with native EFI drivers#

> behavior for EFI to only load EFI specific video modules.
> Also include a new environment variable to restore the old
> behavior if needed.
>
> Fixes: https://savannah.gnu.org/bugs/index.php?66200
>
> Signed-off-by: Andrew Hamilton <[email protected]>
> ---
>  util/grub.d/00_header.in | 16 +++++++++++++++-
>  1 file changed, 15 insertions(+), 1 deletion(-)
>
> diff --git a/util/grub.d/00_header.in b/util/grub.d/00_header.in
> index 9d36feda3..a4a516aec 100644
> --- a/util/grub.d/00_header.in
> +++ b/util/grub.d/00_header.in
> @@ -124,10 +124,24 @@ if [ -n "${GRUB_VIDEO_BACKEND}" ]; then
>    insmod ${GRUB_VIDEO_BACKEND}
>  EOF
>  else
> +# For EFI, use EFI video by default to avoid conflict between

s/EFI video/EFI video drivers only/

> +# bochs / cirrus and EFI video. If GRUB_FORCE_EFI_ALLVIDEO is

s#bochs / cirrus and EFI video#GRUB Bochs/Cirrus and native EFI drivers#

... or something like that...

> +# set/true then defer back to all_video even for EFI.
> +if [ "${GRUB_FORCE_EFI_ALLVIDEO}" = "1" ]; then

s/GRUB_FORCE_EFI_ALLVIDEO/GRUB_FORCE_EFI_ALL_VIDEO/

... to make naming consistent with currently existing names...

And I would merge all three patches into one.

Daniel

_______________________________________________
Grub-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/grub-devel