[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v14 03/34] target/i386: Use generic DISAS_* enum
From: |
Lluís Vilanova |
Subject: |
Re: [Qemu-devel] [PATCH v14 03/34] target/i386: Use generic DISAS_* enumerators |
Date: |
Sat, 22 Jul 2017 01:35:26 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Richard Henderson writes:
> This target is not sophisticated in its use of cleanups at the
> end of the translation loop. For the most part, any condition
> that exits the TB is dealt with by emitting the exiting opcode
> right then and there. Therefore the only is_jmp indicator that
> is needed is DISAS_NORETURN.
> For two stack segment modifying cases, we have not yet exited
> the TB (therefore DISAS_NORETURN feels wrong), but intend to exit.
> The caller of gen_movl_seg_T0 currently checks for any non-zero
> value, therefore DISAS_TOO_MANY seems acceptable for that usage.
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> target/i386/translate.c | 28 +++++++++++++++-------------
> 1 file changed, 15 insertions(+), 13 deletions(-)
> diff --git a/target/i386/translate.c b/target/i386/translate.c
> index cab9e32..3ffbf1b 100644
> --- a/target/i386/translate.c
> +++ b/target/i386/translate.c
> @@ -31,6 +31,7 @@
> #include "trace-tcg.h"
> #include "exec/log.h"
> +#define DISAS_TOO_MANY 5
Why is this one not added as a generic define too (like DISAS_NORETURN in prev
patch)?
Thanks,
Lluis
- [Qemu-devel] [PATCH v14 00/34] Generic translation framework, Richard Henderson, 2017/07/15
- [Qemu-devel] [PATCH v14 02/34] tcg: Add generic DISAS_NORETURN, Richard Henderson, 2017/07/15
- [Qemu-devel] [PATCH v14 01/34] Pass generic CPUState to gen_intermediate_code(), Richard Henderson, 2017/07/15
- [Qemu-devel] [PATCH v14 03/34] target/i386: Use generic DISAS_* enumerators, Richard Henderson, 2017/07/15
- [Qemu-devel] [PATCH v14 04/34] target/arm: Use DISAS_NORETURN, Richard Henderson, 2017/07/15
- [Qemu-devel] [PATCH v14 06/34] target/arm: Delay check for magic kernel page, Richard Henderson, 2017/07/15
- [Qemu-devel] [PATCH v14 07/34] target/arm: Set is_jmp properly after single-stepping, Richard Henderson, 2017/07/15