diff options
author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-06-30 17:17:40 +0200 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-07-23 00:27:33 +0200 |
commit | a76a9a485d730024a7cbd76efcd9c6eb46003829 (patch) | |
tree | 356c3e8f06894f64c6917e434d6345cb2fe547c3 /kernel/rcu/update.c | |
parent | rcu: Rename rcu_lockdep_assert() to RCU_LOCKDEP_WARN() (diff) | |
download | linux-a76a9a485d730024a7cbd76efcd9c6eb46003829.tar.xz linux-a76a9a485d730024a7cbd76efcd9c6eb46003829.zip |
rcu: Fix backwards RCU_LOCKDEP_WARN() in synchronize_rcu_tasks()
The RCU_LOCKDEP_WARN() in synchronize_rcu_tasks() triggers if the
scheduler is active, which is backwards. This commit therefore
negates the test.
Reported-by: Ingo Molnar <mingo@kernel.org>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Diffstat (limited to 'kernel/rcu/update.c')
-rw-r--r-- | kernel/rcu/update.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/rcu/update.c b/kernel/rcu/update.c index 47268fb1d27b..7a0b3bc7c5ed 100644 --- a/kernel/rcu/update.c +++ b/kernel/rcu/update.c @@ -589,7 +589,7 @@ EXPORT_SYMBOL_GPL(call_rcu_tasks); void synchronize_rcu_tasks(void) { /* Complain if the scheduler has not started. */ - RCU_LOCKDEP_WARN(rcu_scheduler_active, + RCU_LOCKDEP_WARN(!rcu_scheduler_active, "synchronize_rcu_tasks called too soon"); /* Wait for the grace period. */ |