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

[ron minnich (Code Review)]

ron minnich 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:

(1 comment)

Can we try, first, guarding this code, rather than the full revert?

It kinda looks to me like this is tickling some FSP issue, is that correct or am I missing something.

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 may be the issue.

yeah if I had to guess it's this and not the struct. Why not guard it and see if that works? 
If it is this does that indicate and FSP problem?