From: Ingo Molnar make x86_64 stack backtraces human-readable. Before: sshd S ffff810033d05bf8 0 3119 2816 3121 (NOTLB) ffff810033d05bf8 00000000000614d7 ffff810034b230b8 ffff810034b22ee0 ffffffff8062c900 0000003835024076 0000000034b23680 ffff810034b22ee0 0000000000000000 ffffffff80502099 Call Trace: [] schedule_timeout+0x22/0xb3 [] unix_stream_recvmsg+0x274/0x561 [] do_sock_read+0x9b/0x9f [] sock_aio_read+0x57/0x67 [] do_sync_read+0xf0/0x12e [] vfs_read+0xe5/0x17e [] sys_read+0x45/0x6e [] system_call+0x7e/0x83 After: sshd S ffff81003981dbf8 0 2875 2805 2878 (NOTLB) ffff81003981dbf8 0000000000001592 ffff81003f5ce3e8 ffff81003f5ce210 ffff81003ffaa0d0 00000009afd9e025 000000013f5ce9b0 ffff81003f5ce210 0000000000000000 ffffffff80502049 Call Trace: [] schedule_timeout+0x22/0xb3 [] unix_stream_recvmsg+0x274/0x561 [] do_sock_read+0x9b/0x9f [] sock_aio_read+0x57/0x67 [] do_sync_read+0xf0/0x12e [] vfs_read+0xe5/0x17e [] sys_read+0x45/0x6e [] system_call+0x7e/0x83 Signed-off-by: Ingo Molnar Cc: Andi Kleen Signed-off-by: Andrew Morton --- arch/x86_64/kernel/process.c | 2 +- arch/x86_64/kernel/traps.c | 30 +++++++++++++----------------- arch/x86_64/mm/fault.c | 1 - include/asm-x86_64/kdebug.h | 2 +- 4 files changed, 15 insertions(+), 20 deletions(-) diff -puN arch/x86_64/kernel/process.c~lock-validator-beautify-x86_64-stacktraces-fix-2 arch/x86_64/kernel/process.c --- devel/arch/x86_64/kernel/process.c~lock-validator-beautify-x86_64-stacktraces-fix-2 2006-06-09 15:18:36.000000000 -0700 +++ devel-akpm/arch/x86_64/kernel/process.c 2006-06-09 15:18:36.000000000 -0700 @@ -295,7 +295,7 @@ void __show_regs(struct pt_regs * regs) init_utsname()->version); printk("RIP: %04lx:[<%016lx>] ", regs->cs & 0xffff, regs->rip); printk_address(regs->rip); - printk("\nRSP: %04lx:%016lx EFLAGS: %08lx\n", regs->ss, regs->rsp, + printk("RSP: %04lx:%016lx EFLAGS: %08lx\n", regs->ss, regs->rsp, regs->eflags); printk("RAX: %016lx RBX: %016lx RCX: %016lx\n", regs->rax, regs->rbx, regs->rcx); diff -puN arch/x86_64/kernel/traps.c~lock-validator-beautify-x86_64-stacktraces-fix-2 arch/x86_64/kernel/traps.c --- devel/arch/x86_64/kernel/traps.c~lock-validator-beautify-x86_64-stacktraces-fix-2 2006-06-09 15:18:36.000000000 -0700 +++ devel-akpm/arch/x86_64/kernel/traps.c 2006-06-09 15:18:36.000000000 -0700 @@ -111,7 +111,7 @@ static int call_trace = 1; #ifdef CONFIG_KALLSYMS # include -int printk_address(unsigned long address) +void printk_address(unsigned long address) { unsigned long offset = 0, symsize; const char *symname; @@ -121,17 +121,19 @@ int printk_address(unsigned long address symname = kallsyms_lookup(address, &symsize, &offset, &modname, namebuf); - if (!symname) - return printk(" [<%016lx>]", address); + if (!symname) { + printk(" [<%016lx>]\n", address); + return; + } if (!modname) modname = delim = ""; - return printk(" [<%016lx>] %s%s%s%s+0x%lx/0x%lx", + printk(" [<%016lx>] %s%s%s%s+0x%lx/0x%lx\n", address, delim, modname, delim, symname, offset, symsize); } #else -int printk_address(unsigned long address) +void printk_address(unsigned long address) { - return printk(" [<%016lx>]", address); + printk(" [<%016lx>]\n", address); } #endif @@ -194,20 +196,14 @@ static unsigned long *in_exception_stack static int show_trace_unwind(struct unwind_frame_info *info, void *context) { - int i = 11, n = 0; + int n = 0; while (unwind(info) == 0 && UNW_PC(info)) { - ++n; - if (i > 50) { - printk("\n "); - i = 7; - } else - i += printk(" "); - i += printk_address(UNW_PC(info)); + n++; + printk_address(UNW_PC(info)); if (arch_unw_user_mode(info)) break; } - printk("\n"); return n; } @@ -333,8 +329,8 @@ static void _show_stack(struct task_stru break; } if (i && ((i % 4) == 0)) - printk("\n "); - printk("%016lx ", *stack++); + printk("\n"); + printk(" %016lx", *stack++); touch_nmi_watchdog(); } show_trace(tsk, regs, rsp); diff -puN arch/x86_64/mm/fault.c~lock-validator-beautify-x86_64-stacktraces-fix-2 arch/x86_64/mm/fault.c --- devel/arch/x86_64/mm/fault.c~lock-validator-beautify-x86_64-stacktraces-fix-2 2006-06-09 15:18:36.000000000 -0700 +++ devel-akpm/arch/x86_64/mm/fault.c 2006-06-09 15:18:36.000000000 -0700 @@ -569,7 +569,6 @@ no_context: printk(KERN_ALERT "Unable to handle kernel paging request"); printk(" at %016lx RIP: \n" KERN_ALERT,address); printk_address(regs->rip); - printk("\n"); dump_pagetable(address); tsk->thread.cr2 = address; tsk->thread.trap_no = 14; diff -puN include/asm-x86_64/kdebug.h~lock-validator-beautify-x86_64-stacktraces-fix-2 include/asm-x86_64/kdebug.h --- devel/include/asm-x86_64/kdebug.h~lock-validator-beautify-x86_64-stacktraces-fix-2 2006-06-09 15:18:36.000000000 -0700 +++ devel-akpm/include/asm-x86_64/kdebug.h 2006-06-09 15:18:36.000000000 -0700 @@ -49,7 +49,7 @@ static inline int notify_die(enum die_va return atomic_notifier_call_chain(&die_chain, val, &args); } -extern int printk_address(unsigned long address); +extern void printk_address(unsigned long address); extern void die(const char *,struct pt_regs *,long); extern void __die(const char *,struct pt_regs *,long); extern void show_registers(struct pt_regs *regs); _