lkml.org 
[lkml]   [2017]   [Jan]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2 5/6] x86/fpu: Fix CPUID-less FPU detection
    Date
    The old code didn't work at all because it adjusted the current caps
    instead of the forced caps. Anything it did would be undone later
    during cpu identification. Fix that and, while we're at it, improve
    the logging and don't bother running it if CPUID is available.

    Reported-by: Matthew Whitehead <tedheadster@gmail.com>
    Signed-off-by: Andy Lutomirski <luto@kernel.org>
    ---
    arch/x86/kernel/fpu/init.c | 29 ++++++++++++++++-------------
    1 file changed, 16 insertions(+), 13 deletions(-)

    diff --git a/arch/x86/kernel/fpu/init.c b/arch/x86/kernel/fpu/init.c
    index 60dece392b3a..23601db7b90f 100644
    --- a/arch/x86/kernel/fpu/init.c
    +++ b/arch/x86/kernel/fpu/init.c
    @@ -48,13 +48,7 @@ void fpu__init_cpu(void)
    fpu__init_cpu_xstate();
    }

    -/*
    - * The earliest FPU detection code.
    - *
    - * Set the X86_FEATURE_FPU CPU-capability bit based on
    - * trying to execute an actual sequence of FPU instructions:
    - */
    -static void fpu__init_system_early_generic(struct cpuinfo_x86 *c)
    +static bool fpu__probe_without_cpuid(void)
    {
    unsigned long cr0;
    u16 fsw, fcw;
    @@ -65,14 +59,23 @@ static void fpu__init_system_early_generic(struct cpuinfo_x86 *c)
    cr0 &= ~(X86_CR0_TS | X86_CR0_EM);
    write_cr0(cr0);

    - if (!test_bit(X86_FEATURE_FPU, (unsigned long *)cpu_caps_cleared)) {
    - asm volatile("fninit ; fnstsw %0 ; fnstcw %1"
    - : "+m" (fsw), "+m" (fcw));
    + asm volatile("fninit ; fnstsw %0 ; fnstcw %1"
    + : "+m" (fsw), "+m" (fcw));
    +
    + pr_info("x86/fpu: Probing for FPU: FSW=0x%04hx FCW=0x%04hx\n",
    + fsw, fcw);

    - if (fsw == 0 && (fcw & 0x103f) == 0x003f)
    - set_cpu_cap(c, X86_FEATURE_FPU);
    + return fsw == 0 && (fcw & 0x103f) == 0x003f;
    +}
    +
    +static void fpu__init_system_early_generic(struct cpuinfo_x86 *c)
    +{
    + if (!boot_cpu_has(X86_FEATURE_CPUID) &&
    + !test_bit(X86_FEATURE_FPU, (unsigned long *)cpu_caps_cleared)) {
    + if (fpu__probe_without_cpuid())
    + setup_force_cpu_cap(X86_FEATURE_FPU);
    else
    - clear_cpu_cap(c, X86_FEATURE_FPU);
    + setup_clear_cpu_cap(X86_FEATURE_FPU);
    }

    #ifndef CONFIG_MATH_EMULATION
    --
    2.9.3
    \
     
     \ /
      Last update: 2017-01-18 20:18    [W:4.056 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site