diff options
author | Ingo Molnar <mingo@elte.hu> | 2007-12-07 13:02:47 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2007-12-07 13:02:47 -0500 |
commit | 5f9fa8a62d6a98f5cb2ee2e00b85bfe95e45888d (patch) | |
tree | cb1a27380131465adc0ba372ec9f4869beb520e7 | |
parent | f194d132e4971111f85c18c96067acffb13cee6d (diff) |
lockdep: make cli/sti annotation warnings clearer
make cli/sti annotation warnings easier to interpret.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
-rw-r--r-- | kernel/lockdep.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/kernel/lockdep.c b/kernel/lockdep.c index 0f389621bb6b..723bd9f92556 100644 --- a/kernel/lockdep.c +++ b/kernel/lockdep.c | |||
@@ -2654,10 +2654,15 @@ static void check_flags(unsigned long flags) | |||
2654 | if (!debug_locks) | 2654 | if (!debug_locks) |
2655 | return; | 2655 | return; |
2656 | 2656 | ||
2657 | if (irqs_disabled_flags(flags)) | 2657 | if (irqs_disabled_flags(flags)) { |
2658 | DEBUG_LOCKS_WARN_ON(current->hardirqs_enabled); | 2658 | if (DEBUG_LOCKS_WARN_ON(current->hardirqs_enabled)) { |
2659 | else | 2659 | printk("possible reason: unannotated irqs-off.\n"); |
2660 | DEBUG_LOCKS_WARN_ON(!current->hardirqs_enabled); | 2660 | } |
2661 | } else { | ||
2662 | if (DEBUG_LOCKS_WARN_ON(!current->hardirqs_enabled)) { | ||
2663 | printk("possible reason: unannotated irqs-on.\n"); | ||
2664 | } | ||
2665 | } | ||
2661 | 2666 | ||
2662 | /* | 2667 | /* |
2663 | * We dont accurately track softirq state in e.g. | 2668 | * We dont accurately track softirq state in e.g. |