target-arm: Correct "preferred return address" for cpreg access exceptions

The architecture defines that when taking an exception trying to
access a coprocessor register, the "preferred return address" for
the exception is the address of the instruction that caused the
exception. Correct an off-by-4 error which meant we were returning
the address after the instruction for traps which happened because
of a failure of a runtime access-check function on an AArch32
register. (Traps caused by translate-time checkable permissions
failures had the correct address, as did traps on AArch64 registers.)

This fixes https://bugs.launchpad.net/qemu/+bug/1463338

Backports commit 3977ee5d7a9f2e3664dd8b233f3224694e23b62b from qemu
This commit is contained in:
Peter Maydell 2018-02-13 14:58:26 -05:00 committed by Lioncash
parent c1554dbd06
commit 13db196792
No known key found for this signature in database
GPG key ID: 4E3C3CC1031BA9C7

View file

@ -7336,7 +7336,7 @@ static int disas_coproc_insn(DisasContext *s, uint32_t insn)
break; break;
} }
gen_set_pc_im(s, s->pc); gen_set_pc_im(s, s->pc - 4);
tmpptr = tcg_const_ptr(tcg_ctx, ri); tmpptr = tcg_const_ptr(tcg_ctx, ri);
tcg_syn = tcg_const_i32(tcg_ctx, syndrome); tcg_syn = tcg_const_i32(tcg_ctx, syndrome);
gen_helper_access_check_cp_reg(tcg_ctx, tcg_ctx->cpu_env, tmpptr, tcg_syn); gen_helper_access_check_cp_reg(tcg_ctx, tcg_ctx->cpu_env, tmpptr, tcg_syn);