Messages in this thread Patch in this message | | | From | Ashok Raj <> | Subject | [PATCH] x86/microcode: Check microcode revision before updating sibling threads | Date | Fri, 16 Feb 2018 10:46:48 -0800 |
| |
After updating microcode on one of the threads in the core, the thread sibling automatically gets the update since the microcode resources are shared. Check the ucode revision on the cpu before performing a ucode update.
Signed-off-by: Ashok Raj <ashok.raj@intel.com> Cc: X86 ML <x86@kernel.org> Cc: LKML <linux-kernel@vger.kernel.org> --- arch/x86/kernel/cpu/microcode/intel.c | 11 +++++++++++ 1 file changed, 11 insertions(+)
diff --git a/arch/x86/kernel/cpu/microcode/intel.c b/arch/x86/kernel/cpu/microcode/intel.c index 09b95a7..5802c2f 100644 --- a/arch/x86/kernel/cpu/microcode/intel.c +++ b/arch/x86/kernel/cpu/microcode/intel.c @@ -786,11 +786,21 @@ static enum ucode_state apply_microcode_intel(int cpu) uci = ucode_cpu_info + cpu; mc = uci->mc; + if (!mc) { /* Look for a newer patch in our cache: */ mc = find_patch(uci); if (!mc) return UCODE_NFOUND; + } else { + rev = intel_get_microcode_revision(); + /* + * Its possible the microcode got udpated + * because its sibling update was done earlier. + * Skip the udpate in that case. + */ + if (rev == mc->hdr.rev) + goto done; } /* write microcode via MSR 0x79 */ @@ -813,6 +823,7 @@ static enum ucode_state apply_microcode_intel(int cpu) prev_rev = rev; } +done: c = &cpu_data(cpu); uci->cpu_sig.rev = rev; -- 2.7.4
| |