From: Andrew Hamilton <adhamilt@gmail.com>
To: grub-devel@gnu.org
Cc: daniel.kiper@oracle.com, safinaskar@zohomail.com,
benh@debian.org, bluca@debian.org, kibi@debian.org,
Andrew Hamilton <adhamilt@gmail.com>
Subject: [PATCH 0/3] Update 00_header to load efi video on EFI platforms
Date: Wed, 5 Nov 2025 20:47:53 -0600 [thread overview]
Message-ID: <20251106024756.7050-1-adhamilt@gmail.com> (raw)
This is a new attempt to fix the issue with video corruption that can
occur in some cases on EFI when all_video is loaded. Previously, the
proposed change was to disable building video_bochs and video_cirrus
for EFI completely, however there was concern around this approach.
Previous thread:
https://lists.gnu.org/archive/html/grub-devel/2025-09/msg00265.html
This new approach modifies the 00_header script to default to only
load efi_gop and efi_uga for EFI platforms. A new env variable is added
to force the old behavior of loading all_video for EFI if desired.
There is likely an additional change required on distros that
bundle all_video into the signed EFI GRUB image, such as Debian:
https://sources.debian.org/src/grub2/2.14~git20250718.0e36779-2/debian/build-efi-images
There is some additional analysis from the Qemu team in this thread:
https://gitlab.com/qemu-project/qemu/-/issues/2562
This is an issue on some major distros such as Debian on EFI
when running under Qemu and potentially other scenarios.
There is some discussion on the Debian side in this thread:
https://salsa.debian.org/kernel-team/linux/-/merge_requests/1453
This relates to this discussion from grub-devel also:
https://lists.gnu.org/archive/html/grub-devel/2025-09/msg00244.html
Fixes: https://savannah.gnu.org/bugs/index.php?66200
Andrew Hamilton (3):
util/grub.d/00_header.in: Disable loading all_video for EFI
util/grub-mkconfig: Add new environment variable
docs: Document new GRUB_FORCE_EFI_ALLVIDEO variable
docs/grub.texi | 7 +++++++
util/grub-mkconfig.in | 3 ++-
util/grub.d/00_header.in | 16 +++++++++++++++-
3 files changed, 24 insertions(+), 2 deletions(-)
--
2.43.0
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel
next reply other threads:[~2025-11-06 2:49 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-06 2:47 Andrew Hamilton [this message]
2025-11-06 2:47 ` [PATCH 1/3] util/grub.d/00_header.in: Disable loading all_video for EFI Andrew Hamilton
2025-11-06 13:06 ` Daniel Kiper
2025-11-06 2:47 ` [PATCH 2/3] util/grub-mkconfig: Add new environment variable Andrew Hamilton
2025-11-06 2:47 ` [PATCH 3/3] docs: Document new GRUB_FORCE_EFI_ALLVIDEO variable Andrew Hamilton
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20251106024756.7050-1-adhamilt@gmail.com \
--to=adhamilt@gmail.com \
--cc=benh@debian.org \
--cc=bluca@debian.org \
--cc=daniel.kiper@oracle.com \
--cc=grub-devel@gnu.org \
--cc=kibi@debian.org \
--cc=safinaskar@zohomail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).