[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v12 05/27] target/i386: [tcg] Port to DisasConte
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v12 05/27] target/i386: [tcg] Port to DisasContextBase |
Date: |
Wed, 12 Jul 2017 10:18:27 +0100 |
User-agent: |
mu4e 0.9.19; emacs 25.2.50.3 |
Lluís Vilanova <address@hidden> writes:
> Incrementally paves the way towards using the generic instruction translation
> loop.
>
> Signed-off-by: Lluís Vilanova <address@hidden>
> Reviewed-by: Emilio G. Cota <address@hidden>
> Reviewed-by: Richard Henderson <address@hidden>
> ---
> target/i386/translate.c | 142
> +++++++++++++++++++++++------------------------
> 1 file changed, 70 insertions(+), 72 deletions(-)
>
> diff --git a/target/i386/translate.c b/target/i386/translate.c
> index b118fcb834..f61f5c7227 100644
> --- a/target/i386/translate.c
> +++ b/target/i386/translate.c
> @@ -99,6 +99,8 @@ static int x86_64_hregs;
> #endif
>
> typedef struct DisasContext {
> + DisasContextBase base;
> +
> /* current insn context */
> int override; /* -1 if no override */
> int prefix;
> @@ -106,8 +108,6 @@ typedef struct DisasContext {
> TCGMemOp dflag;
> target_ulong pc_start;
> target_ulong pc; /* pc = eip + cs_base */
> - int is_jmp; /* 1 = means jump (stop translation), 2 means CPU
> - static state change (stop translation) */
> /* current block context */
> target_ulong cs_base; /* base of CS segment */
> int pe; /* protected mode */
> @@ -128,12 +128,10 @@ typedef struct DisasContext {
> int cpl;
> int iopl;
> int tf; /* TF cpu flag */
> - int singlestep_enabled; /* "hardware" single step enabled */
> int jmp_opt; /* use direct block chaining for direct jumps */
> int repz_opt; /* optimize jumps within repz instructions */
> int mem_index; /* select memory access functions */
> uint64_t flags; /* all execution flags */
> - struct TranslationBlock *tb;
> int popl_esp_hack; /* for correct popl with esp base handling */
> int rip_offset; /* only used in x86_64, but left for simplicity */
> int cpuid_features;
> @@ -1123,7 +1121,7 @@ static void gen_bpt_io(DisasContext *s, TCGv_i32
> t_port, int ot)
>
> static inline void gen_ins(DisasContext *s, TCGMemOp ot)
> {
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> gen_string_movl_A0_EDI(s);
> @@ -1138,14 +1136,14 @@ static inline void gen_ins(DisasContext *s, TCGMemOp
> ot)
> gen_op_movl_T0_Dshift(ot);
> gen_op_add_reg_T0(s->aflag, R_EDI);
> gen_bpt_io(s, cpu_tmp2_i32, ot);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> }
> }
>
> static inline void gen_outs(DisasContext *s, TCGMemOp ot)
> {
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> gen_string_movl_A0_ESI(s);
> @@ -1158,7 +1156,7 @@ static inline void gen_outs(DisasContext *s, TCGMemOp
> ot)
> gen_op_movl_T0_Dshift(ot);
> gen_op_add_reg_T0(s->aflag, R_ESI);
> gen_bpt_io(s, cpu_tmp2_i32, ot);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> }
> }
> @@ -2141,7 +2139,7 @@ static inline int insn_const_size(TCGMemOp ot)
> static inline bool use_goto_tb(DisasContext *s, target_ulong pc)
> {
> #ifndef CONFIG_USER_ONLY
> - return (pc & TARGET_PAGE_MASK) == (s->tb->pc & TARGET_PAGE_MASK) ||
> + return (pc & TARGET_PAGE_MASK) == (s->base.tb->pc & TARGET_PAGE_MASK) ||
> (pc & TARGET_PAGE_MASK) == (s->pc_start & TARGET_PAGE_MASK);
> #else
> return true;
> @@ -2156,7 +2154,7 @@ static inline void gen_goto_tb(DisasContext *s, int
> tb_num, target_ulong eip)
> /* jump to same page: we can use a direct jump */
> tcg_gen_goto_tb(tb_num);
> gen_jmp_im(eip);
> - tcg_gen_exit_tb((uintptr_t)s->tb + tb_num);
> + tcg_gen_exit_tb((uintptr_t)s->base.tb + tb_num);
> } else {
> /* jump to another page */
> gen_jmp_im(eip);
> @@ -2177,7 +2175,7 @@ static inline void gen_jcc(DisasContext *s, int b,
>
> gen_set_label(l1);
> gen_goto_tb(s, 1, val);
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> } else {
> l1 = gen_new_label();
> l2 = gen_new_label();
> @@ -2248,11 +2246,11 @@ static void gen_movl_seg_T0(DisasContext *s, int
> seg_reg)
> stop as a special handling must be done to disable hardware
> interrupts for the next instruction */
> if (seg_reg == R_SS || (s->code32 && seg_reg < R_FS))
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> } else {
> gen_op_movl_seg_T0_vm(seg_reg);
> if (seg_reg == R_SS)
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> }
> }
>
> @@ -2424,7 +2422,7 @@ static void gen_exception(DisasContext *s, int trapno,
> target_ulong cur_eip)
> gen_update_cc_op(s);
> gen_jmp_im(cur_eip);
> gen_helper_raise_exception(cpu_env, tcg_const_i32(trapno));
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> }
>
> /* Generate #UD for the current instruction. The assumption here is that
> @@ -2462,7 +2460,7 @@ static void gen_interrupt(DisasContext *s, int intno,
> gen_jmp_im(cur_eip);
> gen_helper_raise_interrupt(cpu_env, tcg_const_i32(intno),
> tcg_const_i32(next_eip - cur_eip));
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> }
>
> static void gen_debug(DisasContext *s, target_ulong cur_eip)
> @@ -2470,7 +2468,7 @@ static void gen_debug(DisasContext *s, target_ulong
> cur_eip)
> gen_update_cc_op(s);
> gen_jmp_im(cur_eip);
> gen_helper_debug(cpu_env);
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> }
>
> static void gen_set_hflag(DisasContext *s, uint32_t mask)
> @@ -2526,10 +2524,10 @@ do_gen_eob_worker(DisasContext *s, bool inhibit, bool
> recheck_tf, TCGv jr)
> gen_reset_hflag(s, HF_INHIBIT_IRQ_MASK);
> }
>
> - if (s->tb->flags & HF_RF_MASK) {
> + if (s->base.tb->flags & HF_RF_MASK) {
> gen_helper_reset_rf(cpu_env);
> }
> - if (s->singlestep_enabled) {
> + if (s->base.singlestep_enabled) {
> gen_helper_debug(cpu_env);
> } else if (recheck_tf) {
> gen_helper_rechecking_single_step(cpu_env);
> @@ -2545,7 +2543,7 @@ do_gen_eob_worker(DisasContext *s, bool inhibit, bool
> recheck_tf, TCGv jr)
> } else {
> tcg_gen_exit_tb(0);
> }
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> }
>
> static inline void
> @@ -2584,7 +2582,7 @@ static void gen_jmp_tb(DisasContext *s, target_ulong
> eip, int tb_num)
> set_cc_op(s, CC_OP_DYNAMIC);
> if (s->jmp_opt) {
> gen_goto_tb(s, tb_num, eip);
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> } else {
> gen_jmp_im(eip);
> gen_eob(s);
> @@ -4419,7 +4417,7 @@ static void gen_sse(CPUX86State *env, DisasContext *s,
> int b,
> }
> }
>
> -/* convert one instruction. s->is_jmp is set if the translation must
> +/* convert one instruction. s->base.is_jmp is set if the translation must
> be stopped. Return the next pc value */
> static target_ulong disas_insn(CPUX86State *env, DisasContext *s,
> target_ulong pc_start)
> @@ -5379,7 +5377,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_movl_seg_T0(s, reg);
> gen_pop_update(s, ot);
> /* Note that reg == R_SS in gen_movl_seg_T0 always sets is_jmp. */
> - if (s->is_jmp) {
> + if (s->base.is_jmp) {
> gen_jmp_im(s->pc - s->cs_base);
> if (reg == R_SS) {
> s->tf = 0;
> @@ -5394,7 +5392,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> ot = gen_pop_T0(s);
> gen_movl_seg_T0(s, (b >> 3) & 7);
> gen_pop_update(s, ot);
> - if (s->is_jmp) {
> + if (s->base.is_jmp) {
> gen_jmp_im(s->pc - s->cs_base);
> gen_eob(s);
> }
> @@ -5445,7 +5443,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_ldst_modrm(env, s, modrm, MO_16, OR_TMP0, 0);
> gen_movl_seg_T0(s, reg);
> /* Note that reg == R_SS in gen_movl_seg_T0 always sets is_jmp. */
> - if (s->is_jmp) {
> + if (s->base.is_jmp) {
> gen_jmp_im(s->pc - s->cs_base);
> if (reg == R_SS) {
> s->tf = 0;
> @@ -5654,7 +5652,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_movl_seg_T0(s, op);
> /* then put the data */
> gen_op_mov_reg_v(ot, reg, cpu_T1);
> - if (s->is_jmp) {
> + if (s->base.is_jmp) {
> gen_jmp_im(s->pc - s->cs_base);
> gen_eob(s);
> }
> @@ -6310,7 +6308,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_repz_ins(s, ot, pc_start - s->cs_base, s->pc - s->cs_base);
> } else {
> gen_ins(s, ot);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_jmp(s, s->pc - s->cs_base);
> }
> }
> @@ -6325,7 +6323,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_repz_outs(s, ot, pc_start - s->cs_base, s->pc - s->cs_base);
> } else {
> gen_outs(s, ot);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_jmp(s, s->pc - s->cs_base);
> }
> }
> @@ -6341,14 +6339,14 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> tcg_gen_movi_tl(cpu_T0, val);
> gen_check_io(s, ot, pc_start - s->cs_base,
> SVM_IOIO_TYPE_MASK | svm_is_rep(prefixes));
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> tcg_gen_movi_i32(cpu_tmp2_i32, val);
> gen_helper_in_func(ot, cpu_T1, cpu_tmp2_i32);
> gen_op_mov_reg_v(ot, R_EAX, cpu_T1);
> gen_bpt_io(s, cpu_tmp2_i32, ot);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> gen_jmp(s, s->pc - s->cs_base);
> }
> @@ -6362,14 +6360,14 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> svm_is_rep(prefixes));
> gen_op_mov_v_reg(ot, cpu_T1, R_EAX);
>
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> tcg_gen_movi_i32(cpu_tmp2_i32, val);
> tcg_gen_trunc_tl_i32(cpu_tmp3_i32, cpu_T1);
> gen_helper_out_func(ot, cpu_tmp2_i32, cpu_tmp3_i32);
> gen_bpt_io(s, cpu_tmp2_i32, ot);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> gen_jmp(s, s->pc - s->cs_base);
> }
> @@ -6380,14 +6378,14 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> tcg_gen_ext16u_tl(cpu_T0, cpu_regs[R_EDX]);
> gen_check_io(s, ot, pc_start - s->cs_base,
> SVM_IOIO_TYPE_MASK | svm_is_rep(prefixes));
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> tcg_gen_trunc_tl_i32(cpu_tmp2_i32, cpu_T0);
> gen_helper_in_func(ot, cpu_T1, cpu_tmp2_i32);
> gen_op_mov_reg_v(ot, R_EAX, cpu_T1);
> gen_bpt_io(s, cpu_tmp2_i32, ot);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> gen_jmp(s, s->pc - s->cs_base);
> }
> @@ -6400,14 +6398,14 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> svm_is_rep(prefixes));
> gen_op_mov_v_reg(ot, cpu_T1, R_EAX);
>
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> tcg_gen_trunc_tl_i32(cpu_tmp2_i32, cpu_T0);
> tcg_gen_trunc_tl_i32(cpu_tmp3_i32, cpu_T1);
> gen_helper_out_func(ot, cpu_tmp2_i32, cpu_tmp3_i32);
> gen_bpt_io(s, cpu_tmp2_i32, ot);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> gen_jmp(s, s->pc - s->cs_base);
> }
> @@ -6946,7 +6944,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_update_cc_op(s);
> gen_jmp_im(pc_start - s->cs_base);
> gen_helper_pause(cpu_env, tcg_const_i32(s->pc - pc_start));
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> }
> break;
> case 0x9b: /* fwait */
> @@ -7115,11 +7113,11 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> case 0x131: /* rdtsc */
> gen_update_cc_op(s);
> gen_jmp_im(pc_start - s->cs_base);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> gen_helper_rdtsc(cpu_env);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> gen_jmp(s, s->pc - s->cs_base);
> }
> @@ -7191,7 +7189,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_update_cc_op(s);
> gen_jmp_im(pc_start - s->cs_base);
> gen_helper_hlt(cpu_env, tcg_const_i32(s->pc - pc_start));
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> }
> break;
> case 0x100:
> @@ -7374,7 +7372,7 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_helper_vmrun(cpu_env, tcg_const_i32(s->aflag - 1),
> tcg_const_i32(s->pc - pc_start));
> tcg_gen_exit_tb(0);
> - s->is_jmp = DISAS_TB_JUMP;
> + s->base.is_jmp = DISAS_TB_JUMP;
> break;
>
> case 0xd9: /* VMMCALL */
> @@ -7574,11 +7572,11 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> }
> gen_update_cc_op(s);
> gen_jmp_im(pc_start - s->cs_base);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> gen_helper_rdtscp(cpu_env);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> gen_jmp(s, s->pc - s->cs_base);
> }
> @@ -7943,24 +7941,24 @@ static target_ulong disas_insn(CPUX86State *env,
> DisasContext *s,
> gen_update_cc_op(s);
> gen_jmp_im(pc_start - s->cs_base);
> if (b & 2) {
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> gen_op_mov_v_reg(ot, cpu_T0, rm);
> gen_helper_write_crN(cpu_env, tcg_const_i32(reg),
> cpu_T0);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> }
> gen_jmp_im(s->pc - s->cs_base);
> gen_eob(s);
> } else {
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_start();
> }
> gen_helper_read_crN(cpu_T0, cpu_env, tcg_const_i32(reg));
> gen_op_mov_reg_v(ot, rm, cpu_T0);
> - if (s->tb->cflags & CF_USE_ICOUNT) {
> + if (s->base.tb->cflags & CF_USE_ICOUNT) {
> gen_io_end();
> }
> }
> @@ -8386,15 +8384,13 @@ void gen_intermediate_code(CPUState *cs,
> TranslationBlock *tb)
> {
> CPUX86State *env = cs->env_ptr;
> DisasContext dc1, *dc = &dc1;
> - target_ulong pc_ptr;
> uint32_t flags;
> - target_ulong pc_start;
> target_ulong cs_base;
> int num_insns;
> int max_insns;
>
> /* generate intermediate code */
> - pc_start = tb->pc;
> + dc->base.pc_first = tb->pc;
> cs_base = tb->cs_base;
> flags = tb->flags;
>
> @@ -8407,11 +8403,11 @@ void gen_intermediate_code(CPUState *cs,
> TranslationBlock *tb)
> dc->cpl = (flags >> HF_CPL_SHIFT) & 3;
> dc->iopl = (flags >> IOPL_SHIFT) & 3;
> dc->tf = (flags >> TF_SHIFT) & 1;
> - dc->singlestep_enabled = cs->singlestep_enabled;
> + dc->base.singlestep_enabled = cs->singlestep_enabled;
> dc->cc_op = CC_OP_DYNAMIC;
> dc->cc_op_dirty = false;
> dc->cs_base = cs_base;
> - dc->tb = tb;
> + dc->base.tb = tb;
> dc->popl_esp_hack = 0;
> /* select memory access functions */
> dc->mem_index = 0;
> @@ -8461,8 +8457,8 @@ void gen_intermediate_code(CPUState *cs,
> TranslationBlock *tb)
> cpu_ptr1 = tcg_temp_new_ptr();
> cpu_cc_srcT = tcg_temp_local_new();
>
> - dc->is_jmp = DISAS_NEXT;
> - pc_ptr = pc_start;
> + dc->base.is_jmp = DISAS_NEXT;
> + dc->base.pc_next = dc->base.pc_first;
Given the point of the DisasContextBase is it should be common
boilerplate it might be worth having a:
init_dcbase(dc->base, tb, cs->singlestep_enabled)
So we can keep this all in one place rather than in each targets
preamble code.
> num_insns = 0;
> max_insns = tb->cflags & CF_COUNT_MASK;
> if (max_insns == 0) {
> @@ -8474,37 +8470,38 @@ void gen_intermediate_code(CPUState *cs,
> TranslationBlock *tb)
>
> gen_tb_start(tb);
> for(;;) {
> - tcg_gen_insn_start(pc_ptr, dc->cc_op);
> + tcg_gen_insn_start(dc->base.pc_next, dc->cc_op);
> num_insns++;
>
> /* If RF is set, suppress an internally generated breakpoint. */
> - if (unlikely(cpu_breakpoint_test(cs, pc_ptr,
> + if (unlikely(cpu_breakpoint_test(cs, dc->base.pc_next,
> tb->flags & HF_RF_MASK
> ? BP_GDB : BP_ANY))) {
> - gen_debug(dc, pc_ptr - dc->cs_base);
> + gen_debug(dc, dc->base.pc_next - dc->cs_base);
> /* The address covered by the breakpoint must be included in
> [tb->pc, tb->pc + tb->size) in order to for it to be
> properly cleared -- thus we increment the PC here so that
> the logic setting tb->size below does the right thing. */
> - pc_ptr += 1;
> + dc->base.pc_next += 1;
> goto done_generating;
> }
> if (num_insns == max_insns && (tb->cflags & CF_LAST_IO)) {
> gen_io_start();
> }
>
> - pc_ptr = disas_insn(env, dc, pc_ptr);
> + dc->base.pc_next = disas_insn(env, dc, dc->base.pc_next);
> /* stop translation if indicated */
> - if (dc->is_jmp)
> + if (dc->base.is_jmp) {
> break;
> + }
> /* if single step mode, we generate only one instruction and
> generate an exception */
> /* if irq were inhibited with HF_INHIBIT_IRQ_MASK, we clear
> the flag and abort the translation to give the irqs a
> change to be happen */
> - if (dc->tf || dc->singlestep_enabled ||
> + if (dc->tf || dc->base.singlestep_enabled ||
> (flags & HF_INHIBIT_IRQ_MASK)) {
> - gen_jmp_im(pc_ptr - dc->cs_base);
> + gen_jmp_im(dc->base.pc_next - dc->cs_base);
> gen_eob(dc);
> break;
> }
> @@ -8515,23 +8512,23 @@ void gen_intermediate_code(CPUState *cs,
> TranslationBlock *tb)
> because an exception hasn't stopped this code.
> */
> if ((tb->cflags & CF_USE_ICOUNT)
> - && ((pc_ptr & TARGET_PAGE_MASK)
> - != ((pc_ptr + TARGET_MAX_INSN_SIZE - 1) & TARGET_PAGE_MASK)
> - || (pc_ptr & ~TARGET_PAGE_MASK) == 0)) {
> - gen_jmp_im(pc_ptr - dc->cs_base);
> + && ((dc->base.pc_next & TARGET_PAGE_MASK)
> + != ((dc->base.pc_next + TARGET_MAX_INSN_SIZE - 1) &
> TARGET_PAGE_MASK)
> + || (dc->base.pc_next & ~TARGET_PAGE_MASK) == 0)) {
> + gen_jmp_im(dc->base.pc_next - dc->cs_base);
> gen_eob(dc);
> break;
> }
> /* if too long translation, stop generation too */
> if (tcg_op_buf_full() ||
> - (pc_ptr - pc_start) >= (TARGET_PAGE_SIZE - 32) ||
> + (dc->base.pc_next - dc->base.pc_first) >= (TARGET_PAGE_SIZE -
> 32) ||
> num_insns >= max_insns) {
> - gen_jmp_im(pc_ptr - dc->cs_base);
> + gen_jmp_im(dc->base.pc_next - dc->cs_base);
> gen_eob(dc);
> break;
> }
> if (singlestep) {
> - gen_jmp_im(pc_ptr - dc->cs_base);
> + gen_jmp_im(dc->base.pc_next - dc->cs_base);
> gen_eob(dc);
> break;
> }
> @@ -8543,24 +8540,25 @@ done_generating:
>
> #ifdef DEBUG_DISAS
> if (qemu_loglevel_mask(CPU_LOG_TB_IN_ASM)
> - && qemu_log_in_addr_range(pc_start)) {
> + && qemu_log_in_addr_range(dc->base.pc_first)) {
> int disas_flags;
> qemu_log_lock();
> qemu_log("----------------\n");
> - qemu_log("IN: %s\n", lookup_symbol(pc_start));
> + qemu_log("IN: %s\n", lookup_symbol(dc->base.pc_first));
> #ifdef TARGET_X86_64
> if (dc->code64)
> disas_flags = 2;
> else
> #endif
> disas_flags = !dc->code32;
> - log_target_disas(cs, pc_start, pc_ptr - pc_start, disas_flags);
> + log_target_disas(cs, dc->base.pc_first, dc->base.pc_next -
> dc->base.pc_first,
> + disas_flags);
> qemu_log("\n");
> qemu_log_unlock();
> }
> #endif
>
> - tb->size = pc_ptr - pc_start;
> + tb->size = dc->base.pc_next - dc->base.pc_first;
> tb->icount = num_insns;
> }
>
That said:
Reviewed-by: Alex Bennée <address@hidden>
--
Alex Bennée
- Re: [Qemu-devel] [PATCH v12 03/27] target: [tcg] Use a generic enum for DISAS_ values, (continued)
- [Qemu-devel] [PATCH v12 04/27] target: [tcg] Add generic translation framework, Lluís Vilanova, 2017/07/07
- [Qemu-devel] [PATCH v12 05/27] target/i386: [tcg] Port to DisasContextBase, Lluís Vilanova, 2017/07/07
- Re: [Qemu-devel] [PATCH v12 05/27] target/i386: [tcg] Port to DisasContextBase,
Alex Bennée <=
- [Qemu-devel] [PATCH v12 06/27] target/i386: [tcg] Port to init_disas_context, Lluís Vilanova, 2017/07/07
- [Qemu-devel] [PATCH v12 07/27] target/i386: [tcg] Port to insn_start, Lluís Vilanova, 2017/07/07
- [Qemu-devel] [PATCH v12 08/27] target/i386: [tcg] Port to breakpoint_check, Lluís Vilanova, 2017/07/07
- [Qemu-devel] [PATCH v12 09/27] target/i386: [tcg] Port to translate_insn, Lluís Vilanova, 2017/07/07
- [Qemu-devel] [PATCH v12 10/27] target/i386: [tcg] Port to tb_stop, Lluís Vilanova, 2017/07/07
- [Qemu-devel] [PATCH v12 11/27] target/i386: [tcg] Port to disas_log, Lluís Vilanova, 2017/07/07
- [Qemu-devel] [PATCH v12 12/27] target/i386: [tcg] Port to generic translation framework, Lluís Vilanova, 2017/07/07