Skip to content

Conversation

@ludfjig
Copy link
Contributor

@ludfjig ludfjig commented Dec 16, 2025

MSRs will be added in another PR.

  • Removed padding fields from fpu register to simplify comparisons. Not sure why I added them to begin with, we don't need C repr on these. It could possibly allow more efficient into() implementation for kvm/mshv due to single memcyp, but that seems like premature optimization to me
  • Reset vcpu state on snapshot::restore
  • Added bunch of tests
  • MSRs are not included in this PR. Will add MSRs in future PR

Addresses #791 partially

@ludfjig ludfjig added the kind/enhancement For PRs adding features, improving functionality, docs, tests, etc. label Dec 16, 2025
@ludfjig ludfjig force-pushed the reset_vcpu branch 3 times, most recently from 9cfcc9c to 8507c7a Compare December 16, 2025 19:52
@ludfjig ludfjig force-pushed the reset_vcpu branch 11 times, most recently from d09b1fc to 18d4ff9 Compare December 17, 2025 06:46
@ludfjig ludfjig requested a review from Copilot December 17, 2025 06:53

This comment was marked as outdated.

@ludfjig ludfjig marked this pull request as ready for review December 17, 2025 07:26
@ludfjig ludfjig force-pushed the reset_vcpu branch 7 times, most recently from 1e8d95a to 71d6a38 Compare January 22, 2026 00:09
@ludfjig ludfjig force-pushed the reset_vcpu branch 3 times, most recently from 2091ef4 to 5ebe998 Compare January 26, 2026 21:09
Copy link
Contributor

@dblnz dblnz left a comment

Choose a reason for hiding this comment

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

Good work! I left some comments below.

Copy link
Contributor

@jsturtevant jsturtevant left a comment

Choose a reason for hiding this comment

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

looking good. A few comments to address otherwise I like it!

dblnz
dblnz previously approved these changes Feb 2, 2026
@ludfjig ludfjig force-pushed the reset_vcpu branch 2 times, most recently from a5d8efa to 54bc12a Compare February 2, 2026 17:55
jsturtevant
jsturtevant previously approved these changes Feb 2, 2026
Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement For PRs adding features, improving functionality, docs, tests, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants