Re: powerpc: Teach oops about radix vectors

[Michael Ellerman]

On Thu, 2017-03-16 at 03:04:40 UTC, Michael Neuling wrote:
> Currently if we oops caused by an 0x380 or 0x480 exception, we get a
> print which assumes SLB problems. With radix, these vectors have
> different meanings.
> 
> This patch updates the oops message to reflect these different
> meanings.
> 
> Signed-off-by: Michael Neuling 

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/8915bcd68b2547c819b8a2a33c0981

cheers

[PATCH] powerpc: Teach oops about radix vectors

[Michael Neuling]

Currently if we oops caused by an 0x380 or 0x480 exception, we get a
print which assumes SLB problems. With radix, these vectors have
different meanings.

This patch updates the oops message to reflect these different
meanings.

Signed-off-by: Michael Neuling 
---
 arch/powerpc/xmon/xmon.c | 14 --
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c
index 16321ad9e7..7781f78964 100644
--- a/arch/powerpc/xmon/xmon.c
+++ b/arch/powerpc/xmon/xmon.c
@@ -1347,9 +1347,19 @@ const char *getvecname(unsigned long vec)
case 0x100: ret = "(System Reset)"; break;
case 0x200: ret = "(Machine Check)"; break;
case 0x300: ret = "(Data Access)"; break;
-   case 0x380: ret = "(Data SLB Access)"; break;
+   case 0x380:
+   if (radix_enabled())
+   ret = "(Data Access Out of Range)";
+   else
+   ret = "(Data SLB Access)";
+   break;
case 0x400: ret = "(Instruction Access)"; break;
-   case 0x480: ret = "(Instruction SLB Access)"; break;
+   case 0x480:
+   if (radix_enabled())
+   ret = "(Instruction Access Out of Range)";
+   else
+   ret = "(Instruction SLB Access)";
+   break;
case 0x500: ret = "(Hardware Interrupt)"; break;
case 0x600: ret = "(Alignment)"; break;
case 0x700: ret = "(Program Check)"; break;
-- 
2.9.3