aboutsummaryrefslogtreecommitdiffstats
path: root/kernel/rcu/tree.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/tree.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/tree.c')
-rw-r--r--kernel/rcu/tree.c28
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 */
3162void synchronize_sched(void) 3162void 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 */
3189void synchronize_rcu_bh(void) 3189void 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())