[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/5] target/tricore: Implement a qemu excptio
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/5] target/tricore: Implement a qemu excptions helper |
Date: |
Wed, 21 Aug 2019 16:06:42 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 8/21/19 4:05 PM, Richard Henderson wrote:
> On 8/21/19 5:23 AM, Bastian Koppelmann wrote:
>> @@ -3928,7 +3937,7 @@ static void decode_sr_system(DisasContext *ctx)
>> ctx->base.is_jmp = DISAS_NORETURN;
>> break;
>> case OPC2_16_SR_DEBUG:
>> - /* raise EXCP_DEBUG */
>> + generate_qemu_excp(ctx, EXCP_DEBUG);
>> break;
>> case OPC2_16_SR_FRET:
>> gen_fret(ctx);
>> @@ -8354,7 +8363,7 @@ static void decode_sys_interrupts(DisasContext *ctx)
>>
>> switch (op2) {
>> case OPC2_32_SYS_DEBUG:
>> - /* raise EXCP_DEBUG */
>> + generate_qemu_excp(ctx, EXCP_DEBUG);
>> break;
>> case OPC2_32_SYS_DISABLE:
>> tcg_gen_andi_tl(cpu_ICR, cpu_ICR, ~MASK_ICR_IE_1_3);
>
> This is not correct -- EXCP_DEBUG is an internal qemu exception.
>
> The manual I have only describes the ISA and does not describe what a "Debug
> Event" would be. I note that you're missing the DBGSR.DE check. I also note
> that whatever a "Debug Event" is, RFM appears to be the return from it. So
> one
> can deduce some things about what it should be.
Anyway, remove these hunks and the rest of the patch is ok.
Reviewed-by: Richard Henderson <address@hidden>
r~
- [Qemu-devel] [PATCH v2 0/5] tricore: Convert to translate_loop (resend), Bastian Koppelmann, 2019/08/21
- [Qemu-devel] [PATCH v2 1/5] target/tricore: Use DisasContextBase API, Bastian Koppelmann, 2019/08/21
- [Qemu-devel] [PATCH v2 3/5] target/tricore: Use translate_loop, Bastian Koppelmann, 2019/08/21
- [Qemu-devel] [PATCH v2 4/5] target/tricore: Implement a qemu excptions helper, Bastian Koppelmann, 2019/08/21
- [Qemu-devel] [PATCH v2 5/5] target/tricore: Fix tricore_tr_translate_insn, Bastian Koppelmann, 2019/08/21
- [Qemu-devel] [PATCH v2 2/5] target-tricore: Make env a member of DisasContext, Bastian Koppelmann, 2019/08/21
- Re: [Qemu-devel] [PATCH v2 0/5] tricore: Convert to translate_loop (resend), no-reply, 2019/08/21
- Re: [Qemu-devel] [PATCH v2 0/5] tricore: Convert to translate_loop (resend), no-reply, 2019/08/21
- Re: [Qemu-devel] [PATCH v2 0/5] tricore: Convert to translate_loop (resend), no-reply, 2019/08/21