Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46055 )
Change subject: security/intel/txt: Use `smm_region()` to get TSEG base ......................................................................
Patch Set 3:
(3 comments)
https://review.coreboot.org/c/coreboot/+/46055/2/src/security/intel/txt/logg... File src/security/intel/txt/logging.c:
https://review.coreboot.org/c/coreboot/+/46055/2/src/security/intel/txt/logg... PS2, Line 216: tseg_size
size_t
Done here and in ramstage.c
https://review.coreboot.org/c/coreboot/+/46055/2/src/security/intel/txt/logg... PS2, Line 227: * MiB
This is probably not needed anymore.
Nope, dropped. I changed this to also show the size of TSEG in MiB
https://review.coreboot.org/c/coreboot/+/46055/2/src/security/intel/txt/rams... File src/security/intel/txt/ramstage.c:
https://review.coreboot.org/c/coreboot/+/46055/2/src/security/intel/txt/rams... PS2, Line 239: >> 20
I chose to use a shift because I'm copying the field over, but I can use MiB too.
Done