Module Name: src Committed By: maxv Date: Sat Nov 11 11:00:47 UTC 2017
Modified Files: src/sys/arch/x86/x86: cpu.c fpu.c identcpu.c src/sys/arch/xen/x86: cpu.c Log Message: Recommit http://mail-index.netbsd.org/source-changes/2017/11/08/msg089525.html but use __INITIAL_MXCSR_MASK__ on Xen until someone figures out what's wrong with the Xen fpu. To generate a diff of this commit: cvs rdiff -u -r1.140 -r1.141 src/sys/arch/x86/x86/cpu.c cvs rdiff -u -r1.26 -r1.27 src/sys/arch/x86/x86/fpu.c cvs rdiff -u -r1.66 -r1.67 src/sys/arch/x86/x86/identcpu.c cvs rdiff -u -r1.115 -r1.116 src/sys/arch/xen/x86/cpu.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/arch/x86/x86/cpu.c diff -u src/sys/arch/x86/x86/cpu.c:1.140 src/sys/arch/x86/x86/cpu.c:1.141 --- src/sys/arch/x86/x86/cpu.c:1.140 Sat Nov 11 09:10:19 2017 +++ src/sys/arch/x86/x86/cpu.c Sat Nov 11 11:00:46 2017 @@ -1,4 +1,4 @@ -/* $NetBSD: cpu.c,v 1.140 2017/11/11 09:10:19 bouyer Exp $ */ +/* $NetBSD: cpu.c,v 1.141 2017/11/11 11:00:46 maxv Exp $ */ /* * Copyright (c) 2000-2012 NetBSD Foundation, Inc. @@ -62,7 +62,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.140 2017/11/11 09:10:19 bouyer Exp $"); +__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.141 2017/11/11 11:00:46 maxv Exp $"); #include "opt_ddb.h" #include "opt_mpbios.h" /* for MPDEBUG */ @@ -541,6 +541,7 @@ cpu_childdetached(device_t self, device_ void cpu_init(struct cpu_info *ci) { + extern int x86_fpu_save; uint32_t cr4 = 0; lcr0(rcr0() | CR0_WP); @@ -584,6 +585,10 @@ cpu_init(struct cpu_info *ci) lcr4(cr4); } + if (x86_fpu_save >= FPU_SAVE_FXSAVE) { + fpuinit_mxcsr_mask(); + } + /* If xsave is enabled, enable all fpu features */ if (cr4 & CR4_OSXSAVE) wrxcr(0, x86_xsave_features & XCR0_FPU); Index: src/sys/arch/x86/x86/fpu.c diff -u src/sys/arch/x86/x86/fpu.c:1.26 src/sys/arch/x86/x86/fpu.c:1.27 --- src/sys/arch/x86/x86/fpu.c:1.26 Sat Nov 11 09:10:19 2017 +++ src/sys/arch/x86/x86/fpu.c Sat Nov 11 11:00:46 2017 @@ -1,4 +1,4 @@ -/* $NetBSD: fpu.c,v 1.26 2017/11/11 09:10:19 bouyer Exp $ */ +/* $NetBSD: fpu.c,v 1.27 2017/11/11 11:00:46 maxv Exp $ */ /* * Copyright (c) 2008 The NetBSD Foundation, Inc. All @@ -96,7 +96,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: fpu.c,v 1.26 2017/11/11 09:10:19 bouyer Exp $"); +__KERNEL_RCSID(0, "$NetBSD: fpu.c,v 1.27 2017/11/11 11:00:46 maxv Exp $"); #include "opt_multiprocessor.h" @@ -235,21 +235,20 @@ fpuinit_mxcsr_mask(void) { #ifndef XEN union savefpu fpusave __aligned(16); - u_long cr0, psl; + u_long psl; memset(&fpusave, 0, sizeof(fpusave)); /* Disable interrupts, and enable FPU */ psl = x86_read_psl(); x86_disable_intr(); - cr0 = rcr0(); - lcr0(cr0 & ~(CR0_EM|CR0_TS)); + clts(); /* Fill in the FPU area */ fxsave(&fpusave); /* Restore previous state */ - lcr0(cr0); + stts(); x86_write_psl(psl); if (fpusave.sv_xmm.fx_mxcsr_mask == 0) { @@ -258,9 +257,6 @@ fpuinit_mxcsr_mask(void) x86_fpu_mxcsr_mask = fpusave.sv_xmm.fx_mxcsr_mask; } #else - /* - * XXX: Does the detection above work on Xen? - */ x86_fpu_mxcsr_mask = __INITIAL_MXCSR_MASK__; #endif } Index: src/sys/arch/x86/x86/identcpu.c diff -u src/sys/arch/x86/x86/identcpu.c:1.66 src/sys/arch/x86/x86/identcpu.c:1.67 --- src/sys/arch/x86/x86/identcpu.c:1.66 Sat Nov 11 09:10:19 2017 +++ src/sys/arch/x86/x86/identcpu.c Sat Nov 11 11:00:46 2017 @@ -1,4 +1,4 @@ -/* $NetBSD: identcpu.c,v 1.66 2017/11/11 09:10:19 bouyer Exp $ */ +/* $NetBSD: identcpu.c,v 1.67 2017/11/11 11:00:46 maxv Exp $ */ /*- * Copyright (c) 1999, 2000, 2001, 2006, 2007, 2008 The NetBSD Foundation, Inc. @@ -30,7 +30,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: identcpu.c,v 1.66 2017/11/11 09:10:19 bouyer Exp $"); +__KERNEL_RCSID(0, "$NetBSD: identcpu.c,v 1.67 2017/11/11 11:00:46 maxv Exp $"); #include "opt_xen.h" @@ -753,8 +753,6 @@ cpu_probe_fpu(struct cpu_info *ci) x86_fpu_save = FPU_SAVE_FXSAVE; - fpuinit_mxcsr_mask(); - /* See if xsave (for AVX) is supported */ if ((ci->ci_feat_val[1] & CPUID2_XSAVE) == 0) return; Index: src/sys/arch/xen/x86/cpu.c diff -u src/sys/arch/xen/x86/cpu.c:1.115 src/sys/arch/xen/x86/cpu.c:1.116 --- src/sys/arch/xen/x86/cpu.c:1.115 Sat Nov 11 09:10:19 2017 +++ src/sys/arch/xen/x86/cpu.c Sat Nov 11 11:00:47 2017 @@ -1,4 +1,4 @@ -/* $NetBSD: cpu.c,v 1.115 2017/11/11 09:10:19 bouyer Exp $ */ +/* $NetBSD: cpu.c,v 1.116 2017/11/11 11:00:47 maxv Exp $ */ /*- * Copyright (c) 2000 The NetBSD Foundation, Inc. @@ -65,7 +65,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.115 2017/11/11 09:10:19 bouyer Exp $"); +__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.116 2017/11/11 11:00:47 maxv Exp $"); #include "opt_ddb.h" #include "opt_multiprocessor.h" @@ -540,6 +540,10 @@ cpu_init(struct cpu_info *ci) lcr4(rcr4() | CR4_OSXMMEXCPT); } + if (x86_fpu_save >= FPU_SAVE_FXSAVE) { + fpuinit_mxcsr_mask(); + } + atomic_or_32(&ci->ci_flags, CPUF_RUNNING); }