kernel - remove syscall_mpsafe and trap_mpsafe sysctls
authorMatthew Dillon <dillon@apollo.backplane.com>
Mon, 30 Aug 2010 04:45:24 +0000 (21:45 -0700)
committerMatthew Dillon <dillon@apollo.backplane.com>
Mon, 30 Aug 2010 04:45:24 +0000 (21:45 -0700)
* traps and system calls are always mpsafe, we don't need
  these sysctls any more.

sys/platform/pc32/i386/trap.c
sys/platform/pc64/x86_64/trap.c
sys/platform/vkernel/i386/trap.c
sys/platform/vkernel64/x86_64/trap.c

index 5e8f5cf..a20964b 100644 (file)
@@ -183,16 +183,6 @@ SYSCTL_INT(_machdep, OID_AUTO, fast_release, CTLFLAG_RW,
 static int slow_release;
 SYSCTL_INT(_machdep, OID_AUTO, slow_release, CTLFLAG_RW,
        &slow_release, 0, "Passive Release was nonoptimal");
-#ifdef SMP
-static int syscall_mpsafe = 1;
-SYSCTL_INT(_kern, OID_AUTO, syscall_mpsafe, CTLFLAG_RW,
-       &syscall_mpsafe, 0, "Allow MPSAFE marked syscalls to run without BGL");
-TUNABLE_INT("kern.syscall_mpsafe", &syscall_mpsafe);
-static int trap_mpsafe = 1;
-SYSCTL_INT(_kern, OID_AUTO, trap_mpsafe, CTLFLAG_RW,
-       &trap_mpsafe, 0, "Allow traps to mostly run without the BGL");
-TUNABLE_INT("kern.trap_mpsafe", &trap_mpsafe);
-#endif
 
 MALLOC_DEFINE(M_SYSMSG, "sysmsg", "sysmsg structure");
 extern int max_sysmsg;
@@ -440,11 +430,6 @@ trap(struct trapframe *frame)
                cpu_enable_intr();
        }
 
-#ifdef SMP
-       if (trap_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
-#endif
-
        --gd->gd_trap_nesting_level;
 
        if (!(frame->tf_eflags & PSL_I)) {
@@ -1199,9 +1184,8 @@ syscall2(struct trapframe *frame)
                frame->tf_eax);
 
 #ifdef SMP
-       KASSERT(td->td_mpcount == 0, ("badmpcount syscall2 from %p", (void *)frame->tf_eip));
-       if (syscall_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
+       KASSERT(td->td_mpcount == 0,
+               ("badmpcount syscall2 from %p", (void *)frame->tf_eip));
 #endif
        userenter(td, p);       /* lazy raise our priority */
 
index 6b47488..cb0f3ba 100644 (file)
@@ -154,16 +154,6 @@ SYSCTL_INT(_machdep, OID_AUTO, fast_release, CTLFLAG_RW,
 static int slow_release;
 SYSCTL_INT(_machdep, OID_AUTO, slow_release, CTLFLAG_RW,
        &slow_release, 0, "Passive Release was nonoptimal");
-#ifdef SMP
-static int syscall_mpsafe = 1;
-SYSCTL_INT(_kern, OID_AUTO, syscall_mpsafe, CTLFLAG_RW,
-       &syscall_mpsafe, 0, "Allow MPSAFE marked syscalls to run without BGL");
-TUNABLE_INT("kern.syscall_mpsafe", &syscall_mpsafe);
-static int trap_mpsafe = 1;
-SYSCTL_INT(_kern, OID_AUTO, trap_mpsafe, CTLFLAG_RW,
-       &trap_mpsafe, 0, "Allow traps to mostly run without the BGL");
-TUNABLE_INT("kern.trap_mpsafe", &trap_mpsafe);
-#endif
 
 /*
  * Passively intercepts the thread switch function to increase
@@ -403,14 +393,6 @@ trap(struct trapframe *frame)
 
        eva = 0;
 
-#ifdef SMP
-        if (trap_mpsafe == 0) {
-               ++gd->gd_trap_nesting_level;
-               MAKEMPSAFE(have_mplock);
-               --gd->gd_trap_nesting_level;
-       }
-#endif
-
        if ((frame->tf_rflags & PSL_I) == 0) {
                /*
                 * Buggy application or kernel code has disabled interrupts
@@ -1057,9 +1039,8 @@ syscall2(struct trapframe *frame)
                frame->tf_eax);
 
 #ifdef SMP
-       KASSERT(td->td_mpcount == 0, ("badmpcount syscall2 from %p", (void *)frame->tf_rip));
-       if (syscall_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
+       KASSERT(td->td_mpcount == 0,
+               ("badmpcount syscall2 from %p", (void *)frame->tf_rip));
 #endif
        userenter(td, p);       /* lazy raise our priority */
 
index c0e4389..e46541a 100644 (file)
@@ -168,16 +168,6 @@ SYSCTL_INT(_machdep, OID_AUTO, fast_release, CTLFLAG_RW,
 static int slow_release;
 SYSCTL_INT(_machdep, OID_AUTO, slow_release, CTLFLAG_RW,
        &slow_release, 0, "Passive Release was nonoptimal");
-#ifdef SMP
-static int syscall_mpsafe = 1;
-SYSCTL_INT(_kern, OID_AUTO, syscall_mpsafe, CTLFLAG_RW,
-       &syscall_mpsafe, 0, "Allow MPSAFE marked syscalls to run without BGL");
-TUNABLE_INT("kern.syscall_mpsafe", &syscall_mpsafe);
-static int trap_mpsafe = 1;
-SYSCTL_INT(_kern, OID_AUTO, trap_mpsafe, CTLFLAG_RW,
-       &trap_mpsafe, 0, "Allow traps to mostly run without the BGL");
-TUNABLE_INT("kern.trap_mpsafe", &trap_mpsafe);
-#endif
 
 MALLOC_DEFINE(M_SYSMSG, "sysmsg", "sysmsg structure");
 extern int max_sysmsg;
@@ -425,14 +415,6 @@ user_trap(struct trapframe *frame)
        }
 #endif
 
-       ++gd->gd_trap_nesting_level;
-#ifdef SMP
-       if (trap_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
-#endif
-
-       --gd->gd_trap_nesting_level;
-
 #if defined(I586_CPU) && !defined(NO_F00F_HACK)
 restart:
 #endif
@@ -698,16 +680,6 @@ kern_trap(struct trapframe *frame)
                goto out2;
        }
 #endif
-
-       ++gd->gd_trap_nesting_level;
-
-#ifdef SMP
-       if (trap_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
-#endif
-
-       --gd->gd_trap_nesting_level;
-
        type = frame->tf_trapno;
        code = frame->tf_err;
 
@@ -1108,9 +1080,8 @@ syscall2(struct trapframe *frame)
                frame->tf_eax);
 
 #ifdef SMP
-       KASSERT(td->td_mpcount == 0, ("badmpcount syscall2 from %p", (void *)frame->tf_eip));
-       if (syscall_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
+       KASSERT(td->td_mpcount == 0,
+               ("badmpcount syscall2 from %p", (void *)frame->tf_eip));
 #endif
        userenter(td, p);       /* lazy raise our priority */
 
index beccf0c..e0aa541 100644 (file)
@@ -168,16 +168,6 @@ SYSCTL_INT(_machdep, OID_AUTO, fast_release, CTLFLAG_RW,
 static int slow_release;
 SYSCTL_INT(_machdep, OID_AUTO, slow_release, CTLFLAG_RW,
        &slow_release, 0, "Passive Release was nonoptimal");
-#ifdef SMP
-static int syscall_mpsafe = 1;
-SYSCTL_INT(_kern, OID_AUTO, syscall_mpsafe, CTLFLAG_RW,
-       &syscall_mpsafe, 0, "Allow MPSAFE marked syscalls to run without BGL");
-TUNABLE_INT("kern.syscall_mpsafe", &syscall_mpsafe);
-static int trap_mpsafe = 1;
-SYSCTL_INT(_kern, OID_AUTO, trap_mpsafe, CTLFLAG_RW,
-       &trap_mpsafe, 0, "Allow traps to mostly run without the BGL");
-TUNABLE_INT("kern.trap_mpsafe", &trap_mpsafe);
-#endif
 
 MALLOC_DEFINE(M_SYSMSG, "sysmsg", "sysmsg structure");
 extern int max_sysmsg;
@@ -419,14 +409,6 @@ user_trap(struct trapframe *frame)
        }
 #endif
 
-       ++gd->gd_trap_nesting_level;
-#ifdef SMP
-       if (trap_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
-#endif
-
-       --gd->gd_trap_nesting_level;
-
 #if defined(I586_CPU) && !defined(NO_F00F_HACK)
 restart:
 #endif
@@ -675,15 +657,6 @@ kern_trap(struct trapframe *frame)
        }
 #endif
 
-       ++gd->gd_trap_nesting_level;
-
-#ifdef SMP
-       if (trap_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
-#endif
-
-       --gd->gd_trap_nesting_level;
-
        type = frame->tf_trapno;
        code = frame->tf_err;
 
@@ -1158,9 +1131,8 @@ syscall2(struct trapframe *frame)
                frame->tf_eax);
 
 #ifdef SMP
-       KASSERT(td->td_mpcount == 0, ("badmpcount syscall2 from %p", (void *)frame->tf_rip));
-       if (syscall_mpsafe == 0)
-               MAKEMPSAFE(have_mplock);
+       KASSERT(td->td_mpcount == 0,
+               ("badmpcount syscall2 from %p", (void *)frame->tf_rip));
 #endif
        userenter(td, p);       /* lazy raise our priority */