Skip to content

Conversation

@russell-islam
Copy link
Contributor

@russell-islam russell-islam commented Dec 5, 2025

The MSR enumeration logic in MSHV does not accurately
account for the capabilities of the underlying
hardware and hypervisor. Previously, integration tests
passed because only the default MSRs—universally supported
across platforms—were involved. However, recent feature
additions in MSHV have introduced MSRs that are not
supported on Intel hardware, causing failures.
This patch retrieves the MSRs from the VM specific
API call that filters out the unsupported MSRs.

@russell-islam russell-islam requested a review from a team as a code owner December 5, 2025 03:02
@rbradford
Copy link
Member

You will need to rebase for the actions change in #7544.

The MSR enumeration logic in MSHV does not accurately
account for the capabilities of the underlying
hardware and hypervisor. Previously, integration tests
passed because only the default MSRs—universally supported
across platforms—were involved. However, recent feature
additions in MSHV have introduced MSRs that are not
supported on Intel hardware, causing failures.

This patch retrieves the MSRs from the VM specific
API call that filters out the unsupported MSRs.

Signed-off-by: Muminul Islam <[email protected]>
These tests are expected to fail.

See: cloud-hypervisor#7542

Signed-off-by: Muminul Islam <[email protected]>
@rbradford rbradford added this pull request to the merge queue Dec 6, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 6, 2025
@liuw liuw added this pull request to the merge queue Dec 7, 2025
Merged via the queue into cloud-hypervisor:main with commit 7c83724 Dec 7, 2025
46 of 47 checks passed
@likebreath likebreath added the bug-fix Bug fix to include in release notes label Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Bug fix to include in release notes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants