Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42712 )
Change subject: security/intel/txt: Add Intel TXT support
......................................................................
Patch Set 4:
Patch Set 4:
Patch Set 4: Code-Review+2
LGTM. This is cheery-pick of "5f9f77672d security/intel/txt: Add Intel TXT support" to master branch.
There were updates (most notably, refactoring of the assembly code for getsec) so it would be good to test the current patchset to make sure nothing has broken.
I will test now, and report back.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/42712
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: 4.11_branch
Gerrit-Change-Id: Iaf4be7f016cc12d3971e1e1fe171e6665e44c284
Gerrit-Change-Number: 42712
Gerrit-PatchSet: 4
Gerrit-Owner: Jonathan Zhang
jonzhang@fb.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Jonathan Zhang
jonzhang@fb.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Philipp Deppenwiese
zaolin.daisuki@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Fri, 31 Jul 2020 17:03:00 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment