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

Linux EFI Stub fixes #235

Merged
merged 5 commits into from
Mar 15, 2023

Conversation

rbradford
Copy link
Member

Here are some fixes I needed to make for enabling booting Linux using the EFI
Stub support.

  • efi: Include a firmware vendor string
  • efi: Populate CopyMem/SetMem EFI functions
  • efi: Make the heap size more reasonable

The Linux kernel requires this when running as an EFI Stub on RISC-V.
The simplest solution is a static "RHF" string encoded as UCS-2 bytes.

Signed-off-by: Rob Bradford <rbradford@rivosinc.com>
The Linux EFI stub uses these during boot process.

Signed-off-by: Rob Bradford <rbradford@rivosinc.com>
The container is published for this architecture now.

Signed-off-by: Rob Bradford <rbradford@rivosinc.com>
@rbradford rbradford force-pushed the 2023-03-15-efi-stub-fixes branch from 7ad08f6 to 8585a5d Compare March 15, 2023 14:04
There are extra dependencies required to build the container that are
not necessarily included on the CI hosts.

Signed-off-by: Rob Bradford <rbradford@rivosinc.com>
When booting a VM with 512MiB assigned to it almost half was being lost
to the heap allocator and the amount of heap allocations that the
firmware does should easily fit into the new 2MiB limit.

Signed-off-by: Rob Bradford <rbradford@rivosinc.com>
@rbradford rbradford requested a review from sboeuf March 15, 2023 15:49
@rbradford rbradford merged commit 0f27b6e into cloud-hypervisor:main Mar 15, 2023
@rbradford rbradford deleted the 2023-03-15-efi-stub-fixes branch March 15, 2023 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants