diff options
author | Ingo Molnar <mingo@elte.hu> | 2010-05-03 03:17:46 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-05-03 03:17:46 -0400 |
commit | 0e417fe1f247bb3ac056ee04604332005c334fac (patch) | |
tree | b1786f5789aa015f488a834a675c20120791bce3 | |
parent | 53ba4f2fa73225113a488584df0d85d3cba52943 (diff) | |
parent | 913769f24eadcd38a936ffae41d9b4895ec02e43 (diff) |
Merge branch 'core/locking' of git://git.kernel.org/pub/scm/linux/kernel/git/frederic/random-tracing into core/locking
-rw-r--r-- | kernel/lockdep.c | 7 | ||||
-rw-r--r-- | kernel/lockdep_internals.h | 10 |
2 files changed, 8 insertions, 9 deletions
diff --git a/kernel/lockdep.c b/kernel/lockdep.c index 78325f8f1139..1b58a1bbcc87 100644 --- a/kernel/lockdep.c +++ b/kernel/lockdep.c | |||
@@ -2298,7 +2298,12 @@ void trace_hardirqs_on_caller(unsigned long ip) | |||
2298 | return; | 2298 | return; |
2299 | 2299 | ||
2300 | if (unlikely(curr->hardirqs_enabled)) { | 2300 | if (unlikely(curr->hardirqs_enabled)) { |
2301 | debug_atomic_inc(redundant_hardirqs_on); | 2301 | /* |
2302 | * Neither irq nor preemption are disabled here | ||
2303 | * so this is racy by nature but loosing one hit | ||
2304 | * in a stat is not a big deal. | ||
2305 | */ | ||
2306 | this_cpu_inc(lockdep_stats.redundant_hardirqs_on); | ||
2302 | return; | 2307 | return; |
2303 | } | 2308 | } |
2304 | /* we'll do an OFF -> ON transition: */ | 2309 | /* we'll do an OFF -> ON transition: */ |
diff --git a/kernel/lockdep_internals.h b/kernel/lockdep_internals.h index 8d7d4b6c741a..2b174762fa0e 100644 --- a/kernel/lockdep_internals.h +++ b/kernel/lockdep_internals.h | |||
@@ -140,19 +140,13 @@ struct lockdep_stats { | |||
140 | DECLARE_PER_CPU(struct lockdep_stats, lockdep_stats); | 140 | DECLARE_PER_CPU(struct lockdep_stats, lockdep_stats); |
141 | 141 | ||
142 | #define debug_atomic_inc(ptr) { \ | 142 | #define debug_atomic_inc(ptr) { \ |
143 | struct lockdep_stats *__cpu_lockdep_stats; \ | ||
144 | \ | ||
145 | WARN_ON_ONCE(!irqs_disabled()); \ | 143 | WARN_ON_ONCE(!irqs_disabled()); \ |
146 | __cpu_lockdep_stats = &__get_cpu_var(lockdep_stats); \ | 144 | this_cpu_inc(lockdep_stats.ptr); \ |
147 | __cpu_lockdep_stats->ptr++; \ | ||
148 | } | 145 | } |
149 | 146 | ||
150 | #define debug_atomic_dec(ptr) { \ | 147 | #define debug_atomic_dec(ptr) { \ |
151 | struct lockdep_stats *__cpu_lockdep_stats; \ | ||
152 | \ | ||
153 | WARN_ON_ONCE(!irqs_disabled()); \ | 148 | WARN_ON_ONCE(!irqs_disabled()); \ |
154 | __cpu_lockdep_stats = &__get_cpu_var(lockdep_stats); \ | 149 | this_cpu_inc(lockdep_stats.ptr); \ |
155 | __cpu_lockdep_stats->ptr--; \ | ||
156 | } | 150 | } |
157 | 151 | ||
158 | #define debug_atomic_read(ptr) ({ \ | 152 | #define debug_atomic_read(ptr) ({ \ |