Skip to content

Replace Linux kernel BDS patch with linuxboot.efi BDS implementation#32

Merged
rminnich merged 26 commits intolinuxboot:masterfrom
osresearch:bds-pr
Jan 8, 2020
Merged

Replace Linux kernel BDS patch with linuxboot.efi BDS implementation#32
rminnich merged 26 commits intolinuxboot:masterfrom
osresearch:bds-pr

Conversation

@osresearch
Copy link
Contributor

@osresearch osresearch commented Aug 9, 2018

This removes a large set of patches (currently in heads/patches/linux-4.9.80/0000-efi_bds.patch) and replaces this code with a DXE module that does the same thing. Keeping these patches updated to new kernel versions has slowed our ability to track kernel updates.

The advantage of doing this in the linuxboot.efi DXE module is that we can boot a (relatively) unmodified Linux kernel, we add flexibility to finding the initrd in the ROM, we can enable flash regions on systems that have additoinal free space, and we provide a way to pass arguments to the kernel from UEFI NVRAM variables.

Merging this will require coordinating with the Heads tree since it will need to replace its patch and change all of the kernel configurations to use CONFIG_EFI_STUB, but not CONFIG_EFI_STUB_BDS.

This has been tested with BOARD=qemu-linuxboot, but not on real hardware yet.

@vejmarie
Copy link
Contributor

How relevant is dxe/README.md within this PR context ?

@lsiudut
Copy link

lsiudut commented Jan 29, 2019

Hey guys. We just tested in on the real hardware and it actually works. Just though you may want to know :).

Edit: by this I mean the linuxboot DXE. Kernel was attached separately with appropriate GUID.

@insomniacslk
Copy link

ping @osresearch , any plan to merge this?

@bofhbug
Copy link

bofhbug commented Nov 6, 2019

ping @rminnich @vejmarie @osresearch any plan to merge this?

@rminnich rminnich merged commit b60259c into linuxboot:master Jan 8, 2020
@JulienVdG JulienVdG mentioned this pull request Feb 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants