Benjamin Herrenschmidt wrote:
On Wed, 2018-04-18 at 14:32 +0530, Naveen N. Rao wrote:

+#ifdef CONFIG_PPC_BOOK3S_64
+static char *print_trap(unsigned long trapno)
+{
+       trapno &= 0xff0;
+       switch (trapno) {
+               case 0x100:     return "SRESET";
+               case 0x200:     return "MCE";
+               case 0x300:     return "DSI";
+               case 0x380:     return "DSISLB";
+               case 0x400:     return "ISI";
+               case 0x480:     return "ISISLB";

380 and 480 should probably be "DSEG" and "ISEG" ... they may or may
not involve the SLB.

Ok, thanks for the review.

- Naveen


Reply via email to