diff options
| author |   Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-06-30 08:17:40 -0700 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-07-22 15:27:33 -0700 |
| commit | a76a9a485d730024a7cbd76efcd9c6eb46003829 (patch) | |
| tree | 356c3e8f06894f64c6917e434d6345cb2fe547c3 /kernel/rcu | |
| parent | rcu: Rename rcu_lockdep_assert() to RCU_LOCKDEP_WARN() (diff) | |
| download | linux-dev-a76a9a485d730024a7cbd76efcd9c6eb46003829.tar.xz linux-dev-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')
| -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. */ |