[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/14] target-arm: Set correct syndrome for faul
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCH 03/14] target-arm: Set correct syndrome for faults on MSR DAIF*, imm |
Date: |
Thu, 28 May 2015 21:44:52 +1000 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, May 28, 2015 at 12:40:42PM +0100, Peter Maydell wrote:
> On 28 May 2015 at 09:30, Peter Maydell <address@hidden> wrote:
> > On 28 May 2015 at 06:30, Edgar E. Iglesias <address@hidden> wrote:
> >>> @@ -381,6 +381,9 @@ void HELPER(msr_i_pstate)(CPUARMState *env, uint32_t
> >>> op, uint32_t imm)
> >>> */
> >>> if (arm_current_el(env) == 0 && !(env->cp15.sctlr_el[1] &
> >>> SCTLR_UMA)) {
> >>> env->exception.target_el = exception_target_el(env);
> >>> + env->exception.syndrome = syn_aa64_sysregtrap(0, extract32(op,
> >>> 0, 3),
> >>> + extract32(op, 3,
> >>> 3), 4,
> >>> + 0x1f, imm, 0);
> >>
> >> Did you possibly reverse the argument order of 0x1f and imm?
> >
> > Ah, you're right; I was confused because the argument order of our
> > syn_aa64_sysregtrap() and the pseudocode AArch64.SystemRegisterTrap
> > is different (the latter follows the field order in the syndrome
> > register and ours doesn't).
>
> I also managed to forget the signoff:
> Signed-off-by: Peter Maydell <address@hidden>
>
> If this is the only fix in this series (I think you have one or two
> patches still left to review) then I propose to add it as I put
> the patches in target-arm.next:
>
> - 0x1f, imm, 0);
> + imm, 0x1f, 0);
Yes, with these changes:
Reviewed-by: Edgar E. Iglesias <address@hidden>
One of the patches had a missing target_el assigment for
CP_ACCESS_TRAP_UNCATEGORIZED that I commented on a while back.
I'll have a look again.
Thanks,
Edgar
>
> thanks
> -- PMM
- Re: [Qemu-devel] [PATCH 12/14] target-arm: Move TB flags down to fill gap, (continued)
- [Qemu-devel] [PATCH 04/14] target-arm: Move setting of exception info into tlb_fill, Peter Maydell, 2015/05/19
- [Qemu-devel] [PATCH 02/14] target-arm: Extend helpers to route exceptions, Peter Maydell, 2015/05/19
- [Qemu-devel] [PATCH 06/14] target-arm: Make raise_exception() take syndrome and target EL, Peter Maydell, 2015/05/19
- [Qemu-devel] [PATCH 03/14] target-arm: Set correct syndrome for faults on MSR DAIF*, imm, Peter Maydell, 2015/05/19
- Re: [Qemu-devel] [PATCH 03/14] target-arm: Set correct syndrome for faults on MSR DAIF*, imm, Edgar E. Iglesias, 2015/05/28
- Re: [Qemu-devel] [PATCH 03/14] target-arm: Set correct syndrome for faults on MSR DAIF*, imm, Peter Maydell, 2015/05/28
- Re: [Qemu-devel] [PATCH 03/14] target-arm: Set correct syndrome for faults on MSR DAIF*, imm, Peter Maydell, 2015/05/28
- Re: [Qemu-devel] [PATCH 03/14] target-arm: Set correct syndrome for faults on MSR DAIF*, imm,
Edgar E. Iglesias <=
- Re: [Qemu-devel] [PATCH 03/14] target-arm: Set correct syndrome for faults on MSR DAIF*, imm, Peter Maydell, 2015/05/28
[Qemu-devel] [PATCH 09/14] target-arm: Add AArch64 CPTR registers, Peter Maydell, 2015/05/19
[Qemu-devel] [PATCH 14/14] target-arm: Add WFx instruction trap support, Peter Maydell, 2015/05/19
[Qemu-devel] [PATCH 11/14] target-arm: Extend FP checks to use an EL, Peter Maydell, 2015/05/19