-
Notifications
You must be signed in to change notification settings - Fork 159
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
vTPM : fix arm issue by selecting correct dev name based on arch #4251
Conversation
199110c
to
c4523eb
Compare
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.
Looks good
c4523eb
to
be35031
Compare
The vTPM device name is different on ARM and x86. This patch fixes the the issue by selecting the correct device name based on the architecture. Signed-off-by: Shahriyar Jalayeri <shahriyar@zededa.com>
be35031
to
9fe5015
Compare
By the way, do we have a test that could detect the error? |
If we had eden arm runners, yes, the vTPM test would have failed. |
Talked to @uncleDecart , unfortunately currently we don't have ARM host runners performing Eden tests.... 😢 |
So, we still do not have access to the GitHub ARM runners? https://github.blog/news-insights/product-news/arm64-on-github-actions-powering-faster-more-efficient-build-systems/ |
The vTPM device name is different on ARM and x86. This patch fixes the the issue by selecting the correct device name based on the architecture.
Thanks to @rene for bringing it to my attention.