Commit 81539169 authored by Josh Poimboeuf's avatar Josh Poimboeuf Committed by Ingo Molnar

x86/dumpstack: Remove NULL task pointer convention

show_stack_log_lvl() and friends allow a NULL pointer for the
task_struct to indicate the current task.  This creates confusion and
can cause sneaky bugs.

Instead require the caller to pass 'current' directly.

This only changes the internal workings of the dumpstack code.  The
dump_trace() and show_stack() interfaces still allow a NULL task
pointer.  Those interfaces should also probably be fixed as well.
Signed-off-by: default avatarJosh Poimboeuf <jpoimboe@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: default avatarIngo Molnar <mingo@kernel.org>
parent ac496bf4
...@@ -94,7 +94,7 @@ get_frame_pointer(struct task_struct *task, struct pt_regs *regs) ...@@ -94,7 +94,7 @@ get_frame_pointer(struct task_struct *task, struct pt_regs *regs)
if (regs) if (regs)
return (unsigned long *)regs->bp; return (unsigned long *)regs->bp;
if (!task || task == current) if (task == current)
return __builtin_frame_address(0); return __builtin_frame_address(0);
return (unsigned long *)((struct inactive_task_frame *)task->thread.sp)->bp; return (unsigned long *)((struct inactive_task_frame *)task->thread.sp)->bp;
...@@ -113,7 +113,7 @@ get_stack_pointer(struct task_struct *task, struct pt_regs *regs) ...@@ -113,7 +113,7 @@ get_stack_pointer(struct task_struct *task, struct pt_regs *regs)
if (regs) if (regs)
return (unsigned long *)kernel_stack_pointer(regs); return (unsigned long *)kernel_stack_pointer(regs);
if (!task || task == current) if (task == current)
return __builtin_frame_address(0); return __builtin_frame_address(0);
return (unsigned long *)task->thread.sp; return (unsigned long *)task->thread.sp;
......
...@@ -175,11 +175,13 @@ void show_stack(struct task_struct *task, unsigned long *sp) ...@@ -175,11 +175,13 @@ void show_stack(struct task_struct *task, unsigned long *sp)
{ {
unsigned long bp = 0; unsigned long bp = 0;
task = task ? : current;
/* /*
* Stack frames below this one aren't interesting. Don't show them * Stack frames below this one aren't interesting. Don't show them
* if we're printing for %current. * if we're printing for %current.
*/ */
if (!sp && (!task || task == current)) { if (!sp && task == current) {
sp = get_stack_pointer(current, NULL); sp = get_stack_pointer(current, NULL);
bp = (unsigned long)get_frame_pointer(current, NULL); bp = (unsigned long)get_frame_pointer(current, NULL);
} }
......
...@@ -205,7 +205,7 @@ void show_regs(struct pt_regs *regs) ...@@ -205,7 +205,7 @@ void show_regs(struct pt_regs *regs)
u8 *ip; u8 *ip;
pr_emerg("Stack:\n"); pr_emerg("Stack:\n");
show_stack_log_lvl(NULL, regs, NULL, 0, KERN_EMERG); show_stack_log_lvl(current, regs, NULL, 0, KERN_EMERG);
pr_emerg("Code:"); pr_emerg("Code:");
......
...@@ -278,7 +278,7 @@ void show_regs(struct pt_regs *regs) ...@@ -278,7 +278,7 @@ void show_regs(struct pt_regs *regs)
u8 *ip; u8 *ip;
printk(KERN_DEFAULT "Stack:\n"); printk(KERN_DEFAULT "Stack:\n");
show_stack_log_lvl(NULL, regs, NULL, 0, KERN_DEFAULT); show_stack_log_lvl(current, regs, NULL, 0, KERN_DEFAULT);
printk(KERN_DEFAULT "Code: "); printk(KERN_DEFAULT "Code: ");
......
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