-
Notifications
You must be signed in to change notification settings - Fork 98
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
AA: Update CcaAttester to use TSM Report ABI #595
AA: Update CcaAttester to use TSM Report ABI #595
Conversation
f7dcee8
to
7579cc8
Compare
7579cc8
to
14e1283
Compare
@mathias-arm what is your plan to move this away from draft and get it review/merged? |
6a7a989
to
52778c2
Compare
Hi, I was waiting to see if there was additional feedback, then summer happened and I needed/wanted to rebase the changes. |
ok. I guess the maintainers were waiting to get this moved away from draft first before taking a look. I'll do my final review shortly. |
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.
LGTM, just one minor open/suggestion
Signed-off-by: Mathias Brossard <[email protected]>
52778c2
to
22700d3
Compare
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.
lgtm
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.
Thanks! some nits
Co-authored-by: Xynnn_ <[email protected]> Signed-off-by: Mathias Brossard <[email protected]>
The latest Arm CCA Linux guest support (see
cca-guest/v2
andcca-guest/v3
branches in https://gitlab.arm.com/linux-arm/linux-cca) have added support for TSM Reports ABI.