Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Discover unified kernel images on EFI #119

Closed
wants to merge 3 commits into from

Conversation

osteffenrh
Copy link

@osteffenrh osteffenrh commented Feb 23, 2023

The blscfg module implements the Boot Loader Specification for "Type 1" entries = configuration snippets.
This PR extends this module to also handle "Type 2" entries: unified kernel images (UKI) by adding
a new command "blscfg_uki". It searches for UKI in /EFI/Linux on the EFI partition and on $root
(probably /boot) and adds them to the menu.

@osteffenrh osteffenrh changed the title Discover unified kernel images Draft: Discover unified kernel images Feb 23, 2023
@osteffenrh osteffenrh changed the title Draft: Discover unified kernel images Draft: Discover unified kernel images on EFI Feb 23, 2023
@osteffenrh osteffenrh marked this pull request as draft February 23, 2023 16:13
@osteffenrh osteffenrh closed this Feb 23, 2023
@osteffenrh osteffenrh reopened this Feb 27, 2023
@osteffenrh osteffenrh marked this pull request as ready for review February 27, 2023 08:01
@osteffenrh osteffenrh changed the title Draft: Discover unified kernel images on EFI Discover unified kernel images on EFI Feb 27, 2023
@osteffenrh
Copy link
Author

Reopened and ready for comments. Sorry for the mess 😅

@osteffenrh
Copy link
Author

Improved os-rel parsing.

@osteffenrh
Copy link
Author

The Boot Loader Interface patches are now upstream:
https://lists.gnu.org/archive/html/grub-devel/2023-05/msg00156.html

Any comments on this PR are very welcome.

Copy link
Contributor

@nfrayer nfrayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

grub-core/commands/blscfg.c Show resolved Hide resolved
grub-core/commands/blscfg.c Outdated Show resolved Hide resolved
This avoids compiler warnings in case bls_add_keyval() is called
with const char* as arguments.

Signed-off-by: Oliver Steffen <osteffen@redhat.com>
Add a new command "blscfg_uki" to the blscfg module.  It looks for
unified kernel images (UKI) in the locations defined by the Boot Loader
Specification.  Additionally check on the device indicated by the $root
variable.  Add all UKIs found to the menu.

The UKI are booted using the 'chainloader' command on EFI.

Signed-off-by: Oliver Steffen <osteffen@redhat.com>
@osteffenrh osteffenrh changed the base branch from fedora-39 to fedora-40 October 24, 2023 13:53
@osteffenrh
Copy link
Author

@vathpela ping?

GitHub still shows one change request as open, although I marked it as resolved. Strange.

Rebased and updated PR target branch.

@osteffenrh
Copy link
Author

@vathpela ping?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants