- Notifications
You must be signed in to change notification settings - Fork291
source/kernel: Add kvm availability#2367
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Conversation
linux-foundation-easyclabot commentedNov 7, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
|
netlifybot commentedNov 7, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Deploy Preview forkubernetes-sigs-nfd ready!
To edit notification comments on pull requests, go to yourNetlify project configuration. |
k8s-ci-robot commentedNov 7, 2025
Welcome@seungukshin! |
k8s-ci-robot commentedNov 7, 2025
Hi@seungukshin. Thanks for your PR. I'm waiting for agithub.com member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listedhere. DetailsInstructions for interacting with me using PR comments are availablehere. If you have questions or suggestions related to my behavior, please file an issue against thekubernetes-sigs/prow repository. |
marquiz left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thank you@seungukshin for the contribution. Could you sign the CLA?
Uh oh!
There was an error while loading.Please reload this page.
marquiz commentedDec 9, 2025
/ok-to-test |
seungukshin commentedDec 9, 2025
Thank you@marquiz and sorry for the delay. |
marquiz commentedDec 10, 2025
No problem |
52d9f6c tod6c54b3Compare
marquiz left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thank you@seungukshin, looks good to me now.
One ask: could you rebase so that we get rid of the CI netlify errors?`
/assign@fmuyassarov@ArangoGutierrez
k8s-ci-robot commentedDec 12, 2025
[APPROVALNOTIFIER] This PR isAPPROVED This pull-request has been approved by:marquiz,seungukshin The full list of commands accepted by this bot can be foundhere. The pull request process is describedhere DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This commit adds a feature to check kvm availability.Signed-off-by: Seunguk Shin <seunguk.shin@arm.com>Reviewed-by: Nick Connolly <nick.connolly@arm.com>
seungukshin commentedDec 12, 2025
Thank you@marquiz for your review.
I forgot syncing my forked repo. I just rebased it. |
Add support for testing if a node can use virtualization for use by kata-containers (seehttps://katacontainers.slack.com/archives/C879ACQ00/p1761731112286659). On AMD and Intel platforms this can be determined with CPU feature flags, but on arm64 the situation is more complex. There 2 two flags -
ID_AA64PFR0_EL1.EL2(doc) andID_AA64MMFR1_EL1.VH(doc)ID_AA64PFR0_EL1.EL2& 0b0001 != 0 (kernel runs at EL1, hyp world at EL2 - nVHE)ID_AA64MMFR1_EL1.VH& 0b0001 != 0 (kernel is at EL2 using VHE), not visible from user spaceInstead, check if
kvmis initialized and has created/sys/devices/virtual/misc/kvm.