mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-04 00:03:54 -06:00
target/arm: Introduce gen_exception_insn
Create a new wrapper function that passes the default exception target to gen_exception_insn_el. Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Message-id: 20220609202901.1177572-13-richard.henderson@linaro.org Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
parent
8c5d24dc7d
commit
486d6c9699
5 changed files with 22 additions and 29 deletions
|
@ -1106,6 +1106,11 @@ void gen_exception_insn_el(DisasContext *s, uint64_t pc, int excp,
|
|||
gen_exception_insn_el_v(s, pc, excp, syn, tcg_constant_i32(target_el));
|
||||
}
|
||||
|
||||
void gen_exception_insn(DisasContext *s, uint64_t pc, int excp, uint32_t syn)
|
||||
{
|
||||
gen_exception_insn_el(s, pc, excp, syn, default_exception_el(s));
|
||||
}
|
||||
|
||||
static void gen_exception_bkpt_insn(DisasContext *s, uint32_t syn)
|
||||
{
|
||||
gen_set_condexec(s);
|
||||
|
@ -1117,8 +1122,7 @@ static void gen_exception_bkpt_insn(DisasContext *s, uint32_t syn)
|
|||
void unallocated_encoding(DisasContext *s)
|
||||
{
|
||||
/* Unallocated and reserved encodings are uncategorized */
|
||||
gen_exception_insn_el(s, s->pc_curr, EXCP_UDEF, syn_uncategorized(),
|
||||
default_exception_el(s));
|
||||
gen_exception_insn(s, s->pc_curr, EXCP_UDEF, syn_uncategorized());
|
||||
}
|
||||
|
||||
/* Force a TB lookup after an instruction that changes the CPU state. */
|
||||
|
@ -2731,8 +2735,6 @@ static bool msr_banked_access_decode(DisasContext *s, int r, int sysm, int rn,
|
|||
* an exception and return false. Otherwise it will return true,
|
||||
* and set *tgtmode and *regno appropriately.
|
||||
*/
|
||||
int exc_target = default_exception_el(s);
|
||||
|
||||
/* These instructions are present only in ARMv8, or in ARMv7 with the
|
||||
* Virtualization Extensions.
|
||||
*/
|
||||
|
@ -2869,8 +2871,7 @@ static bool msr_banked_access_decode(DisasContext *s, int r, int sysm, int rn,
|
|||
|
||||
undef:
|
||||
/* If we get here then some access check did not pass */
|
||||
gen_exception_insn_el(s, s->pc_curr, EXCP_UDEF,
|
||||
syn_uncategorized(), exc_target);
|
||||
gen_exception_insn(s, s->pc_curr, EXCP_UDEF, syn_uncategorized());
|
||||
return false;
|
||||
}
|
||||
|
||||
|
@ -8583,8 +8584,7 @@ static bool trans_LE(DisasContext *s, arg_LE *a)
|
|||
tmp = load_cpu_field(v7m.ltpsize);
|
||||
tcg_gen_brcondi_i32(TCG_COND_EQ, tmp, 4, skipexc);
|
||||
tcg_temp_free_i32(tmp);
|
||||
gen_exception_insn_el(s, s->pc_curr, EXCP_INVSTATE, syn_uncategorized(),
|
||||
default_exception_el(s));
|
||||
gen_exception_insn(s, s->pc_curr, EXCP_INVSTATE, syn_uncategorized());
|
||||
gen_set_label(skipexc);
|
||||
}
|
||||
|
||||
|
@ -9054,8 +9054,7 @@ static void disas_arm_insn(DisasContext *s, unsigned int insn)
|
|||
* UsageFault exception.
|
||||
*/
|
||||
if (arm_dc_feature(s, ARM_FEATURE_M)) {
|
||||
gen_exception_insn_el(s, s->pc_curr, EXCP_INVSTATE, syn_uncategorized(),
|
||||
default_exception_el(s));
|
||||
gen_exception_insn(s, s->pc_curr, EXCP_INVSTATE, syn_uncategorized());
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -9064,8 +9063,7 @@ static void disas_arm_insn(DisasContext *s, unsigned int insn)
|
|||
* Illegal execution state. This has priority over BTI
|
||||
* exceptions, but comes after instruction abort exceptions.
|
||||
*/
|
||||
gen_exception_insn_el(s, s->pc_curr, EXCP_UDEF,
|
||||
syn_illegalstate(), default_exception_el(s));
|
||||
gen_exception_insn(s, s->pc_curr, EXCP_UDEF, syn_illegalstate());
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -9634,8 +9632,7 @@ static void thumb_tr_translate_insn(DisasContextBase *dcbase, CPUState *cpu)
|
|||
* Illegal execution state. This has priority over BTI
|
||||
* exceptions, but comes after instruction abort exceptions.
|
||||
*/
|
||||
gen_exception_insn_el(dc, dc->pc_curr, EXCP_UDEF,
|
||||
syn_illegalstate(), default_exception_el(dc));
|
||||
gen_exception_insn(dc, dc->pc_curr, EXCP_UDEF, syn_illegalstate());
|
||||
return;
|
||||
}
|
||||
|
||||
|
@ -9708,8 +9705,8 @@ static void thumb_tr_translate_insn(DisasContextBase *dcbase, CPUState *cpu)
|
|||
*/
|
||||
tcg_remove_ops_after(dc->insn_eci_rewind);
|
||||
dc->condjmp = 0;
|
||||
gen_exception_insn_el(dc, dc->pc_curr, EXCP_INVSTATE, syn_uncategorized(),
|
||||
default_exception_el(dc));
|
||||
gen_exception_insn(dc, dc->pc_curr, EXCP_INVSTATE,
|
||||
syn_uncategorized());
|
||||
}
|
||||
|
||||
arm_post_translate_insn(dc);
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue