Attention is currently required from: Michał Żygowski. Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59519 )
Change subject: security/intel/txt/ramstage.c: Fix HEAP_ACM element size calculation ......................................................................
Patch Set 2:
(2 comments)
File src/security/intel/txt/ramstage.c:
https://review.coreboot.org/c/coreboot/+/59519/comment/e164ddc3_c76da5f2 PS2, Line 271: * FIXME: these calculations handle the lack of SINIT ACM in CBFS. maybe we can use 2 structs, one with NumAcms == 1 and one with NumAcms == 2. I don't think that there are any other cases.
https://review.coreboot.org/c/coreboot/+/59519/comment/ff8f3afa_359581f4 PS2, Line 278: size this will fix the HEAP_EXTDATA_TYPE_ACM size for case NumAcms == 1, but it then will not point to HEAP_EXTDATA_TYPE_END.