diff options
author | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-06-18 18:50:02 -0400 |
---|---|---|
committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2015-07-22 18:27:32 -0400 |
commit | f78f5b90c4ffa559e400c3919a02236101f29f3f (patch) | |
tree | ded8a53b5c2bc06ab532d4b9a2e6e99a12912574 /kernel/rcu/srcu.c | |
parent | 46f00d18fca42cc954c2e9e99a48b6f3a7741ed7 (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.c | 10 |
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); |