GRUB needs to support 64-bit efi linear frame buffer address

Bug #1785033 reported by Ivan Hu on 2018-08-02
20
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HWE Next
High
Unassigned
OEM Priority Project
Critical
Unassigned
grub2 (Ubuntu)
Status tracked in Cosmic
Trusty
Undecided
Unassigned
Xenial
Undecided
Unassigned
Bionic
Undecided
Unassigned
Cosmic
Undecided
Unassigned
grub2-signed (Ubuntu)
Status tracked in Cosmic
Trusty
Undecided
Unassigned
Xenial
Undecided
Unassigned
Bionic
Undecided
Unassigned
Cosmic
Undecided
Unassigned

Bug Description

[Rationale]
More firmwares support above 4G mmio configureation and the EFI Graphics Output Protocol can return a 64-bit linear frame buffer address have been implemented in some firmware/BIOS. Grub2 currently only pass 32-bit framebuffer base to kernel. The Linux kernel has already added support to handle 64-bit linear framebuffer address in the efifb driver now. So GRUB2 should support 64-bit EFI linear frame buffer address.

[Impact]
Some machines block booting with firmware/bios implemented 64-bit EFI linear frame buffer address,due to Grub passing incorrect(only 32-bit) EFI linear frame buffer.

[Test cases]

Need Bios/Firmware support,
1) Make sure the machine with the Bios implemented 64-bit EFI linear frame buffer address. some machine need to enable above 4G mmio on bios setup menu.
2) Boot up.

[Solution]
A patch has been committed and accepted by maintainer
http://git.savannah.gnu.org/cgit/grub.git/commit/?id=886edba8770ccbc3def0af2a7d6b346d00d0af2f

[Regression Potential]
Minimal, it's unlikely completing the whole framebuffer address will affect those which hasn't used above 32-bits framebuffer address.

Ivan Hu (ivan.hu) wrote :

Attach the fix patch which has been accepted and committed.

Ivan Hu (ivan.hu) wrote :

The patch on comment#1 have been tested can fix the block booting issue on some Dell machines.

summary: - GRUB need to support 64-bit efi linear frame buffer address
+ GRUB needs to support 64-bit efi linear frame buffer address

The attachment "0001-i386-linux-Add-support-for-ext_lfb_base.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Ivan Hu (ivan.hu) on 2018-08-03
description: updated
description: updated
Ivan Hu (ivan.hu) on 2018-08-16
description: updated
tags: added: originate-from-1758231 somerville
Changed in hwe-next:
status: New → In Progress
importance: Undecided → High
Changed in oem-priority:
importance: Undecided → Critical
status: New → Triaged
Ivan Hu (ivan.hu) wrote :

Attached the debdiff on xenial

Timo Aaltonen (tjaalton) wrote :

trusty/xenial/bionic uploaded

An upload of grub2 to bionic-proposed has been rejected from the upload queue for the following reason: "as discussed on IRC we aren't ready to flip the switch yet on kernel sig enforcement".

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package grub2-signed - 1.104

---------------
grub2-signed (1.104) cosmic; urgency=medium

  * Rebuild against grub2 2.02+dfsg1-5ubuntu2. (LP: #1785033)

 -- Mathieu Trudel-Lapierre <email address hidden> Wed, 05 Sep 2018 14:32:39 -0400

Changed in grub2-signed (Ubuntu Cosmic):
status: New → Fix Released
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package grub2 - 2.02+dfsg1-5ubuntu2

---------------
grub2 (2.02+dfsg1-5ubuntu2) cosmic; urgency=medium

  * debian/patches/add_ext_lfb_base_support.patch: i386/linux: Add support for
    ext_lfb_base. (LP: #1785033)

 -- Mathieu Trudel-Lapierre <email address hidden> Wed, 05 Sep 2018 14:29:04 -0400

Changed in grub2 (Ubuntu Cosmic):
status: New → Fix Released
cktenn (cktenn) on 2018-10-11
tags: added: beaver-breckenridge-mlk-cfl-p
cktenn (cktenn) on 2018-10-11
tags: removed: beaver-breckenridge-mlk-cfl-p

Hello Ivan, or anyone else affected,

Accepted grub2 into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/grub2/2.02-2ubuntu8.7 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in grub2 (Ubuntu Bionic):
status: New → Fix Committed
tags: added: verification-needed verification-needed-bionic
Changed in grub2-signed (Ubuntu Bionic):
status: New → Fix Committed
Łukasz Zemczak (sil2100) wrote :

Hello Ivan, or anyone else affected,

Accepted grub2-signed into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/grub2-signed/1.93.8 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Bug attachments