Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/arch/x86/x86 No, I meant to put the panic in fpudna not ...



details:   https://anonhg.NetBSD.org/src/rev/3c1d75b72b00
branches:  trunk
changeset: 833265:3c1d75b72b00
user:      maxv <maxv%NetBSD.org@localhost>
date:      Sun Jun 17 06:03:40 2018 +0000

description:
No, I meant to put the panic in fpudna not fputrap. Also appease it: panic
only if the fpu already has a state. We're fine with getting a DNA, what
we're not fine with is if the DNA is received while the FPU is busy.

I believe (even though I couldn't trigger it) that the panic would
otherwise fire if PT_SETFPREGS is used. And also ACPI sleep/wakeup,
probably.

diffstat:

 sys/arch/x86/x86/fpu.c |  18 ++++++++++++------
 1 files changed, 12 insertions(+), 6 deletions(-)

diffs (53 lines):

diff -r de9fd9c4e087 -r 3c1d75b72b00 sys/arch/x86/x86/fpu.c
--- a/sys/arch/x86/x86/fpu.c    Sun Jun 17 01:08:15 2018 +0000
+++ b/sys/arch/x86/x86/fpu.c    Sun Jun 17 06:03:40 2018 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: fpu.c,v 1.36 2018/06/16 17:11:13 maxv Exp $    */
+/*     $NetBSD: fpu.c,v 1.37 2018/06/17 06:03:40 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.36 2018/06/16 17:11:13 maxv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: fpu.c,v 1.37 2018/06/17 06:03:40 maxv Exp $");
 
 #include "opt_multiprocessor.h"
 
@@ -397,10 +397,6 @@
        if (!USERMODE(frame->tf_cs))
                panic("fpu trap from kernel, trapframe %p\n", frame);
 
-       if (__predict_false(x86_fpu_eager)) {
-               panic("%s: got DNA with EagerFPU enabled", __func__);
-       }
-
        /*
         * At this point, fpcurlwp should be curlwp.  If it wasn't, the TS bit
         * should be set, and we should have gotten a DNA exception.
@@ -473,6 +469,11 @@
        pcb = lwp_getpcb(l);
        fl = ci->ci_fpcurlwp;
        if (fl != NULL) {
+               if (__predict_false(x86_fpu_eager)) {
+                       panic("%s: FPU busy with EagerFPU enabled",
+                           __func__);
+               }
+
                /*
                 * It seems we can get here on Xen even if we didn't
                 * switch lwp.  In this case do nothing
@@ -488,6 +489,11 @@
 
        /* Save our state if on a remote CPU. */
        if (pcb->pcb_fpcpu != NULL) {
+               if (__predict_false(x86_fpu_eager)) {
+                       panic("%s: LWP busy with EagerFPU enabled",
+                           __func__);
+               }
+
                /* Explicitly disable preemption before dropping spl. */
                kpreempt_disable();
                splx(s);



Home | Main Index | Thread Index | Old Index