diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2008-08-11 16:45:51 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-08-11 16:45:51 -0400 |
commit | 0f2bc27be27ca1dcc66b96131e44bf7648b959c6 (patch) | |
tree | 22ea7b9a28878851f445a5420ff80b64ee1cfdd1 /kernel | |
parent | e5f363e358cf16e4ad13a6826e15088c5495efe9 (diff) |
lockdep: fix debug_lock_alloc
When we enable DEBUG_LOCK_ALLOC but do not enable PROVE_LOCKING and or
LOCK_STAT, lock_alloc() and lock_release() turn into nops, even though
we should be doing hlock checking (check=1).
This causes a false warning and a lockdep self-disable.
Rectify this.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/lockdep.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/kernel/lockdep.c b/kernel/lockdep.c index ab933fecd2a1..1aa91fd6b06e 100644 --- a/kernel/lockdep.c +++ b/kernel/lockdep.c | |||
@@ -2935,9 +2935,6 @@ void lock_acquire(struct lockdep_map *lock, unsigned int subclass, | |||
2935 | { | 2935 | { |
2936 | unsigned long flags; | 2936 | unsigned long flags; |
2937 | 2937 | ||
2938 | if (unlikely(!lock_stat && !prove_locking)) | ||
2939 | return; | ||
2940 | |||
2941 | if (unlikely(current->lockdep_recursion)) | 2938 | if (unlikely(current->lockdep_recursion)) |
2942 | return; | 2939 | return; |
2943 | 2940 | ||
@@ -2958,9 +2955,6 @@ void lock_release(struct lockdep_map *lock, int nested, | |||
2958 | { | 2955 | { |
2959 | unsigned long flags; | 2956 | unsigned long flags; |
2960 | 2957 | ||
2961 | if (unlikely(!lock_stat && !prove_locking)) | ||
2962 | return; | ||
2963 | |||
2964 | if (unlikely(current->lockdep_recursion)) | 2958 | if (unlikely(current->lockdep_recursion)) |
2965 | return; | 2959 | return; |
2966 | 2960 | ||