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.

Sign up for GitHub

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

Jump to bottom

kernel: Add CONFIG_TDX_GUEST_DRIVER to the tdx.conf #8556

Merged
merged 1 commit into from Dec 4, 2023
Merged

kernel: Add CONFIG_TDX_GUEST_DRIVER to the tdx.conf #8556

merged 1 commit into from Dec 4, 2023

Conversation

fidencio
Copy link
Member

@fidencio fidencio commented Dec 4, 2023

The driver enables the userspace interface to communicate with the TDX module to request the TDX guest details, like the attestation report.

Fixes: #8555

@fidencio
kernel: Add CONFIG_TDX_GUEST_DRIVER to the tdx.conf
03c3f42 
The driver enables the userspace interface to communicate with the TDX
module to request the TDX guest details, like the attestation report.

Fixes: kata-containers#8555

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label Dec 4, 2023
@fidencio
Copy link
Member Author

fidencio commented Dec 4, 2023

/test

jodh-intel
jodh-intel approved these changes Dec 4, 2023
View reviewed changes
Copy link
Contributor

@jodh-intel jodh-intel left a comment

Choose a reason for hiding this comment

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

Thanks @fidencio.

lgtm

ChengyuZhu6
ChengyuZhu6 approved these changes Dec 4, 2023
View reviewed changes
Copy link
Member

@ChengyuZhu6 ChengyuZhu6 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @fidencio !

@fidencio
Copy link
Member Author

fidencio commented Dec 4, 2023

With this patch applied I got:

$ sudo /opt/kata/bin/kata-runtime exec busybox
root@localhost:/# ls /dev/tdx_guest
/dev/tdx_guest

arronwy
arronwy approved these changes Dec 4, 2023
View reviewed changes
Copy link
Member

@arronwy arronwy left a comment

Choose a reason for hiding this comment

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

Thanks @fidencio LGTM!

@fidencio fidencio merged commit 8d7e0f7 into kata-containers:main Dec 4, 2023
169 of 241 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arronwy arronwy arronwy approved these changes

@jodh-intel jodh-intel jodh-intel approved these changes

@ChengyuZhu6 ChengyuZhu6 ChengyuZhu6 approved these changes

@danmihai1 danmihai1 Awaiting requested review from danmihai1

Assignees
No one assigned
Labels
ok-to-test size/tiny Smallest and simplest task
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

kernel: tdx: /dev/tdx_guest device is not available on the kata containers VM
5 participants
@fidencio @arronwy @jodh-intel @ChengyuZhu6 @katacontainersbot