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

mshv-ioctls: Add support for aarch64 #135

Merged
merged 2 commits into from
Mar 28, 2024

Conversation

jinankjain
Copy link
Collaborator

@jinankjain jinankjain commented Mar 25, 2024

Summary of the PR

Remove conditional compilation for x86_64.

Requirements

Before submitting your PR, please make sure you addressed the following
requirements:

  • All commits in this PR have Signed-Off-By trailers (with
    git commit -s), and the commit message has max 60 characters for the
    summary and max 75 characters for each description line.
  • All added/changed functionality has a corresponding unit/integration
    test.
  • All added/changed public-facing functionality has entries in the "Upcoming
    Release" section of CHANGELOG.md (if no such section exists, please create one).
  • Any newly added unsafe code is properly documented.

@russell-islam
Copy link
Collaborator

@jinankjain This PR excludes some of the registers. After excluding the registers, rest of them are enough to boot arm64 guest? Don't we need additional registers somewhere?

@jinankjain
Copy link
Collaborator Author

I have just added a compilation support not functional support for ARM64. get and set_reg would have a different implementation for aarch64.

mshv-ioctls/src/ioctls/vcpu.rs Outdated Show resolved Hide resolved
Remove conditional compilation for x86_64.

Signed-off-by: Jinank Jain <[email protected]>
Since 32-bit ARM virtualization is not supported by MSHV. Thus, remove
the check for it as well.

Signed-off-by: Jinank Jain <[email protected]>
@russell-islam russell-islam merged commit 1d5de83 into rust-vmm:main Mar 28, 2024
2 checks passed
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.

4 participants