target/i386: Introduce DISAS_EOB*
Add a few DISAS_TARGET_* aliases to reduce the number of calls to gen_eob() and gen_eob_inhibit_irq(). So far, only update i386_tr_translate_insn for exiting the block because of single-step or previous inhibit irq. Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Message-Id: <20221001140935.465607-8-richard.henderson@linaro.org> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
09e99df4d5
commit
200ef60399
@ -132,6 +132,10 @@ typedef struct DisasContext {
|
|||||||
TCGOp *prev_insn_end;
|
TCGOp *prev_insn_end;
|
||||||
} DisasContext;
|
} DisasContext;
|
||||||
|
|
||||||
|
#define DISAS_EOB_ONLY DISAS_TARGET_0
|
||||||
|
#define DISAS_EOB_NEXT DISAS_TARGET_1
|
||||||
|
#define DISAS_EOB_INHIBIT_IRQ DISAS_TARGET_2
|
||||||
|
|
||||||
/* The environment in which user-only runs is constrained. */
|
/* The environment in which user-only runs is constrained. */
|
||||||
#ifdef CONFIG_USER_ONLY
|
#ifdef CONFIG_USER_ONLY
|
||||||
#define PE(S) true
|
#define PE(S) true
|
||||||
@ -8849,7 +8853,7 @@ static void i386_tr_translate_insn(DisasContextBase *dcbase, CPUState *cpu)
|
|||||||
* the flag and abort the translation to give the irqs a
|
* the flag and abort the translation to give the irqs a
|
||||||
* chance to happen.
|
* chance to happen.
|
||||||
*/
|
*/
|
||||||
dc->base.is_jmp = DISAS_TOO_MANY;
|
dc->base.is_jmp = DISAS_EOB_NEXT;
|
||||||
} else if (!is_same_page(&dc->base, pc_next)) {
|
} else if (!is_same_page(&dc->base, pc_next)) {
|
||||||
dc->base.is_jmp = DISAS_TOO_MANY;
|
dc->base.is_jmp = DISAS_TOO_MANY;
|
||||||
}
|
}
|
||||||
@ -8861,9 +8865,24 @@ static void i386_tr_tb_stop(DisasContextBase *dcbase, CPUState *cpu)
|
|||||||
{
|
{
|
||||||
DisasContext *dc = container_of(dcbase, DisasContext, base);
|
DisasContext *dc = container_of(dcbase, DisasContext, base);
|
||||||
|
|
||||||
if (dc->base.is_jmp == DISAS_TOO_MANY) {
|
switch (dc->base.is_jmp) {
|
||||||
|
case DISAS_NORETURN:
|
||||||
|
break;
|
||||||
|
case DISAS_TOO_MANY:
|
||||||
|
case DISAS_EOB_NEXT:
|
||||||
|
gen_update_cc_op(dc);
|
||||||
gen_update_eip_cur(dc);
|
gen_update_eip_cur(dc);
|
||||||
|
/* fall through */
|
||||||
|
case DISAS_EOB_ONLY:
|
||||||
gen_eob(dc);
|
gen_eob(dc);
|
||||||
|
break;
|
||||||
|
case DISAS_EOB_INHIBIT_IRQ:
|
||||||
|
gen_update_cc_op(dc);
|
||||||
|
gen_update_eip_cur(dc);
|
||||||
|
gen_eob_inhibit_irq(dc, true);
|
||||||
|
break;
|
||||||
|
default:
|
||||||
|
g_assert_not_reached();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user