Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38768 )
Change subject: [UNTESTED] intel/stm: Introduce stm_update_smm_info() ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38768/2/src/security/intel/stm/StmP... File src/security/intel/stm/StmPlatformSmm.c:
https://review.coreboot.org/c/coreboot/+/38768/2/src/security/intel/stm/StmP... PS2, Line 165: * algorithm. (align on 4K) Where is this documented? I skimmed through the STM manual and couldn't find the reason for this. Is this a software limitation to allow the STM to map a page with the STM-generated save state?
Squeezing this with the current SMM module setup won't be easy. The current implementation puts the SMM stub together with `n` save-state areas into 32KiB. Just enough space for 7 threads.