• Torsten Kaiser's avatar
    x86, microcode, AMD: Make cpu_has_amd_erratum() use the correct struct cpuinfo_x86 · 8c6b79bb
    Torsten Kaiser authored
    cpu_has_amd_erratum() is buggy, because it uses the per-cpu cpu_info
    before it is filled by smp_store_boot_cpu_info() / smp_store_cpu_info().
    
    If early microcode loading is enabled its collect_cpu_info_amd_early()
    will fill ->x86 and so the fallback to boot_cpu_data is not used. But
    ->x86_vendor was not filled and is still X86_VENDOR_INTEL resulting in
    no errata fixes getting applied and my system hangs on boot.
    
    Using cpu_info in cpu_has_amd_erratum() is wrong anyway: its only
    caller init_amd() will have a struct cpuinfo_x86 as parameter and the
    set_cpu_bug() that is controlled by cpu_has_amd_erratum() also only uses
    that struct.
    
    So pass the struct cpuinfo_x86 from init_amd() to cpu_has_amd_erratum()
    and the broken fallback can be dropped.
    
    [ Boris: Drop WARN_ON() since we're called only from init_amd() ]
    Signed-off-by: default avatarTorsten Kaiser <just.for.lkml@googlemail.com>
    Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
    8c6b79bb
amd.c 22.1 KB