aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/rcu/srcu.c
diff options
context:
space:
mode:
authorPaul E. McKenney <paulmck@linux.vnet.ibm.com>2015-06-18 18:50:02 -0400
committerPaul E. McKenney <paulmck@linux.vnet.ibm.com>2015-07-22 18:27:32 -0400
commitf78f5b90c4ffa559e400c3919a02236101f29f3f (patch)
treeded8a53b5c2bc06ab532d4b9a2e6e99a12912574 /kernel/rcu/srcu.c
parent46f00d18fca42cc954c2e9e99a48b6f3a7741ed7 (diff)
rcu: Rename rcu_lockdep_assert() to RCU_LOCKDEP_WARN()
This commit renames rcu_lockdep_assert() to RCU_LOCKDEP_WARN() for consistency with the WARN() series of macros. This also requires inverting the sense of the conditional, which this commit also does. Reported-by: Ingo Molnar <mingo@kernel.org> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> Reviewed-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'kernel/rcu/srcu.c')
-rw-r--r--kernel/rcu/srcu.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/kernel/rcu/srcu.c b/kernel/rcu/srcu.c
index de35087c92a5..d3fcb2ec8536 100644
--- a/kernel/rcu/srcu.c
+++ b/kernel/rcu/srcu.c
@@ -415,11 +415,11 @@ static void __synchronize_srcu(struct srcu_struct *sp, int trycount)
415 struct rcu_head *head = &rcu.head; 415 struct rcu_head *head = &rcu.head;
416 bool done = false; 416 bool done = false;
417 417
418 rcu_lockdep_assert(!lock_is_held(&sp->dep_map) && 418 RCU_LOCKDEP_WARN(lock_is_held(&sp->dep_map) ||
419 !lock_is_held(&rcu_bh_lock_map) && 419 lock_is_held(&rcu_bh_lock_map) ||
420 !lock_is_held(&rcu_lock_map) && 420 lock_is_held(&rcu_lock_map) ||
421 !lock_is_held(&rcu_sched_lock_map), 421 lock_is_held(&rcu_sched_lock_map),
422 "Illegal synchronize_srcu() in same-type SRCU (or RCU) read-side critical section"); 422 "Illegal synchronize_srcu() in same-type SRCU (or in RCU) read-side critical section");
423 423
424 might_sleep(); 424 might_sleep();
425 init_completion(&rcu.completion); 425 init_completion(&rcu.completion);