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

TD Compliance test not aligned with SDM and TDX module spec #341

Open
fengyuleidian0615 opened this issue Jul 29, 2024 · 1 comment
Open

Comments

@fengyuleidian0615
Copy link

fengyuleidian0615 commented Jul 29, 2024

Hi

Here is the snippet of test log running from TD guest.

CR4_SET_X86_CR4_PCE skipped is expected, as TD has perfmon enabled
MSR_IA32_PRED_CMD is a Write Only MSR from SDM, why perform Read operation, expecting no exception.
Same with IA32_X2APIC stuff, no write allowed from SDM.
Could you please update test pattern per SDM?
thanks


186: CR4_SET_X86_CR4_PCE: SKIP
Error: MSR compliance test failed,exception 13, but expect_exception 0
196: MSR_IA32_PRED_CMD_read_1.0: FAIL
Error: MSR compliance test failed,exception 13, but expect_exception 0
214: MSR_IA32_ARCH_CAPABILITIES_write_1.0: FAIL
Error: MSR compliance test failed,exception 13, but expect_exception 0
215: MSR_IA32_FLUSH_CMD_read_1.0: FAIL
Error: MSR compliance test failed,exception 0, but expect_exception 20
221: MSR_IA32_MISC_ENABLE_write_1.0: FAIL
Error: MSR compliance test failed,exception 13, but expect_exception 0
352: IA32_X2APIC_PPR_write_1.0: FAIL
Error: MSR compliance test failed,exception 13, but expect_exception 0
356: IA32_X2APIC_ISRX_write_1.0: FAIL
Error: MSR compliance test failed,exception 13, but expect_exception 0
358: IA32_X2APIC_TMRX_write_1.0: FAIL
Error: MSR compliance test failed,exception 13, but expect_exception 0
360: IA32_X2APIC_IRRX_write_1.0: FAIL

@ysun
Copy link
Contributor

ysun commented Aug 21, 2024

Thanks Fan's input. I'd like to update the tests. BTW, could you please provide the public TDX spec you're using.
There're too many versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants