Angel Pons uploaded patch set #13 to the change originally created by Patrick Rudolph.

View Change

[WIP]soc/intel/common/block: Update microcode serialized for HT threads

On HyperThreading-enabled platforms update the microcode on a core
serialized for threads running on that core. This prevents the
"microcode: Update failed" error seen in the console log on boards
with multiple cores and Hyper-threading enabled.

The SDM and most of the BWGs suggest doing this.

TODO:
* Fix MPinit's first microcode update
* Remove comments that state microcode updates should be done after
SMM setup (it should be done at end of MPinit)
* Guard microcode updates on HT where required

TOTEST: Can we update the microcode on all primary threads, then on all
secondary threads? This would make locking much less complex.

Change-Id: I72804753e567a137a5648ca6950009fed332531b
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
---
M src/cpu/intel/common/common.h
M src/cpu/intel/common/hyperthreading.c
M src/cpu/intel/microcode/microcode.c
M src/include/cpu/intel/microcode.h
M src/include/cpu/x86/lapic.h
M src/soc/intel/common/block/cpu/Kconfig
M src/soc/intel/common/block/cpu/mp_init.c
M src/soc/intel/common/block/sgx/sgx.c
8 files changed, 115 insertions(+), 19 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/13

To view, visit change 35739. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b
Gerrit-Change-Number: 35739
Gerrit-PatchSet: 13
Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Christian Walter <christian.walter@9elements.com>
Gerrit-Reviewer: Lijian Zhao <lijian.zhao@intel.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi@intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-CC: Furquan Shaikh <furquan@google.com>
Gerrit-MessageType: newpatchset