4 comments:
Patch Set #2, Line 10: each core, not for each thread.
Other documents talk about cores and threads. The commit message probably needs an update anyway.
Ack. Documentation about MSR scope uses "thread" anyway.
File src/soc/intel/common/block/cpu/mp_init.c:
Patch Set #3, Line 40: Testes
Tests
Gone
File src/soc/intel/common/block/cpu/mp_init.c:
Patch Set #5, Line 46: intel_ht_sibling_spin_lock();
This is assuming that all microcode loading needs to be serialized on hyperthreads. […]
Looks like this was changed. Will mark as resolved since the comment is buried in an old patchset.
Patch Set #5, Line 139: *parallel = 1;
Sounds like a good idea. Bigger question is why we are doing microcode updates above. […]
CB:44400 added documentation. We need to load microcode twice because these are enhanced microcode updates. AFAIUI, they may do security locks regarding SMM.
Only Haswell and Broadwell seem to allow unrestricted parallel microcode loading. No idea why. That only Broadwell is a SoC is annoying.
I'm going to mark this as resolved since it's buried in an old patchset. Feel free to reopen.
To view, visit change 35739. To unsubscribe, or for help writing mail filters, visit settings.