diff options
author | Frederic Weisbecker <frederic@kernel.org> | 2017-11-06 10:01:21 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2017-11-08 05:13:49 -0500 |
commit | 53bef3fd47f69e40b52c9f9acd3551dfff9f8702 (patch) | |
tree | 5eb0a268eef8ad8dadb1ffd79c25c72e54059e51 | |
parent | ebf3adbad012b89c4a51a3beae718a587d988a3a (diff) |
timers/hrtimer: Use lockdep to assert IRQs are disabled/enabled
Use lockdep to check that IRQs are enabled or disabled as expected. This
way the sanity check only shows overhead when concurrency correctness
debug code is enabled.
Signed-off-by: Frederic Weisbecker <frederic@kernel.org>
Acked-by: Thomas Gleixner <tglx@linutronix.de>
Cc: David S . Miller <davem@davemloft.net>
Cc: Lai Jiangshan <jiangshanlai@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Tejun Heo <tj@kernel.org>
Link: http://lkml.kernel.org/r/1509980490-4285-6-git-send-email-frederic@kernel.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | kernel/time/hrtimer.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/kernel/time/hrtimer.c b/kernel/time/hrtimer.c index 88f75f92ef36..d32520840fde 100644 --- a/kernel/time/hrtimer.c +++ b/kernel/time/hrtimer.c | |||
@@ -758,9 +758,7 @@ void clock_was_set(void) | |||
758 | */ | 758 | */ |
759 | void hrtimers_resume(void) | 759 | void hrtimers_resume(void) |
760 | { | 760 | { |
761 | WARN_ONCE(!irqs_disabled(), | 761 | lockdep_assert_irqs_disabled(); |
762 | KERN_INFO "hrtimers_resume() called with IRQs enabled!"); | ||
763 | |||
764 | /* Retrigger on the local CPU */ | 762 | /* Retrigger on the local CPU */ |
765 | retrigger_next_event(NULL); | 763 | retrigger_next_event(NULL); |
766 | /* And schedule a retrigger for all others */ | 764 | /* And schedule a retrigger for all others */ |