Re: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-31 Thread Julien Grall
On 16/08/2022 08:28, Bertrand Marquis wrote: Hi Julien, Hi Bertrand, On 15 Aug 2022, at 18:04, Julien Grall wrote: On 15/08/2022 17:39, Bertrand Marquis wrote: Hi Julien, Hi Bertrand, On 12 Aug 2022, at 20:24, Julien Grall wrote: From: Julien Grall Unlike arm64, on arm32 there a

Re: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-31 Thread Julien Grall
On 16/08/2022 04:29, Henry Wang wrote: Hi Julien, Hi Henry, Sorry for the late reply. -Original Message- From: Julien Grall Subject: Re: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort Hmmm... I dropped a patch from the series and it looks like I

Re: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-16 Thread Bertrand Marquis
Hi Julien, > On 15 Aug 2022, at 18:04, Julien Grall wrote: > > > > On 15/08/2022 17:39, Bertrand Marquis wrote: >> Hi Julien, > > Hi Bertrand, > >>> On 12 Aug 2022, at 20:24, Julien Grall wrote: >>> >>> From: Julien Grall >>> >>> Unlike arm64, on arm32 there are no extra information dump

RE: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-15 Thread Henry Wang
Hi Julien, > -Original Message- > From: Julien Grall > Subject: Re: [PATCH 7/7] xen/arm32: traps: Dump more information for > hypervisor data abort > Hmmm... I dropped a patch from the series and it looks like I didn't > properly clean the directory before doing th

Re: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-15 Thread Julien Grall
On 15/08/2022 17:39, Bertrand Marquis wrote: Hi Julien, Hi Bertrand, On 12 Aug 2022, at 20:24, Julien Grall wrote: From: Julien Grall Unlike arm64, on arm32 there are no extra information dumped (e.g. page table walk) for hypervisor data abort. The code in arch/arm/traps.c has nothin

Re: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-15 Thread Julien Grall
On 15/08/2022 02:40, Henry Wang wrote: Hi Julien, Hi Henry, -Original Message- Subject: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort From: Julien Grall Unlike arm64, on arm32 there are no extra information dumped (e.g. page table walk) for

Re: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-15 Thread Bertrand Marquis
Hi Julien, > On 12 Aug 2022, at 20:24, Julien Grall wrote: > > From: Julien Grall > > Unlike arm64, on arm32 there are no extra information dumped (e.g. > page table walk) for hypervisor data abort. The code in arch/arm/traps.c has nothing arm32 specific like that so could you explain this st

RE: [PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-14 Thread Henry Wang
Hi Julien, > -Original Message- > Subject: [PATCH 7/7] xen/arm32: traps: Dump more information for > hypervisor data abort > > From: Julien Grall > > Unlike arm64, on arm32 there are no extra information dumped (e.g. > page table walk) for hypervisor data ab

[PATCH 7/7] xen/arm32: traps: Dump more information for hypervisor data abort

2022-08-12 Thread Julien Grall
From: Julien Grall Unlike arm64, on arm32 there are no extra information dumped (e.g. page table walk) for hypervisor data abort. For data abort, the HSR will be set properly and so replace the call to do_unexpected_trap() with do_trap_hyp_sync() to dispatch the error. Signed-off-by: Julien Gra