RE: [PATCH 1/3] x86, intel: Output microcode revision v4

From: Yu, Fenghua
Date: Mon Aug 29 2011 - 21:13:31 EST


>--- a/arch/x86/kernel/cpu/intel.c
>+++ b/arch/x86/kernel/cpu/intel.c
>@@ -364,6 +364,15 @@ static void __cpuinit init_intel(struct cpuinfo_x86 *c)
>
> early_init_intel(c);
>
>+ if (c->x86 >= 6 && !cpu_has(c, X86_FEATURE_IA64)) {
>+ unsigned lower_word;
>+
>+ wrmsr(MSR_IA32_UCODE_REV, 0, 0);
>+ /* The CPUID 1 fills in the MSR as documented in the SDM */
>+ cpuid_eax(1);
>+ rdmsr(MSR_IA32_UCODE_REV, lower_word, c->microcode);
>+ }
>+

The patch initializes c->microcode AFTER it was used in early_init_intel(c) in your patch set [2/3].

Thanks.

-Fenghua
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/