Change in coreboot[master]: soc/intel/common/block: Update microcode for each core
Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode for each core ...................................................................... soc/intel/common/block: Update microcode for each core On Hyper-Threading enabled platform update the microcde only once for each core, not for each thread. Follow Intel Software Developer Guidelines as the added comment also states. Change-Id: I72804753e567a137a5648ca6950009fed332531b Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com> --- M src/soc/intel/common/block/cpu/mp_init.c 1 file changed, 20 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/1 diff --git a/src/soc/intel/common/block/cpu/mp_init.c b/src/soc/intel/common/block/cpu/mp_init.c index 2c5061f..e7689cf 100644 --- a/src/soc/intel/common/block/cpu/mp_init.c +++ b/src/soc/intel/common/block/cpu/mp_init.c @@ -26,6 +26,7 @@ #include <intelblocks/fast_spi.h> #include <intelblocks/mp_init.h> #include <intelblocks/msr.h> +#include <cpu/intel/common/common.h> #include <soc/cpu.h> static const void *microcode_patch; @@ -44,7 +45,24 @@ static void init_one_cpu(struct device *dev) { soc_core_init(dev); - intel_microcode_load_unlocked(microcode_patch); + + /* + * Update just on the first CPU in the core. Other siblings + * get the update automatically according to Document: 253668-060US + * Intel SDM Chapter 9.11.6.3 + * "Update in a System Supporting Intel Hyper-Threading Technology" + * Intel Hyper-Threading Technology has implications on the loading of the + * microcode update. The update must be loaded for each core in a physical + * processor. Thus, for a processor supporting Intel Hyper-Threading + * Technology, only one logical processor per core is required to load the + * microcode update. Each individual logical processor can independently + * load the update. However, MP initialization must provide some mechanism + * (e.g. a software semaphore) to force serialization of microcode update + * loads and to prevent simultaneous load attempts to the same core. + */ + if (!intel_ht_sibling()) { + intel_microcode_load_unlocked(microcode_patch); + } } static struct device_operations cpu_dev_ops = { @@ -141,6 +159,7 @@ if (CONFIG(USE_INTEL_FSP_MP_INIT)) return; + /* Update microcode on BSP */ microcode_patch = intel_microcode_find(); intel_microcode_load_unlocked(microcode_patch); -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 1 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-MessageType: newchange
Hello Patrick Rudolph, build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#2). Change subject: soc/intel/common/block: Update microcode for each core ...................................................................... soc/intel/common/block: Update microcode for each core On Hyper-Threading enabled platform update the microcde only once for each core, not for each thread. Follow Intel Software Developer Guidelines as the added comment also states. Change-Id: I72804753e567a137a5648ca6950009fed332531b Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com> --- M src/soc/intel/common/block/cpu/Kconfig M src/soc/intel/common/block/cpu/mp_init.c 2 files changed, 22 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/2 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 2 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-MessageType: newpatchset
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode for each core ...................................................................... Patch Set 2: Code-Review+1 (1 comment) Does this also reduce the boot time? Tested how? https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... PS2, Line 50: Update just on the first CPU in the core Remove *on*? -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 2 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> 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: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-Comment-Date: Tue, 01 Oct 2019 11:54:13 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode for each core ...................................................................... Patch Set 2: This is also done in the cpu/x86/sipi_vector.S. Should that code also serialize/skip MCU on logical CPUs? -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 2 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> 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: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-Comment-Date: Tue, 01 Oct 2019 13:12:02 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode for each core ...................................................................... Patch Set 2: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/2//COMMIT_MSG Commit Message: https://review.coreboot.org/c/coreboot/+/35739/2//COMMIT_MSG@10 PS2, Line 10: each core, not for each thread. Nit: Intel Software Developer Guidelines speaks of cores and logical processors. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 2 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> 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: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-Comment-Date: Tue, 01 Oct 2019 13:15:02 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode for each core ...................................................................... Patch Set 2: Code-Review-1 (4 comments) You can ignore the bikeshedding comments, that was before I looked into the documentation. It seems the SDM is still talking about Pentium4 style Hyper-Threading. Starting with Nehalem, their second HT incarnation, I see the requirement to do it on all logical processors with synchronisation per core. Starting with Ivy Bridge, the synchronisation requirement vanished. @Intel, please assist to get the Software Developer's Manual updated (or the BWGs fixed if they are wrong). https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... PS2, Line 50: CPU CPU, at least to me, usually refers to the whole package. https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... PS2, Line 51: 253668-060US : * Intel SDM Chapter 9.11.6.3 A much better reference would be the name of the document and the chapter. Chapter numbers change, and even if one knows the document revision that doesn't mean they have access to that specific revision. https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... PS2, Line 53: * "Update in a System Supporting Intel Hyper-Threading Technology" : * Intel Hyper-Threading Technology has implications on the loading of the : * microcode update. The update must be loaded for each core in a physical : * processor. Thus, for a processor supporting Intel Hyper-Threading : * Technology, only one logical processor per core is required to load the : * microcode update. Each individual logical processor can independently : * load the update. However, MP initialization must provide some mechanism : * (e.g. a software semaphore) to force serialization of microcode update : * loads and to prevent simultaneous load attempts to the same core. No need for the quote, imho. https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... PS2, Line 162: /* Update microcode on BSP */ That's obvious. Much more interesting would be to know why? -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 2 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-Comment-Date: Sun, 06 Oct 2019 18:05:53 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Hello Lijian Zhao, build bot (Jenkins), Nico Huber, Rizwan Qureshi, Paul Menzel, Subrata Banik, Aaron Durbin, Patrick Rudolph, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#3). Change subject: soc/intel/common/block: Update microcode for each core ...................................................................... soc/intel/common/block: Update microcode for each core On Hyper-Threading enabled platform update the microcde only once for each core, not for each thread. Follow Intel Software Developer Guidelines as the added comment also states. 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/include/cpu/x86/lapic.h M src/soc/intel/common/block/cpu/Kconfig M src/soc/intel/common/block/cpu/mp_init.c 5 files changed, 79 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/3 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 3 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-MessageType: newpatchset
Hello build bot (Jenkins), Lijian Zhao, Nico Huber, Rizwan Qureshi, Paul Menzel, Subrata Banik, Aaron Durbin, Patrick Rudolph, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#4). Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... soc/intel/common/block: Update microcode serialiced for HT threads On Hyper-Threading enabled platforms update the microcde on a core serialized for threads running on that core. This prevents the "microcode: Update failed" error seen in the console log. There's still some confusion as the SDW and BWG state differently. It's not clear if the HT sibling must update the microcode at all, as a core likely shares the microcode unit between HT siblings. 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/include/cpu/x86/lapic.h M src/soc/intel/common/block/cpu/Kconfig M src/soc/intel/common/block/cpu/mp_init.c 5 files changed, 79 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/4 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-MessageType: newpatchset
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... Patch Set 4: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... File src/cpu/intel/common/hyperthreading.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 57: if (!initialized) This would need to be checked inside the `init` lock. But why can't we initialize them statically? Like DECLARE_SPIN_LOCK() does? -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-Comment-Date: Thu, 07 May 2020 09:44:23 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... Patch Set 4: (4 comments) https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG Commit Message: https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@7 PS4, Line 7: soc/intel/common/block: Update microcode serialiced for HT threads serialized https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@9 PS4, Line 9: microcde microcode https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@11 PS4, Line 11: "microcode: Update failed" error seen in the console log. On what board? Or on all? https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@13 PS4, Line 13: state differently
… contradict each other.
Maybe also add the sections? -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-Comment-Date: Thu, 07 May 2020 10:39:15 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... Patch Set 4: (5 comments) https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... File src/cpu/intel/common/hyperthreading.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 40: = 0; no need to initialize to 0 explicitly. https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 46: 1; this value shouldn't be open coded. SPIN_LOCK_UNLOCKED definition should be used. https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 53: /* Is Hyper-Threading supported */ : if (!(cpuid_edx(1) & CPUID_FEAURE_HTT)) : return; : This is duplicated below. Add a helper function? https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 57: if (!initialized)
This would need to be checked inside the `init` lock. […] Yes, this initialization should be done on BSP first instead of racing on the APs.
https://review.coreboot.org/c/coreboot/+/35739/4/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/soc/intel/common/block/... PS4, Line 40: Tests For which parts? There are many different implementations. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-Comment-Date: Thu, 07 May 2020 14:23:21 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Gerrit-MessageType: comment
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... Patch Set 4: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/4/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/soc/intel/common/block/... PS4, Line 42: and updates one Some word seems to be missing here. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-Comment-Date: Thu, 07 May 2020 16:40:21 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... Patch Set 4: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... File src/cpu/intel/common/hyperthreading.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 57: if (!initialized)
Yes, this initialization should be done on BSP first instead of racing on the APs. It's not possible in C to initialize a variable array to something else than zero.
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-Comment-Date: Thu, 07 May 2020 17:43:18 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Comment-In-Reply-To: Aaron Durbin <adurbin@chromium.org> Gerrit-MessageType: comment
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... Patch Set 4: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... File src/cpu/intel/common/hyperthreading.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 57: if (!initialized)
It's not possible in C to initialize a variable array to something else than zero. Are you talking about C the language or C in ramstage with some coreboot limitation? I remember things very differently...
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-Comment-Date: Thu, 07 May 2020 18:40:20 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph@9elements.com> Comment-In-Reply-To: Aaron Durbin <adurbin@chromium.org> Gerrit-MessageType: comment
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... Patch Set 4: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... File src/cpu/intel/common/hyperthreading.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 57: if (!initialized)
Are you talking about C the language or C in ramstage with some coreboot […] Sorry, read it a few times more. You mean, we don't know CONFIG_MAX_CPUS, right? I get it now. There is nothing variable in C, though.
Just doing it on the BSP seems like a good idea. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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-Comment-Date: Thu, 07 May 2020 18:45:42 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph@9elements.com> Comment-In-Reply-To: Aaron Durbin <adurbin@chromium.org> Gerrit-MessageType: comment
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialiced for HT threads ...................................................................... Patch Set 4: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG Commit Message: https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@13 PS4, Line 13: SDW Does SDW stand for "Software Development (╯°□°)╯︵ lɐnuɐW"? 😄 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Comment-Date: Tue, 26 May 2020 15:53:37 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Hello build bot (Jenkins), Lijian Zhao, Nico Huber, Paul Menzel, Rizwan Qureshi, Subrata Banik, Aaron Durbin, Patrick Rudolph, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#5). Change subject: soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... soc/intel/common/block: Update microcode serialized for HT threads On Hyper-Threading 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. There's still some confusion as the Document: 253668-060US Intel SDM Chapter 9.11.6.3 and BWG state differently. It's not clear if the HT sibling must update the microcode at all, as a core likely shares the microcode unit between HT siblings. The recommondation of Linux engineers is to idle the sibling thread at least, and if possible idle as many cores as possible when updating. 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/include/cpu/x86/lapic.h M src/soc/intel/common/block/cpu/Kconfig M src/soc/intel/common/block/cpu/mp_init.c 5 files changed, 87 insertions(+), 2 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/5 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 5 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-MessageType: newpatchset
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... Patch Set 4: (11 comments) https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG Commit Message: https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@7 PS4, Line 7: soc/intel/common/block: Update microcode serialiced for HT threads
serialized Done
https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@9 PS4, Line 9: microcde
microcode Done
https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@11 PS4, Line 11: "microcode: Update failed" error seen in the console log.
On what board? Or on all? Done
https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@13 PS4, Line 13: state differently
… contradict each other. […] Done
https://review.coreboot.org/c/coreboot/+/35739/4//COMMIT_MSG@13 PS4, Line 13: SDW
Does SDW stand for "Software Development (╯°□°)╯︵ lɐnuɐW"? 😄 Done
https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... File src/cpu/intel/common/hyperthreading.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 40: = 0;
no need to initialize to 0 explicitly. Done
https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 46: 1;
this value shouldn't be open coded. SPIN_LOCK_UNLOCKED definition should be used. Done
https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 53: /* Is Hyper-Threading supported */ : if (!(cpuid_edx(1) & CPUID_FEAURE_HTT)) : return; :
This is duplicated below. […] Done
https://review.coreboot.org/c/coreboot/+/35739/4/src/cpu/intel/common/hypert... PS4, Line 57: if (!initialized)
Sorry, read it a few times more. You mean, we don't know CONFIG_MAX_CPUS, […] Done
https://review.coreboot.org/c/coreboot/+/35739/4/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/4/src/soc/intel/common/block/... PS4, Line 40: Tests
For which parts? There are many different implementations. Done
https://review.coreboot.org/c/coreboot/+/35739/4/src/soc/intel/common/block/... PS4, Line 42: and updates one
Some word seems to be missing here. Done
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 4 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Comment-Date: Mon, 15 Jun 2020 14:27:42 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph@9elements.com> Comment-In-Reply-To: Paul Menzel <paulepanter@users.sourceforge.net> Comment-In-Reply-To: Angel Pons <th3fanbus@gmail.com> Comment-In-Reply-To: Aaron Durbin <adurbin@chromium.org> Gerrit-MessageType: comment
Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... Patch Set 5: (2 comments) https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/... PS5, Line 46: intel_ht_sibling_spin_lock(); This is assuming that all microcode loading needs to be serialized on hyperthreads. There's a whole sequence in src/cpu/x86/sipi_vector.S that should be coordinated. I don't think we should have 2 different things colliding here regarding policy. See get_microcode_info() below where it provides the ability to load in parallel when starting up the APs in assembly. https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/... PS5, Line 139: *parallel = 1; See here regarding my previous comment. Clearly you have some devices which need the lock while others don't. I think the policy should follow based on opt-in behavior. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 5 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Comment-Date: Tue, 16 Jun 2020 18:52:55 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... Patch Set 5: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/cpu/mp_init.c File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/cpu/mp_init.c@139 PS5, Line 139: *parallel = 1; > See here regarding my previous comment. […] Should I document the existing policies first and then add a new one? That would be: * Serialize all updates * Serialize on HT cores (if HT enabled) * No serializing of microcode loading -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 5 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Comment-Date: Wed, 17 Jun 2020 06:49:50 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Aaron Durbin <adurbin@chromium.org> Gerrit-MessageType: comment
Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... Patch Set 5: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/... PS5, Line 139: *parallel = 1;
Should I document the existing policies first and then add a new one? […] Sounds like a good idea. Bigger question is why we are doing microcode updates above. Presumably it's because of needing to reload after SMM has been relocated? We should note why we're doing the same load twice. I know there are some cpus that wanted a 2nd reload (haswell, iirc). SGX, while note used, also needs another reload, if my memory serves.
I think we should document as well as ensure we select the right policy for each cpu because I don't think those options are necessary on all cpus. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 5 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Comment-Date: Wed, 17 Jun 2020 16:53:41 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph@9elements.com> Comment-In-Reply-To: Aaron Durbin <adurbin@chromium.org> Gerrit-MessageType: comment
Hello build bot (Jenkins), Lijian Zhao, Nico Huber, Paul Menzel, Rizwan Qureshi, Subrata Banik, Aaron Durbin, Patrick Rudolph, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#6). Change subject: [WIP]soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... [WIP]soc/intel/common/block: Update microcode serialized for HT threads On Hyper-Threading 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 BWG suggest doing this. Documented a few to get a common sense what should be done, when and how microcode should be updated. 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 Change-Id: I72804753e567a137a5648ca6950009fed332531b Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com> --- M Documentation/soc/intel/index.md A Documentation/soc/intel/microcode.md 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 10 files changed, 223 insertions(+), 21 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/6 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 6 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-MessageType: newpatchset
Hello build bot (Jenkins), Lijian Zhao, Nico Huber, Christian Walter, Paul Menzel, Rizwan Qureshi, Subrata Banik, Aaron Durbin, Patrick Rudolph, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#9). Change subject: [WIP]soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... [WIP]soc/intel/common/block: Update microcode serialized for HT threads On Hyper-Threading 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 BWG suggest doing this. Documented a few to get a common sense what should be done, when and how microcode should be updated. 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 Change-Id: I72804753e567a137a5648ca6950009fed332531b Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com> --- M Documentation/soc/intel/index.md A Documentation/soc/intel/microcode.md 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 10 files changed, 235 insertions(+), 21 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/9 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 9 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-MessageType: newpatchset
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP]soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... Patch Set 11: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/2/src/soc/intel/common/block/... PS2, Line 50: Update just on the first CPU in the core
Remove *on*? Ack
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 11 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Comment-Date: Wed, 05 Aug 2020 21:50:12 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Paul Menzel <paulepanter@users.sourceforge.net> Gerrit-MessageType: comment
Angel Pons has uploaded a new patch set (#12) to the change originally created by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP]soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... [WIP]soc/intel/common/block: Update microcode serialized for HT threads On Hyper-Threading 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 BWG suggest doing this. Documented a few to get a common sense what should be done, when and how microcode should be updated. 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 Change-Id: I72804753e567a137a5648ca6950009fed332531b Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com> --- M Documentation/soc/intel/index.md A Documentation/soc/intel/microcode.md 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 10 files changed, 244 insertions(+), 19 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/12 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 12 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> 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: Angel Pons <th3fanbus@gmail.com> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-MessageType: newpatchset
Angel Pons has uploaded a new patch set (#13) to the change originally created by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP]soc/intel/common/block: Update microcode serialized for HT threads ...................................................................... [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 https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/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
Angel Pons has uploaded a new patch set (#14) to the change originally created by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP] soc/intel/common/block: Serialize microcode updates for HT threads ...................................................................... [WIP] soc/intel/common/block: Serialize microcode updates 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 HyperThreading 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/14 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 14 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
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP] soc/intel/common/block: Serialize microcode updates for HT threads ...................................................................... Patch Set 13: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/2//COMMIT_MSG Commit Message: https://review.coreboot.org/c/coreboot/+/35739/2//COMMIT_MSG@10 PS2, Line 10: each core, not for each thread.
Nit: Intel Software Developer Guidelines speaks of cores and logical processors. Other documents talk about cores and threads. The commit message probably needs an update anyway.
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/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-Comment-Date: Tue, 11 Aug 2020 21:27:29 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Arthur Heymans <arthur@aheymans.xyz> Gerrit-MessageType: comment
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP] soc/intel/common/block: Serialize microcode updates for HT threads ...................................................................... Patch Set 14: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/3/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/3/src/soc/intel/common/block/... PS3, Line 40: Testes Tests -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 14 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-CC: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Comment-Date: Wed, 12 Aug 2020 19:44:39 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP] soc/intel/common/block: Serialize microcode updates for HT threads ...................................................................... Patch Set 14: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/3//COMMIT_MSG Commit Message: https://review.coreboot.org/c/coreboot/+/35739/3//COMMIT_MSG@14 PS3, Line 14: Is there a boot time penalty caused by serialization? -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 14 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-CC: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Comment-Date: Wed, 12 Aug 2020 20:00:10 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP] soc/intel/common/block: Serialize microcode updates for HT threads ...................................................................... Patch Set 14: (1 comment) https://review.coreboot.org/c/coreboot/+/35739/3//COMMIT_MSG Commit Message: https://review.coreboot.org/c/coreboot/+/35739/3//COMMIT_MSG@14 PS3, Line 14:
Is there a boot time penalty caused by serialization? I don't think you meant to comment on this old patchset? I haven't tried yet.
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 14 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-CC: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Comment-Date: Wed, 12 Aug 2020 21:53:34 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Paul Menzel <paulepanter@users.sourceforge.net> Gerrit-MessageType: comment
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: [WIP] soc/intel/common/block: Serialize microcode updates for HT threads ...................................................................... Patch Set 15: (4 comments) https://review.coreboot.org/c/coreboot/+/35739/2//COMMIT_MSG Commit Message: https://review.coreboot.org/c/coreboot/+/35739/2//COMMIT_MSG@10 PS2, 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.
https://review.coreboot.org/c/coreboot/+/35739/3/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/3/src/soc/intel/common/block/... PS3, Line 40: Testes
Tests Gone
https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/... File src/soc/intel/common/block/cpu/mp_init.c: https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/... PS5, 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.
https://review.coreboot.org/c/coreboot/+/35739/5/src/soc/intel/common/block/... PS5, 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 https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 15 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-CC: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Comment-Date: Wed, 23 Sep 2020 12:05:04 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph@9elements.com> Comment-In-Reply-To: Tim Wawrzynczak <twawrzynczak@chromium.org> Comment-In-Reply-To: Angel Pons <th3fanbus@gmail.com> Comment-In-Reply-To: Arthur Heymans <arthur@aheymans.xyz> Comment-In-Reply-To: Aaron Durbin <adurbin@chromium.org> Gerrit-MessageType: comment
Angel Pons has uploaded a new patch set (#17) to the change originally created by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/common/block: Serialize microcode updates for HT threads ...................................................................... soc/intel/common/block: Serialize microcode updates 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. Tested on out-of-tree Skylake-U board, still boots. 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/17 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 17 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-CC: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-MessageType: newpatchset
Tim Wawrzynczak 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: Code-Review+1 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 17 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: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Comment-Date: Wed, 28 Oct 2020 20:15:35 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
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! ? -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 17 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: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Comment-Date: Fri, 30 Oct 2020 21:31:38 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
Attention is currently required from: Nico Huber, Angel Pons. Patrick Rudolph 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) File src/cpu/intel/microcode/microcode.c: https://review.coreboot.org/c/coreboot/+/35739/comment/5f449801_51e375b2 PS17, Line 63: Update just on the first CPU in the core.
That's not what's happening here, is it? […] Yes, that's the summary. Do you mind having more background information in the code?
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 17 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: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Attention: Nico Huber <nico.h@gmx.de> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Comment-Date: Mon, 11 Jan 2021 13:13:22 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Gerrit-MessageType: comment
Attention is currently required from: Patrick Rudolph, Angel Pons. 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) File src/cpu/intel/microcode/microcode.c: https://review.coreboot.org/c/coreboot/+/35739/comment/fa65527a_1652dd97 PS17, Line 63: Update just on the first CPU in the core.
Yes, that's the summary. […] Ufff, it's been a while, got to interpret my own comment here...
The part I highlighted seems wrong. That's why I commented in the first place. What I probably meant was that we should not talk about anything beyond these four topics, because everything else is a distraction and may or may not be true. Background information is nice, as long as it doesn't make assumptions. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 17 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: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Attention: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Comment-Date: Thu, 14 Jan 2021 20:56:52 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-MessageType: comment
Attention is currently required from: Patrick Rudolph, Angel Pons. 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: Code-Review+1 (1 comment) Patchset: PS17: Code looks good so far. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 17 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: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Attention: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Comment-Date: Thu, 14 Jan 2021 21:21:57 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Attention is currently required from: Patrick Rudolph, Angel Pons. Hello build bot (Jenkins), Lijian Zhao, Nico Huber, Tim Wawrzynczak, Paul Menzel, Christian Walter, Rizwan Qureshi, Angel Pons, Subrata Banik, Aaron Durbin, Patrick Rudolph, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#18). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c M src/soc/intel/xeon_sp/skx/cpu.c 27 files changed, 171 insertions(+), 60 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/18 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 18 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: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Arthur Heymans <arthur@aheymans.xyz> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Attention: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Hello build bot (Jenkins), Lijian Zhao, Mariusz Szafrański, Paul Menzel, Rizwan Qureshi, Angel Pons, Subrata Banik, Andrey Petrov, Aaron Durbin, Patrick Rudolph, Nico Huber, Suresh Bellampalli, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Michal Motyl, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#20). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c M src/soc/intel/xeon_sp/skx/cpu.c 27 files changed, 171 insertions(+), 60 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/20 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 20 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Hello build bot (Jenkins), Lijian Zhao, Mariusz Szafrański, Paul Menzel, Rizwan Qureshi, Angel Pons, Subrata Banik, Andrey Petrov, Aaron Durbin, Patrick Rudolph, Nico Huber, Suresh Bellampalli, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Michal Motyl, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#22). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c M src/soc/intel/xeon_sp/skx/cpu.c 27 files changed, 171 insertions(+), 55 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/22 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 22 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Hello build bot (Jenkins), Lijian Zhao, Mariusz Szafrański, Paul Menzel, Rizwan Qureshi, Angel Pons, Subrata Banik, Andrey Petrov, Aaron Durbin, Patrick Rudolph, Nico Huber, Suresh Bellampalli, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Michal Motyl, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#23). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c M src/soc/intel/xeon_sp/skx/cpu.c 27 files changed, 172 insertions(+), 55 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/23 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 23 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Hello build bot (Jenkins), Lijian Zhao, Mariusz Szafrański, Paul Menzel, Rizwan Qureshi, Angel Pons, Subrata Banik, Andrey Petrov, Aaron Durbin, Patrick Rudolph, Nico Huber, Suresh Bellampalli, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Michal Motyl, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#25). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c M src/soc/intel/xeon_sp/skx/cpu.c 27 files changed, 172 insertions(+), 55 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/25 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 25 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Hello build bot (Jenkins), Lijian Zhao, Mariusz Szafrański, Paul Menzel, Rizwan Qureshi, Angel Pons, Subrata Banik, Andrey Petrov, Aaron Durbin, Patrick Rudolph, Nico Huber, Suresh Bellampalli, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Michal Motyl, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#26). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c M src/soc/intel/xeon_sp/skx/cpu.c 27 files changed, 172 insertions(+), 55 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/26 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 26 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Hello build bot (Jenkins), Lijian Zhao, Mariusz Szafrański, Paul Menzel, Rizwan Qureshi, Angel Pons, Subrata Banik, Andrey Petrov, Aaron Durbin, Patrick Rudolph, Nico Huber, Suresh Bellampalli, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Michal Motyl, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#27). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c M src/soc/intel/xeon_sp/skx/cpu.c 27 files changed, 172 insertions(+), 55 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/27 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 27 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Hello build bot (Jenkins), Lijian Zhao, Mariusz Szafrański, Paul Menzel, Rizwan Qureshi, Angel Pons, Subrata Banik, Andrey Petrov, Aaron Durbin, Patrick Rudolph, Nico Huber, Suresh Bellampalli, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Michal Motyl, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#28). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c M src/soc/intel/xeon_sp/skx/cpu.c 27 files changed, 172 insertions(+), 55 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/28 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 28 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Patrick Rudolph, Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Hello build bot (Jenkins), Lijian Zhao, Mariusz Szafrański, Paul Menzel, Rizwan Qureshi, Angel Pons, Subrata Banik, Andrey Petrov, Aaron Durbin, Patrick Rudolph, Nico Huber, Suresh Bellampalli, Tim Wawrzynczak, Christian Walter, Vanessa Eusebio, Michal Motyl, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/35739 to look at the new patch set (#29). Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... soc/intel/*: Serialize microcode updates for HT threads This change only affects MPinit related microcode updates and is a NOP for CPUs without HyperThreading enabled. 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 HyperThreading enabled. The SDM and most of the BWGs suggest doing this. 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/haswell/haswell_init.c M src/cpu/intel/microcode/microcode.c M src/cpu/intel/model_1067x/mp_init.c M src/cpu/intel/model_2065x/model_2065x_init.c M src/cpu/intel/model_206ax/model_206ax_init.c M src/include/cpu/intel/microcode.h M src/include/cpu/x86/lapic.h M src/soc/intel/alderlake/cpu.c M src/soc/intel/apollolake/cpu.c M src/soc/intel/baytrail/cpu.c M src/soc/intel/braswell/cpu.c M src/soc/intel/cannonlake/cpu.c 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 M src/soc/intel/denverton_ns/cpu.c M src/soc/intel/elkhartlake/cpu.c M src/soc/intel/icelake/cpu.c M src/soc/intel/jasperlake/cpu.c M src/soc/intel/skylake/chip.c M src/soc/intel/skylake/cpu.c M src/soc/intel/skylake/include/soc/ramstage.h M src/soc/intel/tigerlake/cpu.c M src/soc/intel/xeon_sp/cpx/cpu.c 26 files changed, 169 insertions(+), 55 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/35739/29 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 29 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-MessageType: newpatchset
Attention is currently required from: Nico Huber, Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... Patch Set 30: (1 comment) File src/cpu/intel/microcode/microcode.c: https://review.coreboot.org/c/coreboot/+/35739/comment/3d235e2d_9a783e3b PS17, Line 63: Update just on the first CPU in the core.
Ufff, it's been a while, got to interpret my own comment here... […] Rephrased the text
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 30 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Nico Huber <nico.h@gmx.de> Gerrit-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-Comment-Date: Thu, 28 Jan 2021 13:09:04 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-MessageType: comment
Attention is currently required from: Patrick Rudolph, Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Angel Pons, Michal Motyl, Andrey Petrov. Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... Patch Set 31: (1 comment) File src/cpu/intel/microcode/microcode.c: https://review.coreboot.org/c/coreboot/+/35739/comment/7a7bfd91_d3b42420 PS17, Line 63: Update just on the first CPU in the core.
Rephrased the text Done
-- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 31 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Angel Pons <th3fanbus@gmail.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-Comment-Date: Wed, 03 Feb 2021 19:37:09 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Nico Huber <nico.h@gmx.de> Comment-In-Reply-To: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-MessageType: comment
Attention is currently required from: Patrick Rudolph, Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Michal Motyl, Andrey Petrov. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... Patch Set 31: Code-Review+1 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 31 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-Comment-Date: Wed, 03 Feb 2021 19:49:59 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Attention is currently required from: Patrick Rudolph, Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Michal Motyl, Andrey Petrov. Rocky Phagura has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... Patch Set 31: Code-Review+1 (1 comment) Patchset: PS31: Hi, Any updates on this patch? -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 31 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> 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: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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@mailbox.org> Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi@intel.com> Gerrit-Reviewer: Rocky Phagura Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> Gerrit-Reviewer: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@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-Attention: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-Comment-Date: Tue, 18 May 2021 16:49:44 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Attention is currently required from: Patrick Rudolph, Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Michal Motyl, Andrey Petrov. Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35739 ) Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... Patch Set 31: Code-Review+2 -- To view, visit https://review.coreboot.org/c/coreboot/+/35739 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 31 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com> Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz> Gerrit-Reviewer: Christian Walter <christian.walter@9elements.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao@intel.com> Gerrit-Reviewer: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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@mailbox.org> Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi@intel.com> Gerrit-Reviewer: Rocky Phagura Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> Gerrit-Reviewer: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak@chromium.org> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Furquan Shaikh <furquan@google.com> Gerrit-Attention: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Attention: Mariusz Szafrański <mariuszx.szafranski@intel.com> Gerrit-Attention: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Attention: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Attention: Michal Motyl <michalx.motyl@intel.com> Gerrit-Attention: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-Comment-Date: Sat, 21 Aug 2021 07:48:04 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/35739?usp=email ) Change subject: soc/intel/*: Serialize microcode updates for HT threads ...................................................................... Abandoned This patch has not been touched in over 12 months. Anyone who wants to take over work on this patch, please feel free to restore it and do any work needed to get it merged. If you create a new patch based on this work, please credit the original author. -- To view, visit https://review.coreboot.org/c/coreboot/+/35739?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I72804753e567a137a5648ca6950009fed332531b Gerrit-Change-Number: 35739 Gerrit-PatchSet: 31 Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> Gerrit-Reviewer: Andrey Petrov <andrey.petrov@gmail.com> Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com> Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz> Gerrit-Reviewer: Christian Walter <christian.walter@9elements.com> Gerrit-Reviewer: Lijian Zhao <lijian.zhao@intel.com> Gerrit-Reviewer: Mariusz Szafrański <mariuszx@akumat.pl> Gerrit-Reviewer: Michal Motyl <michalx.motyl@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@mailbox.org> Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi@intel.com> Gerrit-Reviewer: Rocky Phagura Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> Gerrit-Reviewer: Suresh Bellampalli <suresh.bellampalli@intel.com> Gerrit-Reviewer: Tim Wawrzynczak <inforichland@gmail.com> Gerrit-Reviewer: Vanessa Eusebio <vanessa.f.eusebio@intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Furquan Shaikh <furquan.m.shaikh@gmail.com> Gerrit-CC: Martin L Roth <gaumless@gmail.com> Gerrit-MessageType: abandon
participants (9)
-
Aaron Durbin (Code Review) -
Angel Pons (Code Review)
-
Arthur Heymans (Code Review)
-
Martin L Roth (Code Review)
-
Nico Huber (Code Review)
-
Patrick Rudolph (Code Review)
-
Paul Menzel (Code Review)
-
Rocky Phagura (Code Review)
-
Tim Wawrzynczak (Code Review)