diff options
author | Frederic Weisbecker <frederic@kernel.org> | 2017-11-06 10:01:19 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2017-11-08 05:13:48 -0500 |
commit | 8e8eb730759f9cfd7a761b0b4ee41d714e720993 (patch) | |
tree | 0bcdc8927baf26c2288075275a25c5d66d587185 | |
parent | f71b74bca637fca7de78f1d44eaefde5bc900f9f (diff) |
workqueue: 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>
Acked-by: Tejun Heo <tj@kernel.org>
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>
Link: http://lkml.kernel.org/r/1509980490-4285-4-git-send-email-frederic@kernel.org
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | kernel/workqueue.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/workqueue.c b/kernel/workqueue.c index 1070b21ba4aa..13f67b5a0a0c 100644 --- a/kernel/workqueue.c +++ b/kernel/workqueue.c | |||
@@ -1376,7 +1376,7 @@ static void __queue_work(int cpu, struct workqueue_struct *wq, | |||
1376 | * queued or lose PENDING. Grabbing PENDING and queueing should | 1376 | * queued or lose PENDING. Grabbing PENDING and queueing should |
1377 | * happen with IRQ disabled. | 1377 | * happen with IRQ disabled. |
1378 | */ | 1378 | */ |
1379 | WARN_ON_ONCE(!irqs_disabled()); | 1379 | lockdep_assert_irqs_disabled(); |
1380 | 1380 | ||
1381 | debug_work_activate(work); | 1381 | debug_work_activate(work); |
1382 | 1382 | ||