tag 736892 fixed-upstream
thanks

On Tue, 04 Feb 2014, Stephen Powell wrote:
> On Sat, 01 Feb 2014 20:25:30 -0500 (EST), Henrique de Moraes Holschuh wrote:
> > 
> > Can you confirm that, at the moment of this /proc/cpuinfo, sysfs was showing
> > the microcode of cpu1 and cpu3 to be outdated (i.e. not 0x6000822) ?
> 
> Yes, absolutely.

Ah, found this:

commit accd1e823ed1d5980106dd522a4c535084400830
Author: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com>
Date:   Wed Sep 29 19:27:12 2010 -0500

    x86/microcode/AMD: Fix patch level reporting for family 15h
    
    On AMD family 14h, applying microcode patch on the a core (core0)
    would also affect the other core (core1) in the same compute
    unit. The driver would skip applying the patch on core1, but it
    still need to update kernel structures to reflect the proper
    patch level.
    
    The current logic is not updating the struct
    ucode_cpu_info.cpu_sig.rev of the skipped core. This causes the
    /sys/devices/system/cpu/cpu1/microcode/version to report
    incorrect patch level as shown below:
    
      $ grep . cpu?/microcode/version
      cpu0/microcode/version:0x600063d
      cpu1/microcode/version:0x6000626
      cpu2/microcode/version:0x600063d
      cpu3/microcode/version:0x6000626
      cpu4/microcode/version:0x600063d
    
    Signed-off-by: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com>
    Acked-by: Borislav Petkov <b...@suse.de>
    Cc: <b...@alien8.de>
    Cc: <jacob.w.s...@gmail.com>
    Cc: <herrmann.der.u...@googlemail.com>
    Link: 
http://lkml.kernel.org/r/1285806432-1995-1-git-send-email-suravee.suthikulpa...@amd.com
    Signed-off-by: Ingo Molnar <mi...@kernel.org>

diff --git a/arch/x86/kernel/microcode_amd.c
b/arch/x86/kernel/microcode_amd.c
index 7123b5d..af99f71 100644
--- a/arch/x86/kernel/microcode_amd.c
+++ b/arch/x86/kernel/microcode_amd.c
@@ -216,6 +216,7 @@ int apply_microcode_amd(int cpu)
        /* need to apply patch? */
        if (rev >= mc_amd->hdr.patch_id) {
                c->microcode = rev;
+               uci->cpu_sig.rev = rev;
                return 0;
        }

----- 

I will pester kernel upstream to backport this stuff to the stable kernels.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to