cedarhouse1@comcast.net has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33234 )
Change subject: security/intel/stm: Add STM support ......................................................................
Patch Set 54:
(2 comments)
Patch Set 54:
(2 comments)
Do some of the CamelCasedNamed files come from Tianocore? If so, some information on the commit they've been taken from would be good to have to ease future updates.
Yes - the files within security/intel/smm come directly from their Tianocore counterparts. Unnecessary code has been removed and the remaining code has been converted to meet coreboot coding requirements.
Added the gist of the previous statement to the commit
https://review.coreboot.org/c/coreboot/+/33234/54/configs/config.stm File configs/config.stm:
https://review.coreboot.org/c/coreboot/+/33234/54/configs/config.stm@2 PS54, Line 2: CONFIG_VENDOR_PURISM=y : CONFIG_MAINBOARD_VENDOR="google" : CONFIG_MAINBOARD_SMBIOS_MANUFACTURER="Emulation"
that's a weird combination, I guess this config underwent a couple of iterations on different boards […]
I see the error of my ways.. I did not do a make distclean.
https://review.coreboot.org/c/coreboot/+/33234/54/src/security/intel/stm/Kco... File src/security/intel/stm/Kconfig:
https://review.coreboot.org/c/coreboot/+/33234/54/src/security/intel/stm/Kco... PS54, Line 4: bool "Enable STM"
The integration in mp_init. […]
The error message (line 769) in mp_init.c has been changed to read:
"STM not loaded because SMM is not enabled!"
Also, added information in the Kconfig help to indicate that to have an STM, SMM must be enabled.