-
Notifications
You must be signed in to change notification settings - Fork 133
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
User CET Register #349
User CET Register #349
Conversation
There was one spurious change, which I removed. |
Last two flags had "ENABLE" and "DISABLE" as prefix, unlike previous flags. |
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.
Thank you for your contribution!
OK, updated, I have for some reason |
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.
Two small comments, then this is ready to get merged :)
Add support for IA32_U_CET and IA32_S_CET, which are used for Control flow Execution Technology (CET) configuration. Signed-off-by: Jarkko Sakkinen <[email protected]>
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.
Thank you for your contribution!
Thanks for the help :-) |
This embedded to some of SGX structures (at least SECS and REPORT), so that's I need to bring some support to x86_64 crate. Does not cover fields TRACKER but EB_LEG_BITMAP_BASE but:
Actually SGX structures and encl* need only UCetFlags but it would have been crippled just to implement that structure without context.