diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2010-05-03 22:54:47 -0400 |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2010-05-03 23:38:16 -0400 |
commit | 54d47a2be5e7f928fb77b2f5a0761f6bd3c9dbff (patch) | |
tree | f31a627eea303b08da532645fce94cbb4452ec30 /kernel/lockdep_internals.h | |
parent | fa9a97dec611c5356301645d576b523ce3919eba (diff) |
lockdep: No need to disable preemption in debug atomic ops
No need to disable preemption in the debug_atomic_* ops, as
we ensure interrupts are disabled already.
So let's use the __this_cpu_ops() rather than this_cpu_ops() that
enclose the ops in a preempt disabled section.
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Diffstat (limited to 'kernel/lockdep_internals.h')
-rw-r--r-- | kernel/lockdep_internals.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/lockdep_internals.h b/kernel/lockdep_internals.h index 8d929c717d3e..4f560cfedc8f 100644 --- a/kernel/lockdep_internals.h +++ b/kernel/lockdep_internals.h | |||
@@ -144,12 +144,12 @@ DECLARE_PER_CPU(struct lockdep_stats, lockdep_stats); | |||
144 | 144 | ||
145 | #define debug_atomic_inc(ptr) { \ | 145 | #define debug_atomic_inc(ptr) { \ |
146 | WARN_ON_ONCE(!irqs_disabled()); \ | 146 | WARN_ON_ONCE(!irqs_disabled()); \ |
147 | this_cpu_inc(lockdep_stats.ptr); \ | 147 | __this_cpu_inc(lockdep_stats.ptr); \ |
148 | } | 148 | } |
149 | 149 | ||
150 | #define debug_atomic_dec(ptr) { \ | 150 | #define debug_atomic_dec(ptr) { \ |
151 | WARN_ON_ONCE(!irqs_disabled()); \ | 151 | WARN_ON_ONCE(!irqs_disabled()); \ |
152 | this_cpu_dec(lockdep_stats.ptr); \ | 152 | __this_cpu_dec(lockdep_stats.ptr); \ |
153 | } | 153 | } |
154 | 154 | ||
155 | #define debug_atomic_read(ptr) ({ \ | 155 | #define debug_atomic_read(ptr) ({ \ |