[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] target/s390x: Use ILEN_UNWIND in trivial ca
From: |
David Hildenbrand |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] target/s390x: Use ILEN_UNWIND in trivial cases |
Date: |
Tue, 25 Jul 2017 09:18:40 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
> /* FIXME: a) LAP
> diff --git a/target/s390x/misc_helper.c b/target/s390x/misc_helper.c
> index 452b2bd902..376dbd68c2 100644
> --- a/target/s390x/misc_helper.c
> +++ b/target/s390x/misc_helper.c
> @@ -57,7 +57,7 @@ void QEMU_NORETURN runtime_exception(CPUS390XState *env,
> int excp,
>
> cs->exception_index = EXCP_PGM;
> env->int_pgm_code = excp;
> - env->int_pgm_ilen = ILEN_AUTO;
> + env->int_pgm_ilen = ILEN_UNWIND;
This is the only place where int_pgm_ilen is ever set to ILEN_UNWIND.
Think it would be better to drop setting int_pgm_ilen here completely.
>
> /* Use the (ultimate) callers address to find the insn that trapped. */
> cpu_restore_state(cs, retaddr);
>
Reviewed-by: David Hildenbrand <address@hidden>
--
Thanks,
David