[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v9 21/26] target: [tcg, arm] Port to insn_start
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH v9 21/26] target: [tcg, arm] Port to insn_start |
Date: |
Wed, 28 Jun 2017 14:48:07 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Richard Henderson writes:
> On 06/25/2017 03:08 AM, Lluís Vilanova wrote:
>> +static void arm_trblock_insn_start(DisasContextBase *db, CPUState *cpu)
>> +{
>> + DisasContext *dc = container_of(db, DisasContext, base);
>> +
>> + dc->insn_start_idx = tcg_op_buf_count();
>> + tcg_gen_insn_start(dc->pc,
>> + (dc->condexec_cond << 4) | (dc->condexec_mask >> 1),
>> + 0);
>> +
>> +#ifdef CONFIG_USER_ONLY
>> + /* Intercept jump to the magic kernel page. */
>> + if (dc->pc >= 0xffff0000) {
>> + /* We always get here via a jump, so know we are not in a
>> + conditional execution block. */
>> + gen_exception_internal(EXCP_KERNEL_TRAP);
>> + dc->is_jmp = DJ_EXC;
>> + }
>> +#endif
>> +}
> I think this DJ_EXC block is better placed at the start of disas_insn.
> It is, however, the answer to one of my questions re patch 4/26.
It cannot, because DJ_EXC make the generic code break out of trasnlation before
doing anything else, which is what the original code does.
Cheers,
Lluis
- [Qemu-devel] [PATCH v9 14/26] target: [tcg, i386] Replace DISAS_* with DJ_*, (continued)
- [Qemu-devel] [PATCH v9 14/26] target: [tcg, i386] Replace DISAS_* with DJ_*, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 16/26] target: [tcg, arm] Replace DISAS_* with DJ_*, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 17/26] target: [tcg, arm] Port to DisasContextBase, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 18/26] target: [tcg, arm] Port to init_disas_context, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 19/26] target: [tcg, arm] Port to init_globals, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 20/26] target: [tcg,arm] Port to tb_start, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 21/26] target: [tcg, arm] Port to insn_start, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 22/26] target: [tcg, arm] Port to breakpoint_check, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 23/26] target: [tcg, arm] Port to disas_insn, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 24/26] target: [tcg,arm] Port to tb_stop, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 25/26] target: [tcg, arm] Port to disas_flags, Lluís Vilanova, 2017/06/25
- [Qemu-devel] [PATCH v9 26/26] target: [tcg, arm] Port to generic translation framework, Lluís Vilanova, 2017/06/25
- Re: [Qemu-devel] [RFC PATCH v9 00/26] translate: [tcg] Generic translation framework, Alex Bennée, 2017/06/26
- Re: [Qemu-devel] [RFC PATCH v9 00/26] translate: [tcg] Generic translation framework, Eric Blake, 2017/06/26