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.

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

Check the list of CPUID leaves required for TDX #562

Closed
cperezvargas opened this issue Dec 20, 2024 · 4 comments
Closed

Check the list of CPUID leaves required for TDX #562

cperezvargas opened this issue Dec 20, 2024 · 4 comments
Assignees
Labels
tdx TDX specific bugs or features

Comments

@cperezvargas
Copy link
Contributor

Check if the following CPUID leaves are required for TDX and modify the list accordingly.

pub const TDX_REQUIRED_LEAVES: &[(CpuidFunction, Option)] = &[
// The following aren't required from AMD. Need to double-check if they're required for TDX
(CpuidFunction::CacheAndTlbInformation, None),
(CpuidFunction::ExtendedFeatures, Some(1)),
(CpuidFunction::CacheParameters, Some(0)),
(CpuidFunction::CacheParameters, Some(1)),
(CpuidFunction::CacheParameters, Some(2)),
(CpuidFunction::CacheParameters, Some(3)),
];

@stephensons2022
Copy link

Assigning to @vibhutet after speaking with Joe

@vibhutet
Copy link
Contributor

The above CPUIDs are required ones for TDX. The list doesn't need to be modified.

@vibhutet
Copy link
Contributor

This issue can be closed.

@stephensons2022
Copy link

Closing per the above comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tdx TDX specific bugs or features
Projects
None yet
Development

No branches or pull requests

4 participants