[coreboot-gerrit] [L] Change in coreboot[master]: util/cbmem: add parsing of TPM logs per specs

[Sergii Dmytruk (Code Review)]

Attention is currently required from: Arthur Heymans, Benjamin Doron, Christian Walter, Himanshu Sahdev, Julius Werner, Krystian Hebel, Maximilian Brune, Michał Kopeć, Michał Żygowski, Paul Menzel, Sergii Dmytruk.

Hello Angel Pons, Benjamin Doron, Christian Walter, Julius Werner, Krystian Hebel, Maximilian Brune, Michał Kopeć, Michał Żygowski, Paul Menzel, build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/68749?usp=email

to look at the new patch set (#21).

The following approvals got outdated and were removed:
Code-Review+1 by Paul Menzel, Verified+1 by build bot (Jenkins)

Change subject: util/cbmem: add parsing of TPM logs per specs
......................................................................

util/cbmem: add parsing of TPM logs per specs

CBMEM can contain log in different forms (at most one is present):
 - coreboot-specific format (CBMEM_ID_TPM_CB_LOG exported as
   LB_TAG_TPM_CB_LOG)
 - TPM1.2 format (CBMEM_ID_TCPA_TCG_LOG)
 - TPM2 format (CBMEM_ID_TPM2_TCG_LOG)

The last two follow specifications by Trusted Computing Group, but until
now cbmem couldn't print them. (They were added not so long ago in
4191dbf0c9a57088e7a7323d8bf02a8708eacdb9 and
53db677586e3e7c4a874f1ef16f34a6d01c269a0.)

These changes make cbmem utility check for existence of TPM1.2/TPM2 logs
in CBMEM and add code necessary for parsing and printing of their
entries.

TEST=`cbmem -L` for CONFIG_TPM1=y case
TCPA log:
    Specification: 1.21
    Platform class: PC Client
TCPA log entry 1:
    PCR: 2
    Event type: Action
    Digest: 5622416ea417186aa1ac32b32c527ac09009fb5e
    Event data: FMAP: FMAP

TEST=`cbmem -L` for CONFIG_TPM2=y case
TPM2 log:
    Specification: 2.00
    Platform class: PC Client
TPM2 log entry 1:
    PCR: 2
    Event type: Action
    Digests:
    	 SHA256: 68d27f08cb261463a6d004524333ac5db1a3c2166721785a6061327b6538657c
    Event data: FMAP: FMAP

Change-Id: Ib76dc7dec56dd1789a219539a1ac05a958f47a5c
Ticket: https://ticket.coreboot.org/issues/425
Signed-off-by: Krystian Hebel 
Signed-off-by: Michał Żygowski 
Signed-off-by: Sergii Dmytruk 
---
M src/commonlib/bsd/include/commonlib/bsd/tpm_log_defs.h
M util/cbmem/cbmem.c
2 files changed, 300 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/49/68749/21