rcu: Fix idle-task checks
authorPaul E. McKenney <paul.mckenney@linaro.org>
Wed, 2 Nov 2011 14:38:25 +0000 (07:38 -0700)
committerPaul E. McKenney <paulmck@linux.vnet.ibm.com>
Sun, 11 Dec 2011 18:31:44 +0000 (10:31 -0800)
RCU has traditionally relied on idle_cpu() to determine whether a given
CPU is running in the context of an idle task, but commit 908a3283
(Fix idle_cpu()) has invalidated this approach.  After commit 908a3283,
idle_cpu() will return true if the current CPU is currently running the
idle task, and will be doing so for the foreseeable future.  RCU instead
needs to know whether or not the current CPU is currently running the
idle task, regardless of what the near future might bring.

This commit therefore switches from idle_cpu() to "current->pid != 0".

Reported-by: Wu Fengguang <fengguang.wu@intel.com>
Suggested-by: Carsten Emde <C.Emde@osadl.org>
Signed-off-by: Paul E. McKenney <paul.mckenney@linaro.org>
Acked-by: Steven Rostedt <rostedt@goodmis.org>
Tested-by: Wu Fengguang <fengguang.wu@intel.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
kernel/rcutiny.c
kernel/rcutree.c

index 6d70ff71a875cbbfa777186a44c11defcd1f1442..4e16ce36fa03fa73788ce252f913388eb0420710 100644 (file)
@@ -64,7 +64,7 @@ static void rcu_idle_enter_common(long long oldval)
                return;
        }
        RCU_TRACE(trace_rcu_dyntick("Start", oldval, rcu_dynticks_nesting));
-       if (!idle_cpu(smp_processor_id())) {
+       if (current->pid != 0) {
                struct task_struct *idle = idle_task(smp_processor_id());
 
                RCU_TRACE(trace_rcu_dyntick("Error on entry: not idle task",
@@ -118,7 +118,7 @@ static void rcu_idle_exit_common(long long oldval)
                return;
        }
        RCU_TRACE(trace_rcu_dyntick("End", oldval, rcu_dynticks_nesting));
-       if (!idle_cpu(smp_processor_id())) {
+       if (current->pid != 0) {
                struct task_struct *idle = idle_task(smp_processor_id());
 
                RCU_TRACE(trace_rcu_dyntick("Error on exit: not idle task",
index b1711c48a7ec05d577fab1ba2ec61d97e1101390..49e0783fb200faed3358b798dc5e2b7c728cbcf6 100644 (file)
@@ -355,7 +355,7 @@ static void rcu_idle_enter_common(struct rcu_dynticks *rdtp, long long oldval)
                return;
        }
        trace_rcu_dyntick("Start", oldval, rdtp->dynticks_nesting);
-       if (!idle_cpu(smp_processor_id())) {
+       if (current->pid != 0) {
                struct task_struct *idle = idle_task(smp_processor_id());
 
                trace_rcu_dyntick("Error on entry: not idle task",
@@ -449,7 +449,7 @@ static void rcu_idle_exit_common(struct rcu_dynticks *rdtp, long long oldval)
        smp_mb__after_atomic_inc();  /* See above. */
        WARN_ON_ONCE(!(atomic_read(&rdtp->dynticks) & 0x1));
        trace_rcu_dyntick("End", oldval, rdtp->dynticks_nesting);
-       if (!idle_cpu(smp_processor_id())) {
+       if (current->pid != 0) {
                struct task_struct *idle = idle_task(smp_processor_id());
 
                trace_rcu_dyntick("Error on exit: not idle task",