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.

Sign up for GitHub

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

Jump to bottom

Fixed secure boot fallback setup #1879

Merged
merged 1 commit into from
Jul 14, 2021
Merged

Fixed secure boot fallback setup #1879

merged 1 commit into from
Jul 14, 2021

Conversation

schaefi
Copy link
Collaborator

@schaefi schaefi commented Jul 6, 2021

Make sure MokManager gets copied. This Fixes bsc#1187515

@schaefi
Copy link
Collaborator Author

schaefi commented Jul 6, 2021 •
edited
Loading

@Conan-Kudo I think MokManager.efi is a stable name across distributions and afaik also part of the EFI standard. So do you agree with me that we don't have to differentiate the name between the distros ?

Thanks

@Vogtinator
Copy link
Collaborator

@Conan-Kudo I think MokManager.efi is a stable name across distributions and afaik also part of the EFI standard. So do you agree with me that we don't have to differentiate the name between the distros ?

The name of MokManager.efi is configured during build of shim, the default is actually mm$(ARCH).efi...
openSUSE uses MokManager.efi (without any arch suffix), no idea about others.

@Conan-Kudo
Copy link
Member

Conan-Kudo commented Jul 7, 2021 •
edited
Loading

We should probably fix MokManager.efi to go back to the default name with the arch in it... Both Fedora and Ubuntu use the upstream mm<EFIARCH>.efi name.

Conan-Kudo
Conan-Kudo requested changes Jul 7, 2021
View reviewed changes
kiwi/bootloader/config/grub2.py Outdated Show resolved Hide resolved
@schaefi
Fixed secure boot fallback setup
218cfa1 
Make sure MokManager gets copied. The name and location of
the mok manager is distribution specific in the same way as
the shim loader. Thus we need to apply a similar concept
for looking it up. This Fixes bsc#1187515
@schaefi schaefi merged commit dce5a73 into master Jul 14, 2021
@schaefi schaefi deleted the fix_missing_MokManager branch July 14, 2021 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Conan-Kudo Conan-Kudo Conan-Kudo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@schaefi @Vogtinator @Conan-Kudo