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

nixos/iso-image: fix 32bit UEFI boot #47196

Merged
merged 1 commit into from Sep 23, 2018
Merged

Conversation

xeji
Copy link
Contributor

@xeji xeji commented Sep 22, 2018

Motivation for this change

The UEFI iso image for i686 didn't boot. Because the default boot file was incorrectly named bootx32.efi, the firmware didn't find it and dropped down to the UEFI shell.

This caused two tests to fail: tests.boot.uefiUsb.i686-linux and tests.boot.uefiCdrom.i686-linux

The correct name for 32bit is bootia32.efi (which is a little inconsistent with bootx64.efi for 64bit.)

ZHF #45960, please backport.

Things done
  • the two tests pass now

UEFI iso image for i686 didn't boot because the
default boot file was incorrectly named bootx32.efi.
The correct name is bootia32.efi.
@xeji
Copy link
Contributor Author

xeji commented Sep 22, 2018

cc @samueldr

@samueldr
Copy link
Member

Artist's rendition of how I feel

image


Verified to work with QEMU (looked into making qemu boot 32 bit UEFI).

image

$ qemu-system-i386 --enable-kvm -bios /nix/store/sa51g5xa4lhc56r4j844risbak2vva2x-OVMF-2017-12-05-fd/FV/OVMF.fd -device e1000,netdev=net0 -netdev user,id=net0 -m 8G -cdrom result/iso/nixos-minimal-19.03pre56789.gfedcba-i686-linux.iso

Where the OVMF path is from nix-build --no-out-link --option system "i686-linux" -A OVMF.fd.

With current master / current nixos-unstable it hangs on the Tianocore EFI screen.

👍 and backporting.

@samueldr samueldr merged commit f753852 into NixOS:master Sep 23, 2018
@samueldr
Copy link
Member

samueldr commented Sep 23, 2018

[release-18.09 73ceb77] nixos/iso-image: fix 32bit UEFI boot
Date: Sun Sep 23 00:15:28 2018 +0200
1 file changed, 1 insertion(+), 1 deletion(-)

@xeji xeji deleted the p/iso-boot-ia32 branch September 24, 2018 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants