[coreboot-gerrit] Change in coreboot[master]: security/intel/stm: Add STM support

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33234 )

Change subject: security/intel/stm: Add STM support
......................................................................

Patch Set 57:

(1 comment)

https://review.coreboot.org/c/coreboot/+/33234/57/src/cpu/x86/smm/smm_module... 
File src/cpu/x86/smm/smm_module_loader.c:

https://review.coreboot.org/c/coreboot/+/33234/57/src/cpu/x86/smm/smm_module... 
PS57, Line 365: 		base += size - CONFIG_MSEG_SIZE;     // take out the mseg
> > MSEG, aka monitor segment, is where the STM is loaded. […]

Same for the BIOS resource list. Is there any reason not to put it above
the stacks? If we could place everything STM specific on top, all we'd have
to do is to report a lower `perm_smsize` in `.get_smm_info` and the cpu/x86/
code wouldn't have to know anything about STM.