Skip to content

aarch64: unset highmem=off for QEMU >= 7.0 #703

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

Merged
merged 1 commit into from
Mar 10, 2022

Conversation

AkihiroSuda
Copy link
Member

  • QEMU >= 7.0 requires highmem=off NOT to be set, otherwise fails with "Addressing limited to 32 bits, but memory exceeds it by 1073741824 bytes"
  • QEMU < 7.0 requires highmem=off to be set, otherwise fails with "VCPU supports less PA bits (36) than requested by the memory map (40)"

See #680 and #24

Fix #680

@AkihiroSuda
Copy link
Member Author

Tested on Intel Mac, with qemu/qemu@2ad7624900

@jandubois @afbjorklund PTAL

jandubois
jandubois previously approved these changes Mar 9, 2022
Copy link
Member

@jandubois jandubois left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tested it, but LGTM.

@AkihiroSuda
Copy link
Member Author

Can anybody test this with M1 macOS?

@afbjorklund
Copy link
Member

afbjorklund commented Mar 9, 2022

Can anybody test this with M1 macOS?

I can see if I can find a machine tonight, we're doing some testing of the minikube ISO (for aarch64)...

Was there anyone that was willing to sponsor a machine for the CI ? (not sure what options Cirrus had)

Did the last tests on a rented machine (mini).

- QEMU >= 7.0 requires highmem=off NOT to be set, otherwise fails with "Addressing limited to 32 bits, but memory exceeds it by 1073741824 bytes"
- QEMU <  7.0 requires highmem=off to be set, otherwise fails with "VCPU supports less PA bits (36) than requested by the memory map (40)"

See issue 680 and PR 24

Fix issue 680

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
@AkihiroSuda
Copy link
Member Author

Merging and releasing v0.9.0-beta.0 for ease of testing

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.

Memory allocation fails on darwin arm64 with latest qemu
3 participants