x86/reboot: Update nonmi_ipi parameter
authorDon Zickus <dzickus@redhat.com>
Fri, 11 May 2012 18:41:15 +0000 (14:41 -0400)
committerIngo Molnar <mingo@kernel.org>
Mon, 14 May 2012 09:49:38 +0000 (11:49 +0200)
Update the nonmi_ipi parameter to reflect the simple change
instead of the previous complicated one.  There should be less
of a need to use it but there may still be corner cases on older
hardware that stumble into NMI issues.

Signed-off-by: Don Zickus <dzickus@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/1336761675-24296-4-git-send-email-dzickus@redhat.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
arch/x86/kernel/smp.c

index 228e740..48d2b7d 100644 (file)
  */
 
 static atomic_t stopping_cpu = ATOMIC_INIT(-1);
+static bool smp_no_nmi_ipi = false;
 
 /*
  * this function sends a 'reschedule' IPI to another CPU.
@@ -216,7 +217,7 @@ static void native_stop_other_cpus(int wait)
        }
        
        /* if the REBOOT_VECTOR didn't work, try with the NMI */
-       if ((num_online_cpus() > 1))  {
+       if ((num_online_cpus() > 1) && (!smp_no_nmi_ipi))  {
                if (register_nmi_handler(NMI_LOCAL, smp_stop_nmi_callback,
                                         NMI_FLAG_FIRST, "smp_stop"))
                        /* Note: we ignore failures here */
@@ -245,11 +246,6 @@ finish:
        local_irq_restore(flags);
 }
 
-static void native_smp_disable_nmi_ipi(void)
-{
-       smp_ops.stop_other_cpus = native_irq_stop_other_cpus;
-}
-
 /*
  * Reschedule call back.
  */
@@ -283,8 +279,8 @@ void smp_call_function_single_interrupt(struct pt_regs *regs)
 
 static int __init nonmi_ipi_setup(char *str)
 {
-        native_smp_disable_nmi_ipi();
-        return 1;
+       smp_no_nmi_ipi = true;
+       return 1;
 }
 
 __setup("nonmi_ipi", nonmi_ipi_setup);