[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 05/11] target-i386: Remove redundant HF_SOFTM
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v3 05/11] target-i386: Remove redundant HF_SOFTMMU_MASK |
Date: |
Thu, 14 Jul 2016 13:19:10 +0100 |
User-agent: |
mu4e 0.9.17; emacs 25.0.95.9 |
Sergey Fedorov <address@hidden> writes:
> From: Sergey Fedorov <address@hidden>
>
> 'HF_SOFTMMU_MASK' is only set when 'CONFIG_SOFTMMU' is defined. So
> there's no need in this flag: test 'CONFIG_SOFTMMU' instead.
>
> Suggested-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Sergey Fedorov <address@hidden>
> Signed-off-by: Sergey Fedorov <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
> ---
> target-i386/cpu.c | 3 ---
> target-i386/cpu.h | 3 ---
> target-i386/translate.c | 12 ++++--------
> 3 files changed, 4 insertions(+), 14 deletions(-)
>
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index fc209ee1cb8a..6e49e4ca8282 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -2725,9 +2725,6 @@ static void x86_cpu_reset(CPUState *s)
>
> /* init to reset state */
>
> -#ifdef CONFIG_SOFTMMU
> - env->hflags |= HF_SOFTMMU_MASK;
> -#endif
> env->hflags2 |= HF2_GIF_MASK;
>
> cpu_x86_update_cr0(env, 0x60000010);
> diff --git a/target-i386/cpu.h b/target-i386/cpu.h
> index 5c7a2791f34f..a7c752afdad8 100644
> --- a/target-i386/cpu.h
> +++ b/target-i386/cpu.h
> @@ -129,8 +129,6 @@
> positions to ease oring with eflags. */
> /* current cpl */
> #define HF_CPL_SHIFT 0
> -/* true if soft mmu is being used */
> -#define HF_SOFTMMU_SHIFT 2
> /* true if hardware interrupts must be disabled for next instruction */
> #define HF_INHIBIT_IRQ_SHIFT 3
> /* 16 or 32 segments */
> @@ -160,7 +158,6 @@
> #define HF_MPX_IU_SHIFT 26 /* BND registers in-use */
>
> #define HF_CPL_MASK (3 << HF_CPL_SHIFT)
> -#define HF_SOFTMMU_MASK (1 << HF_SOFTMMU_SHIFT)
> #define HF_INHIBIT_IRQ_MASK (1 << HF_INHIBIT_IRQ_SHIFT)
> #define HF_CS32_MASK (1 << HF_CS32_SHIFT)
> #define HF_SS32_MASK (1 << HF_SS32_SHIFT)
> diff --git a/target-i386/translate.c b/target-i386/translate.c
> index 7dea18bd6345..e81fce7bc2b5 100644
> --- a/target-i386/translate.c
> +++ b/target-i386/translate.c
> @@ -8224,9 +8224,9 @@ void gen_intermediate_code(CPUX86State *env,
> TranslationBlock *tb)
> dc->popl_esp_hack = 0;
> /* select memory access functions */
> dc->mem_index = 0;
> - if (flags & HF_SOFTMMU_MASK) {
> - dc->mem_index = cpu_mmu_index(env, false);
> - }
> +#ifdef CONFIG_SOFTMMU
> + dc->mem_index = cpu_mmu_index(env, false);
> +#endif
> dc->cpuid_features = env->features[FEAT_1_EDX];
> dc->cpuid_ext_features = env->features[FEAT_1_ECX];
> dc->cpuid_ext2_features = env->features[FEAT_8000_0001_EDX];
> @@ -8239,11 +8239,7 @@ void gen_intermediate_code(CPUX86State *env,
> TranslationBlock *tb)
> #endif
> dc->flags = flags;
> dc->jmp_opt = !(dc->tf || cs->singlestep_enabled ||
> - (flags & HF_INHIBIT_IRQ_MASK)
> -#ifndef CONFIG_SOFTMMU
> - || (flags & HF_SOFTMMU_MASK)
> -#endif
> - );
> + (flags & HF_INHIBIT_IRQ_MASK));
> /* Do not optimize repz jumps at all in icount mode, because
> rep movsS instructions are execured with different paths
> in !repz_opt and repz_opt modes. The first one was used
--
Alex Bennée
- Re: [Qemu-devel] [PATCH v3 01/11] util/qht: Document memory ordering assumptions, (continued)
[Qemu-devel] [PATCH v3 03/11] tcg: Prepare safe tb_jmp_cache lookup out of tb_lock, Sergey Fedorov, 2016/07/12
[Qemu-devel] [PATCH v3 02/11] cpu-exec: Pass last_tb by value to tb_find_fast(), Sergey Fedorov, 2016/07/12
[Qemu-devel] [PATCH v3 04/11] tcg: Prepare safe access to tb_flushed out of tb_lock, Sergey Fedorov, 2016/07/12
[Qemu-devel] [PATCH v3 05/11] target-i386: Remove redundant HF_SOFTMMU_MASK, Sergey Fedorov, 2016/07/12
- Re: [Qemu-devel] [PATCH v3 05/11] target-i386: Remove redundant HF_SOFTMMU_MASK,
Alex Bennée <=
[Qemu-devel] [PATCH v3 08/11] tcg: set up tb->page_addr before insertion, Sergey Fedorov, 2016/07/12
[Qemu-devel] [PATCH v3 07/11] tcg: Prepare TB invalidation for lockless TB lookup, Sergey Fedorov, 2016/07/12
[Qemu-devel] [PATCH v3 09/11] tcg: cpu-exec: remove tb_lock from the hot-path, Sergey Fedorov, 2016/07/12
[Qemu-devel] [PATCH v3 06/11] tcg: Introduce tb_mark_invalid() and tb_is_invalid(), Sergey Fedorov, 2016/07/12