• Suresh Siddha's avatar
    x86, cpuid: Add "volatile" to asm in native_cpuid() · c9ac6a9e
    Suresh Siddha authored
    commit 45a94d7c upstream.
    
    xsave_cntxt_init() does something like:
    
    	cpuid(0xd, ..);	// find out what features FP/SSE/.. etc are supported
    
    	xsetbv();	// enable the features known to OS
    
    	cpuid(0xd, ..);	// find out the size of the context for features enabled
    
    Depending on what features get enabled in xsetbv(), value of the
    cpuid.eax=0xd.ecx=0.ebx changes correspondingly (representing the
    size of the context that is enabled).
    
    As we don't have volatile keyword for native_cpuid(), gcc 4.1.2
    optimizes away the second cpuid and the kernel continues to use
    the cpuid information obtained before xsetbv(), ultimately leading to kernel
    crash on processors supporting more state than the legacy FP/SSE.
    
    Add "volatile" for native_cpuid().
    Signed-off-by: default avatarSuresh Siddha <suresh.b.siddha@intel.com>
    LKML-Reference: <1261009542.2745.55.camel@sbs-t61.sc.intel.com>
    Signed-off-by: default avatarH. Peter Anvin <hpa@zytor.com>
    Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
    c9ac6a9e
processor.h 25 KB