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/tree.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/tree.c')
| -rw-r--r-- | kernel/rcu/tree.c | 28 | 
1 files changed, 14 insertions, 14 deletions
| diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c index cb64d7e13d24..0a73d26357a2 100644 --- a/kernel/rcu/tree.c +++ b/kernel/rcu/tree.c | |||
| @@ -649,12 +649,12 @@ static void rcu_eqs_enter_common(long long oldval, bool user) | |||
| 649 | * It is illegal to enter an extended quiescent state while | 649 | * It is illegal to enter an extended quiescent state while | 
| 650 | * in an RCU read-side critical section. | 650 | * in an RCU read-side critical section. | 
| 651 | */ | 651 | */ | 
| 652 | rcu_lockdep_assert(!lock_is_held(&rcu_lock_map), | 652 | RCU_LOCKDEP_WARN(lock_is_held(&rcu_lock_map), | 
| 653 | "Illegal idle entry in RCU read-side critical section."); | 653 | "Illegal idle entry in RCU read-side critical section."); | 
| 654 | rcu_lockdep_assert(!lock_is_held(&rcu_bh_lock_map), | 654 | RCU_LOCKDEP_WARN(lock_is_held(&rcu_bh_lock_map), | 
| 655 | "Illegal idle entry in RCU-bh read-side critical section."); | 655 | "Illegal idle entry in RCU-bh read-side critical section."); | 
| 656 | rcu_lockdep_assert(!lock_is_held(&rcu_sched_lock_map), | 656 | RCU_LOCKDEP_WARN(lock_is_held(&rcu_sched_lock_map), | 
| 657 | "Illegal idle entry in RCU-sched read-side critical section."); | 657 | "Illegal idle entry in RCU-sched read-side critical section."); | 
| 658 | } | 658 | } | 
| 659 | 659 | ||
| 660 | /* | 660 | /* | 
| @@ -3161,10 +3161,10 @@ static inline int rcu_blocking_is_gp(void) | |||
| 3161 | */ | 3161 | */ | 
| 3162 | void synchronize_sched(void) | 3162 | void synchronize_sched(void) | 
| 3163 | { | 3163 | { | 
| 3164 | rcu_lockdep_assert(!lock_is_held(&rcu_bh_lock_map) && | 3164 | RCU_LOCKDEP_WARN(lock_is_held(&rcu_bh_lock_map) || | 
| 3165 | !lock_is_held(&rcu_lock_map) && | 3165 | lock_is_held(&rcu_lock_map) || | 
| 3166 | !lock_is_held(&rcu_sched_lock_map), | 3166 | lock_is_held(&rcu_sched_lock_map), | 
| 3167 | "Illegal synchronize_sched() in RCU-sched read-side critical section"); | 3167 | "Illegal synchronize_sched() in RCU-sched read-side critical section"); | 
| 3168 | if (rcu_blocking_is_gp()) | 3168 | if (rcu_blocking_is_gp()) | 
| 3169 | return; | 3169 | return; | 
| 3170 | if (rcu_gp_is_expedited()) | 3170 | if (rcu_gp_is_expedited()) | 
| @@ -3188,10 +3188,10 @@ EXPORT_SYMBOL_GPL(synchronize_sched); | |||
| 3188 | */ | 3188 | */ | 
| 3189 | void synchronize_rcu_bh(void) | 3189 | void synchronize_rcu_bh(void) | 
| 3190 | { | 3190 | { | 
| 3191 | rcu_lockdep_assert(!lock_is_held(&rcu_bh_lock_map) && | 3191 | RCU_LOCKDEP_WARN(lock_is_held(&rcu_bh_lock_map) || | 
| 3192 | !lock_is_held(&rcu_lock_map) && | 3192 | lock_is_held(&rcu_lock_map) || | 
| 3193 | !lock_is_held(&rcu_sched_lock_map), | 3193 | lock_is_held(&rcu_sched_lock_map), | 
| 3194 | "Illegal synchronize_rcu_bh() in RCU-bh read-side critical section"); | 3194 | "Illegal synchronize_rcu_bh() in RCU-bh read-side critical section"); | 
| 3195 | if (rcu_blocking_is_gp()) | 3195 | if (rcu_blocking_is_gp()) | 
| 3196 | return; | 3196 | return; | 
| 3197 | if (rcu_gp_is_expedited()) | 3197 | if (rcu_gp_is_expedited()) | 
