• Suresh Siddha's avatar
    x86, cpuid: Add "volatile" to asm in native_cpuid() · 45a94d7c
    Suresh Siddha authored
    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>
    Cc: stable@kernel.org
    Signed-off-by: default avatarH. Peter Anvin <hpa@zytor.com>
    45a94d7c
processor.h 25 KB