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:
(2 comments)
https://review.coreboot.org/c/coreboot/+/35312/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/35312/1//COMMIT_MSG@12 PS1, Line 12: I could not find any relation between those two in any datasheet. hmmm, with vendor firmware both can be active at the same time
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?