Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35312 )
Change subject: soc/intel/common/block/sgx: Fix crash in MP init ......................................................................
Patch Set 2:
(4 comments)
Wow, nice find!
Could you please separate the mainboard change into its own commit? (Maybe documentation needs to be updated too?)
https://review.coreboot.org/c/coreboot/+/35312/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/35312/2//COMMIT_MSG@9 PS2, Line 9: HyperThreading Hyper-threading
https://review.coreboot.org/c/coreboot/+/35312/2//COMMIT_MSG@11 PS2, Line 11: To prevent such race conditions only call the code for one thread, such Please add a blank line above.
https://review.coreboot.org/c/coreboot/+/35312/2/src/soc/intel/common/block/... File src/soc/intel/common/block/sgx/sgx.c:
https://review.coreboot.org/c/coreboot/+/35312/2/src/soc/intel/common/block/... PS2, Line 83: HyperThreading Hyper-threading
https://review.coreboot.org/c/coreboot/+/35312/2/src/soc/intel/common/block/... PS2, Line 218: an on