Commit 2c522836 authored by Dave Jones's avatar Dave Jones Committed by Ingo Molnar

lockdep: Consolidate bug messages into a single print_lockdep_off() function

Also add some missing printk levels.
Signed-off-by: default avatarDave Jones <davej@redhat.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/20130425174002.GA26769@redhat.com
[ Tweaked the messages a bit. ]
Signed-off-by: default avatarIngo Molnar <mingo@kernel.org>
parent 199e371f
...@@ -380,6 +380,13 @@ static int verbose(struct lock_class *class) ...@@ -380,6 +380,13 @@ static int verbose(struct lock_class *class)
unsigned long nr_stack_trace_entries; unsigned long nr_stack_trace_entries;
static unsigned long stack_trace[MAX_STACK_TRACE_ENTRIES]; static unsigned long stack_trace[MAX_STACK_TRACE_ENTRIES];
static void print_lockdep_off(const char *bug_msg)
{
printk(KERN_DEBUG "%s\n", bug_msg);
printk(KERN_DEBUG "turning off the locking correctness validator.\n");
printk(KERN_DEBUG "Please attach the output of /proc/lock_stat to the bug report\n");
}
static int save_trace(struct stack_trace *trace) static int save_trace(struct stack_trace *trace)
{ {
trace->nr_entries = 0; trace->nr_entries = 0;
...@@ -409,9 +416,7 @@ static int save_trace(struct stack_trace *trace) ...@@ -409,9 +416,7 @@ static int save_trace(struct stack_trace *trace)
if (!debug_locks_off_graph_unlock()) if (!debug_locks_off_graph_unlock())
return 0; return 0;
printk("BUG: MAX_STACK_TRACE_ENTRIES too low!\n"); print_lockdep_off("BUG: MAX_STACK_TRACE_ENTRIES too low!");
printk("turning off the locking correctness validator.\n");
printk("Attach output of /proc/lock_stat to bug report\n");
dump_stack(); dump_stack();
return 0; return 0;
...@@ -764,9 +769,7 @@ register_lock_class(struct lockdep_map *lock, unsigned int subclass, int force) ...@@ -764,9 +769,7 @@ register_lock_class(struct lockdep_map *lock, unsigned int subclass, int force)
} }
raw_local_irq_restore(flags); raw_local_irq_restore(flags);
printk("BUG: MAX_LOCKDEP_KEYS too low!\n"); print_lockdep_off("BUG: MAX_LOCKDEP_KEYS too low!");
printk("turning off the locking correctness validator.\n");
printk("Attach output of /proc/lock_stat to bug report\n");
dump_stack(); dump_stack();
return NULL; return NULL;
} }
...@@ -836,9 +839,7 @@ static struct lock_list *alloc_list_entry(void) ...@@ -836,9 +839,7 @@ static struct lock_list *alloc_list_entry(void)
if (!debug_locks_off_graph_unlock()) if (!debug_locks_off_graph_unlock())
return NULL; return NULL;
printk("BUG: MAX_LOCKDEP_ENTRIES too low!\n"); print_lockdep_off("BUG: MAX_LOCKDEP_ENTRIES too low!");
printk("turning off the locking correctness validator.\n");
printk("Attach output of /proc/lock_stat to bug report\n");
dump_stack(); dump_stack();
return NULL; return NULL;
} }
...@@ -2051,9 +2052,7 @@ static inline int lookup_chain_cache(struct task_struct *curr, ...@@ -2051,9 +2052,7 @@ static inline int lookup_chain_cache(struct task_struct *curr,
if (!debug_locks_off_graph_unlock()) if (!debug_locks_off_graph_unlock())
return 0; return 0;
printk("BUG: MAX_LOCKDEP_CHAINS too low!\n"); print_lockdep_off("BUG: MAX_LOCKDEP_CHAINS too low!");
printk("turning off the locking correctness validator.\n");
printk("Attach output of /proc/lock_stat to bug report\n");
dump_stack(); dump_stack();
return 0; return 0;
} }
...@@ -3192,10 +3191,9 @@ static int __lock_acquire(struct lockdep_map *lock, unsigned int subclass, ...@@ -3192,10 +3191,9 @@ static int __lock_acquire(struct lockdep_map *lock, unsigned int subclass,
#endif #endif
if (unlikely(curr->lockdep_depth >= MAX_LOCK_DEPTH)) { if (unlikely(curr->lockdep_depth >= MAX_LOCK_DEPTH)) {
debug_locks_off(); debug_locks_off();
printk("BUG: MAX_LOCK_DEPTH too low, depth: %i max: %lu!\n", print_lockdep_off("BUG: MAX_LOCK_DEPTH too low!");
printk(KERN_DEBUG "depth: %i max: %lu!\n",
curr->lockdep_depth, MAX_LOCK_DEPTH); curr->lockdep_depth, MAX_LOCK_DEPTH);
printk("turning off the locking correctness validator.\n");
printk("Attach output of /proc/lock_stat to bug report\n");
lockdep_print_held_locks(current); lockdep_print_held_locks(current);
debug_show_all_locks(); debug_show_all_locks();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment