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: test case to validate map VP state page #160

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

russell-islam
Copy link
Collaborator

Add a new test case to test the vp_run method that uses mapped registers page and hv_message page.

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.

mshv-ioctls/src/ioctls/vcpu.rs Outdated Show resolved Hide resolved
mshv-ioctls/src/ioctls/vcpu.rs Outdated Show resolved Hide resolved
mshv-ioctls/src/ioctls/vcpu.rs Outdated Show resolved Hide resolved
mshv-ioctls/src/ioctls/vcpu.rs Outdated Show resolved Hide resolved
@russell-islam
Copy link
Collaborator Author

@NunoDasNeves PTAL

@russell-islam russell-islam enabled auto-merge (rebase) August 30, 2024 21:02
@russell-islam
Copy link
Collaborator Author

Test:

cloud@dom0-2408-27-0-ci-105867774 [ ~/cloud-hypervisor/russell-islam/mshv ]$ cargo test test_run_code Compiling mshv-ioctls v0.3.0 (/home/cloud/cloud-hypervisor/russell-islam/mshv/mshv-ioctls) Finished test` profile [unoptimized + debuginfo] target(s) in 1.18s
Running unittests src/lib.rs (target/debug/deps/mshv_bindings-67b07de978aee5fe)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 361 filtered out; finished in 0.00s

 Running unittests src/lib.rs (target/debug/deps/mshv_ioctls-12bde8a5f3083285)

running 2 tests
test ioctls::vcpu::tests::test_run_code_mmap ... ok
test ioctls::vcpu::tests::test_run_code ... ok

test result: ok. 2 passed; 0 failed; 0 ignored; 0 measured; 37 filtered out; finished in 0.04s

cloud@dom0-2408-27-0-ci-105867774 [ ~/cloud-hypervisor/russell-islam/mshv ]$
`

Add a new test case to test the vp_run method that uses
mapped registers page and hv_message page.

Signed-off-by: Muminul Islam <muislam@microsoft.com>
@russell-islam russell-islam merged commit 495f188 into main Sep 6, 2024
6 checks passed
@russell-islam russell-islam deleted the muislam/mmap-test branch September 6, 2024 05:27
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