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

[Patrick Georgi (Code Review)]

Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38322 )

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

Patch Set 1:

(3 comments)

does it help to guard the three things I point at with a CONFIG(STM)?

https://review.coreboot.org/c/coreboot/+/38322/1/src/cpu/x86/mp_init.c 
File src/cpu/x86/mp_init.c:

https://review.coreboot.org/c/coreboot/+/38322/1/src/cpu/x86/mp_init.c@a1056 
PS1, Line 1056: 	state->smm_save_state_size =
this could be guarded by CONFIG_STM

https://review.coreboot.org/c/coreboot/+/38322/1/src/cpu/x86/smm/smm_stub.S 
File src/cpu/x86/smm/smm_stub.S:

https://review.coreboot.org/c/coreboot/+/38322/1/src/cpu/x86/smm/smm_stub.S@... 
PS1, Line 47: num_cpus:
these fields could be guarded

https://review.coreboot.org/c/coreboot/+/38322/1/src/cpu/x86/smm/smm_stub.S@... 
PS1, Line 99: 	/* gdt selector 0x18, flat code segment (64-bit) */
and these selectors could be guarded by CONFIG_STM.