[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 09/14] target/ppc: Move patching nip from exception handle
From: |
Nicholas Piggin |
Subject: |
Re: [PATCH v3 09/14] target/ppc: Move patching nip from exception handler to helper_scv |
Date: |
Mon, 26 Jun 2023 21:25:55 +1000 |
On Tue Jun 20, 2023 at 8:47 PM AEST, BALATON Zoltan wrote:
> On Tue, 20 Jun 2023, Nicholas Piggin wrote:
> > On Fri Jun 16, 2023 at 9:03 AM AEST, BALATON Zoltan wrote:
> >> From: Nicholas Piggin <npiggin@gmail.com>
> >>
> >> Unlike sc, for scv a facility unavailable interrupt must be generated
> >> if FSCR[SCV]=0 so we can't raise the exception with nip set to next
> >> instruction but we can move advancing nip if the FSCR check passes to
> >> helper_scv so the exception handler does not need to change it.
> >>
> >> [balaton: added commit message]
> >> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
> >
> > Ah you sent it, fine, thank you. But actually now I look again,
> > now we're off by one in the other direction for the dumps.
>
> This is mentioned in the commit message for the patch changing sc. I think
> we should not patch nip in the dump so we actually dump what the CPU
> should have and match the ISA docs.
>
> > So... probably your way is still better because it matches the
> > interrupt semantics of the ISA when executing the instruction,
> > but it needs this patch:
>
> OK so then I'm confused why we need nip - 4 in dumps?
Sorry I missed your reply here. We want nip - 4 in dumps so the
address of the syscall is the sc instruction itself, not the
random one after it.
Thanks,
Nick
- [PATCH v3 07/14] target/ppd: Remove unused define, (continued)
- [PATCH v3 07/14] target/ppd: Remove unused define, BALATON Zoltan, 2023/06/15
- [PATCH v3 05/14] target/ppc: Remove some more local CPUState variables only used once, BALATON Zoltan, 2023/06/15
- [PATCH v3 06/14] target/ppc: Readability improvements in exception handlers, BALATON Zoltan, 2023/06/15
- [PATCH v3 08/14] target/ppc: Fix gen_sc to use correct nip, BALATON Zoltan, 2023/06/15
- [PATCH v3 09/14] target/ppc: Move patching nip from exception handler to helper_scv, BALATON Zoltan, 2023/06/15
Re: [PATCH v3 09/14] target/ppc: Move patching nip from exception handler to helper_scv, Nicholas Piggin, 2023/06/26
[PATCH v3 10/14] target/ppc: Simplify syscall exception handlers, BALATON Zoltan, 2023/06/15
[PATCH v3 11/14] target/ppc: Get CPUState in one step, BALATON Zoltan, 2023/06/15
[PATCH v3 12/14] target/ppc: Clean up ifdefs in excp_helper.c, part 1, BALATON Zoltan, 2023/06/15
[PATCH v3 14/14] target/ppc: Clean up ifdefs in excp_helper.c, part 3, BALATON Zoltan, 2023/06/15