Re: [Xen-devel] [PATCH] xen: arm64: more useful logging on bad trap.

2015-02-18 Thread Jintack Lim
On Wed, Feb 18, 2015 at 11:01 AM, Ian Campbell wrote: > On Wed, 2015-02-18 at 10:47 -0500, Jintack Lim wrote: >> On Wed, Feb 18, 2015 at 10:19 AM, Ian Campbell >> wrote: >> > Dump the register state before panicing so we have some clue where the >> > issue occurred. Also decode the ESR register

Re: [Xen-devel] [PATCH] xen: arm64: more useful logging on bad trap.

2015-02-18 Thread Jintack Lim
On Wed, Feb 18, 2015 at 11:00 AM, Julien Grall wrote: > > On 18/02/2015 15:47, Jintack Lim wrote: >> >> On Wed, Feb 18, 2015 at 10:19 AM, Ian Campbell >> wrote: >>> >>> Dump the register state before panicing so we have some clue where the >>> issue occurred. Also decode the ESR register a bit to

Re: [Xen-devel] [PATCH] xen: arm64: more useful logging on bad trap.

2015-02-18 Thread Julien Grall
On 18/02/2015 15:47, Jintack Lim wrote: On Wed, Feb 18, 2015 at 10:19 AM, Ian Campbell wrote: Dump the register state before panicing so we have some clue where the issue occurred. Also decode the ESR register a bit to save having to grab a pen and paper. ESR_EL2 is a 32-bit register, so use

Re: [Xen-devel] [PATCH] xen: arm64: more useful logging on bad trap.

2015-02-18 Thread Ian Campbell
On Wed, 2015-02-18 at 16:00 +, Julien Grall wrote: > Can you try to get the line of code related to this PC? You could do it > with addr2line. >From private mail I know that this machine sees issues under native Linux too, I suspect a h/w fault so I don't think it is worth diagnosing further

Re: [Xen-devel] [PATCH] xen: arm64: more useful logging on bad trap.

2015-02-18 Thread Ian Campbell
On Wed, 2015-02-18 at 10:47 -0500, Jintack Lim wrote: > On Wed, Feb 18, 2015 at 10:19 AM, Ian Campbell > wrote: > > Dump the register state before panicing so we have some clue where the > > issue occurred. Also decode the ESR register a bit to save having to > > grab a pen and paper. > > > > ESR

Re: [Xen-devel] [PATCH] xen: arm64: more useful logging on bad trap.

2015-02-18 Thread Jintack Lim
On Wed, Feb 18, 2015 at 10:19 AM, Ian Campbell wrote: > Dump the register state before panicing so we have some clue where the > issue occurred. Also decode the ESR register a bit to save having to > grab a pen and paper. > > ESR_EL2 is a 32-bit register, so use SYSREG_READ32 not ..._READ64, as >

Re: [Xen-devel] [PATCH] xen: arm64: more useful logging on bad trap.

2015-02-18 Thread Julien Grall
Hi Ian, On 18/02/2015 15:19, Ian Campbell wrote: diff --git a/xen/arch/arm/arm64/traps.c b/xen/arch/arm/arm64/traps.c index 1693b5d..89b8eb3 100644 --- a/xen/arch/arm/arm64/traps.c +++ b/xen/arch/arm/arm64/traps.c @@ -24,11 +24,6 @@ #include -asmlinkage void do_trap_serror(struct cpu_user_r