-
Notifications
You must be signed in to change notification settings - Fork 32
Add TD Info as Initialization Header #102
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
base: main
Are you sure you want to change the base?
Conversation
This reverts commit 378acf1.
| pub used_size: u64, | ||
| } | ||
|
|
||
| /// Optional launch time configurations for VMs running on TDX platform. |
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.
Is this really optional if it impacts the measurement of the guest? It's optional in the sense of "if you don't specify this, the VMM may choose whatever value it likes" correct?
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.
Yes. It's optional in the sense that the IGVM file should load even if you don't specify these fields. I don't think all guests care if the measurements are different across different hosts, so some guests may choose not to specify these fields.
TDX VMs require additional launch time configurations for better flexibility