Skip to content

Add TDX Attestation - #41

Open
yogeshbdeshpande wants to merge 5 commits into
mainfrom
intel-tdx-musings
Open

yogeshbdeshpande wants to merge 5 commits into
mainfrom
intel-tdx-musings

Conversation

@yogeshbdeshpande

@yogeshbdeshpande yogeshbdeshpande commented May 25, 2023

Copy link
Copy Markdown
Contributor

Signed-off-by: Yogesh Deshpande <yogesh.deshpande@arm.com>

@setrofim setrofim left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Real nice write-up, just a few minor nits. In addition to the comments on specific lines -- there appear to be a few lines (mainly in the glossary, but also a few elsewhere) that aren't wrapped to the same column width as the rest of the document.

Comment thread musings/intel-tdx-attestation.md Outdated
Comment thread musings/intel-tdx-attestation.md Outdated
Comment thread musings/intel-tdx-attestation.md Outdated
Comment thread musings/intel-tdx-attestation.md Outdated
Signed-off-by: Yogesh Deshpande <yogesh.deshpande@arm.com>
Signed-off-by: Yogesh Deshpande <yogesh.deshpande@arm.com>
Signed-off-by: Yogesh Deshpande <yogesh.deshpande@arm.com>
Signed-off-by: Yogesh Deshpande <yogesh.deshpande@arm.com>
@THS-on

THS-on commented Sep 15, 2026

Copy link
Copy Markdown
Member

@yogeshbdeshpande @jraman567 how does it align with the latest discussions we had a month back? Any plans on updating it?

@jraman567

jraman567 commented Sep 16, 2026

Copy link
Copy Markdown

Hi @yogeshbdeshpande , thanks for point me to this doc. I'll review this shortly.

I'm unsure if I followed-up with you, but I'm currently blocked on the TDX implementation due to the following issue. We need a scheme to store the reference of the TD measurements in CoRIM. Are you aware of one, if so could you please point me to it? Thank you!

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

Successfully merging this pull request may close these issues.

4 participants