[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block: Serialize microcode updates for HT threads

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 )

Change subject: soc/intel/common/block: Serialize microcode updates for HT threads
......................................................................

Patch Set 17:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35739/17/src/cpu/intel/microcode/mi... 
File src/cpu/intel/microcode/microcode.c:

https://review.coreboot.org/c/coreboot/+/35739/17/src/cpu/intel/microcode/mi... 
PS17, Line 63: Update just on the first CPU in the core.
That's not what's happening here, is it?

(I almost ran into the good old forgot-to-review-comments.)

Wouldn't it suffice to say that
* some HT capable CPUs need only one thread to do it,
* some need both threads and software-sync,
* some need both and do hardware-sync, and
* software sync'ed version always works!
?