diff options
author | Nicholas Piggin <npiggin@gmail.com> | 2018-02-02 15:34:10 +1000 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2018-02-08 18:21:42 -0600 |
commit | d5378d2a0d54793ee59cd460e465fd0471bdd703 (patch) | |
tree | 800467f2c73b98f79c3e3ec25f10a2d61b78dfb1 /core | |
parent | 92777d335ab3fe8e364014584b454ac48883993b (diff) | |
download | skiboot-d5378d2a0d54793ee59cd460e465fd0471bdd703.zip skiboot-d5378d2a0d54793ee59cd460e465fd0471bdd703.tar.gz skiboot-d5378d2a0d54793ee59cd460e465fd0471bdd703.tar.bz2 |
core/exception: beautify exception handler, add MCE-involved registers
Print DSISR and DAR, to help with deciphering machine check exceptions,
and improve the output a bit, decode NIP symbol, improve alignment, etc.
Also print a specific header for machine check, because we do expect to
see these if there is a hardware failure.
Before:
[ 0.005968779,3] ***********************************************
[ 0.005974102,3] Unexpected exception 200 !
[ 0.005978696,3] SRR0 : 000000003002ad80 SRR1 : 9000000000001000
[ 0.005985239,3] HSRR0: 00000000300027b4 HSRR1: 9000000030001000
[ 0.005991782,3] LR : 000000003002ad80 CTR : 0000000000000000
[ 0.005998130,3] CFAR : 00000000300b58bc
[ 0.006002769,3] CR : 40000004 XER: 20000000
[ 0.006008069,3] GPR00: 000000003002ad80 GPR16: 0000000000000000
[ 0.006015170,3] GPR01: 0000000031c03bd0 GPR17: 0000000000000000
[...]
After:
[ 0.003287941,3] ***********************************************
[ 0.003561769,3] Fatal MCE at 000000003002ad80 .nvram_init+0x24
[ 0.003579628,3] CFAR : 00000000300b5964
[ 0.003584268,3] SRR0 : 000000003002ad80 SRR1 : 9000000000001000
[ 0.003590812,3] HSRR0: 00000000300027b4 HSRR1: 9000000030001000
[ 0.003597355,3] DSISR: 00000000 DAR : 0000000000000000
[ 0.003603480,3] LR : 000000003002ad68 CTR : 0000000030093d80
[ 0.003609930,3] CR : 40000004 XER : 20000000
[ 0.003615698,3] GPR00: 00000000300149e8 GPR16: 0000000000000000
[ 0.003622799,3] GPR01: 0000000031c03bc0 GPR17: 0000000000000000
[...]
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'core')
-rw-r--r-- | core/exceptions.c | 39 |
1 files changed, 36 insertions, 3 deletions
diff --git a/core/exceptions.c b/core/exceptions.c index c4acf85..4880fa2 100644 --- a/core/exceptions.c +++ b/core/exceptions.c @@ -21,17 +21,19 @@ #include <cpu.h> #define REG "%016llx" +#define REG32 "%08x" #define REGS_PER_LINE 4 static void dump_regs(struct stack_frame *stack) { unsigned int i; + prerror("CFAR : "REG"\n", stack->cfar); prerror("SRR0 : "REG" SRR1 : "REG"\n", stack->srr0, stack->srr1); prerror("HSRR0: "REG" HSRR1: "REG"\n", stack->hsrr0, stack->hsrr1); + prerror("DSISR: "REG32" DAR : "REG"\n", stack->dsisr, stack->dar); prerror("LR : "REG" CTR : "REG"\n", stack->lr, stack->ctr); - prerror("CFAR : "REG"\n", stack->cfar); - prerror("CR : %08x XER: %08x\n", stack->cr, stack->xer); + prerror("CR : "REG32" XER : "REG32"\n", stack->cr, stack->xer); for (i = 0; i < 16; i++) prerror("GPR%02d: "REG" GPR%02d: "REG"\n", i, stack->gpr[i], i + 16, stack->gpr[i + 16]); @@ -42,8 +44,39 @@ void exception_entry(struct stack_frame *stack) __noreturn; void exception_entry(struct stack_frame *stack) { + const size_t max = 320; + char buf[max]; + size_t l; + prerror("***********************************************\n"); - prerror("Unexpected exception %llx !\n", stack->type); + if (stack->type == 0x200) { + l = 0; + l += snprintf(buf + l, max - l, "Fatal MCE at "REG" ", stack->srr0); + l += snprintf_symbol(buf + l, max - l, stack->srr0); + prerror("%s\n", buf); + } else { + uint64_t nip; + switch (stack->type) { + case 0x500: + case 0x980: + case 0xe00: + case 0xe20: + case 0xe40: + case 0xe60: + case 0xe80: + case 0xea0: + case 0xf80: + nip = stack->hsrr0; + break; + default: + nip = stack->srr0; + break; + } + l = 0; + l += snprintf(buf + l, max - l, "Fatal Exception 0x%llx at "REG" ", stack->type, nip); + l += snprintf_symbol(buf + l, max - l, nip); + prerror("%s\n", buf); + } dump_regs(stack); abort(); } |