[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake: Fix crash in MP init

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35312 )

Change subject: soc/intel/skylake: Fix crash in MP init
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35312/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/35312/1//COMMIT_MSG@14 
PS1, Line 14: To fully migitate L1TF or E2E attacks, hyperthreading should be disabled
> do we really want to disable HT by default?

ah, well, sgx_enable only comes into effect when SOC_INTEL_COMMON_BLOCK_SGX=y, so SGX and HT are controllable by config FSP_HYPERTHREADING, which is on by default, right?