[coreboot-gerrit] Change in coreboot[master]: src/security/vboot: Setup secure counter space in TPM NVRAM

[Julius Werner (Code Review)]

Attention is currently required from: Andrey Pronin, Raul Rangel, Yu-Ping Wu, Karthik Ramasubramanian.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59476 )

Change subject: src/security/vboot: Setup secure counter space in TPM NVRAM
......................................................................

Patch Set 1:

(1 comment)

File src/security/vboot/secdata_tpm.c:

https://review.coreboot.org/c/coreboot/+/59476/comment/1a149069_a09fc72e 
PS1, Line 368: 		rv = tlcl_read(index, &value, SECURE_COUNTER_SIZE);
> actually, it may exist. […]

But that is true for all our TPM spaces. We have never had recovery code from an aborted partial factory init for any of them (and that would be more complicated, because some of them need to be initialized right). It has never been a problem as far as I'm aware. It would be inconsistent to now do it only for this space.