-
Notifications
You must be signed in to change notification settings - Fork 602
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
qemu: riscv64: switch from u-boot to EDK2 #2592
Conversation
cd4fe01
to
44e269d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, and works fine on macOS aarch64 host.
Needs to be rebased to resolve merge conflicts!
44e269d
to
92bd545
Compare
https://github.com/lima-vm/lima/actions/runs/11138475165/job/30953501156?pr=2592
|
u-boot no longer needs to be specified as the "kernel" image, as EDK2 (`edk2-riscv-code.fd`) is available since QEMU v9.1. ACPI is disabled, as suggested in https://github.com/tianocore/edk2/blob/edk2-stable202408/OvmfPkg/RiscVVirt/README.md#test Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
216ccf9
to
5f27045
Compare
To avoid "failed to load YAML file \"templates/experimental/riscv64.yaml\": can't parse builtin Lima version \"3f3a6f6\": 3f3a6f6 is not in dotted-tri format" Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
5f27045
to
762e510
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, LGTM
u-boot no longer needs to be specified as the "kernel" image, as EDK2 (
edk2-riscv-code.fd
) is available since QEMU v9.1.ACPI is disabled, as suggested in https://github.com/tianocore/edk2/blob/edk2-stable202408/OvmfPkg/RiscVVirt/README.md#test