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

Use GRUB2 legacy loader only on some Intel Atom boards specifically #3498

Merged
merged 2 commits into from
Jul 31, 2024

Conversation

sairon
Copy link
Member

@sairon sairon commented Jul 30, 2024

Previous revert of GRUB2 change that introduced usage of the generic EFI loader for all x86 boards in #3324 caused regressions, the one confirmed is #3348. This commit adds a specific patch that identifies the broken platforms based on SMBIOS data gathered in #3305 and falls back to the legacy loader there.

Tested on Intel D525MW (falls back) and QEMU (no fallback).

Also enable the smbios module for future debugging purposes.

sairon added 2 commits July 30, 2024 18:38
Previous revert of GRUB2 change that introduced usage of the generic EFI loader
for all x86 boards in #3324 caused regressions, the one confirmed is #3348.
This commit adds a specific patch that identifies the broken platforms based
on SMBIOS data gathered in #3305 and falls back to the legacy loader there.

Tested on Intel D525MW (falls back) and QEMU (no fallback).
Having smbios command in GRUB can help in future debugging, e.g. to add more
CPUs that should use the linux loader fallback.
@sairon sairon added board/generic-x86-64 Generic x86-64 Boards (like Intel NUC) grub2 labels Jul 30, 2024
@sairon sairon requested a review from agners July 30, 2024 16:47
@sairon sairon merged commit a32974e into dev Jul 31, 2024
2 checks passed
@sairon sairon deleted the grub-nm10-use-legacy-loader branch July 31, 2024 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
board/generic-x86-64 Generic x86-64 Boards (like Intel NUC) cla-signed grub2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants