Commit 1540912c authored by Anton Blanchard's avatar Anton Blanchard Committed by Linus Torvalds

[PATCH] : ppc64: enhance oops printing

Here are some changes to the oops printout, stuff that would have been
useful when I was chasing various bugs.

- print out instructions around the fail (3/4 before 1/4 after).
- print out CTR and CR registers, make some space by cutting down XER
  (its only 32bit)
- always print the DAR and DSISR, its sometimes useful
- print_modules() like x86
Signed-off-by: default avatarAnton Blanchard <anton@samba.org>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent e2c4ae10
...@@ -214,23 +214,57 @@ struct task_struct *__switch_to(struct task_struct *prev, ...@@ -214,23 +214,57 @@ struct task_struct *__switch_to(struct task_struct *prev,
return last; return last;
} }
static int instructions_to_print = 16;
static void show_instructions(struct pt_regs *regs)
{
int i;
unsigned long pc = regs->nip - (instructions_to_print * 3 / 4 *
sizeof(int));
printk("Instruction dump:");
for (i = 0; i < instructions_to_print; i++) {
int instr;
if (!(i % 8))
printk("\n");
if (((REGION_ID(pc) != KERNEL_REGION_ID) &&
(REGION_ID(pc) != VMALLOC_REGION_ID)) ||
__get_user(instr, (unsigned int *)pc)) {
printk("XXXXXXXX ");
} else {
if (regs->nip == pc)
printk("<%08x> ", instr);
else
printk("%08x ", instr);
}
pc += sizeof(int);
}
printk("\n");
}
void show_regs(struct pt_regs * regs) void show_regs(struct pt_regs * regs)
{ {
int i; int i;
unsigned long trap; unsigned long trap;
printk("NIP: %016lX XER: %016lX LR: %016lX\n", printk("NIP: %016lX XER: %08X LR: %016lX CTR: %016lX\n",
regs->nip, regs->xer, regs->link); regs->nip, (unsigned int)regs->xer, regs->link, regs->ctr);
printk("REGS: %p TRAP: %04lx %s (%s)\n", printk("REGS: %p TRAP: %04lx %s (%s)\n",
regs, regs->trap, print_tainted(), UTS_RELEASE); regs, regs->trap, print_tainted(), UTS_RELEASE);
printk("MSR: %016lx EE: %01x PR: %01x FP: %01x ME: %01x IR/DR: %01x%01x\n", printk("MSR: %016lx EE: %01x PR: %01x FP: %01x ME: %01x "
"IR/DR: %01x%01x CR: %08X\n",
regs->msr, regs->msr&MSR_EE ? 1 : 0, regs->msr&MSR_PR ? 1 : 0, regs->msr, regs->msr&MSR_EE ? 1 : 0, regs->msr&MSR_PR ? 1 : 0,
regs->msr & MSR_FP ? 1 : 0,regs->msr&MSR_ME ? 1 : 0, regs->msr & MSR_FP ? 1 : 0,regs->msr&MSR_ME ? 1 : 0,
regs->msr&MSR_IR ? 1 : 0, regs->msr&MSR_IR ? 1 : 0,
regs->msr&MSR_DR ? 1 : 0); regs->msr&MSR_DR ? 1 : 0,
(unsigned int)regs->ccr);
trap = TRAP(regs); trap = TRAP(regs);
if (trap == 0x300 || trap == 0x380 || trap == 0x600) printk("DAR: %016lx DSISR: %016lx\n", regs->dar, regs->dsisr);
printk("DAR: %016lx, DSISR: %016lx\n", regs->dar, regs->dsisr);
printk("TASK: %p[%d] '%s' THREAD: %p", printk("TASK: %p[%d] '%s' THREAD: %p",
current, current->pid, current->comm, current->thread_info); current, current->pid, current->comm, current->thread_info);
...@@ -257,6 +291,8 @@ void show_regs(struct pt_regs * regs) ...@@ -257,6 +291,8 @@ void show_regs(struct pt_regs * regs)
printk("LR [%016lx] ", regs->link); printk("LR [%016lx] ", regs->link);
print_symbol("%s\n", regs->link); print_symbol("%s\n", regs->link);
show_stack(current, (unsigned long *)regs->gpr[1]); show_stack(current, (unsigned long *)regs->gpr[1]);
if (!user_mode(regs))
show_instructions(regs);
} }
void exit_thread(void) void exit_thread(void)
......
...@@ -127,6 +127,7 @@ int die(const char *str, struct pt_regs *regs, long err) ...@@ -127,6 +127,7 @@ int die(const char *str, struct pt_regs *regs, long err)
} }
if (nl) if (nl)
printk("\n"); printk("\n");
print_modules();
show_regs(regs); show_regs(regs);
bust_spinlocks(0); bust_spinlocks(0);
spin_unlock_irq(&die_lock); spin_unlock_irq(&die_lock);
......
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