[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v3 16/49] target-i386: update fp status fix
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [RFC PATCH v3 16/49] target-i386: update fp status fix |
Date: |
Thu, 31 Jul 2014 17:12:42 +0100 |
Pavel Dovgalyuk writes:
> This patch adds calls to update_fp_status() function from several
> places where FPU state is changed.
>
> Signed-off-by: Pavel Dovgalyuk <address@hidden>
> ---
> target-i386/cpu.c | 1 +
> target-i386/cpu.h | 1 +
> target-i386/fpu_helper.c | 5 ++++-
> target-i386/machine.c | 2 +-
> 4 files changed, 7 insertions(+), 2 deletions(-)
Hmm I wonder if this function should be renamed. It seems to be more
about ensuring the softfloat library is correctly configured according
to the emulated CPU configuration.
>
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index 6d008ab..906d7d7 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -2573,6 +2573,7 @@ static void x86_cpu_reset(CPUState *s)
> env->fptags[i] = 1;
> }
> env->fpuc = 0x37f;
> + update_fp_status(env);
>
> env->mxcsr = 0x1f80;
> env->xstate_bv = XSTATE_FP | XSTATE_SSE;
> diff --git a/target-i386/cpu.h b/target-i386/cpu.h
> index e634d83..42bda46 100644
> --- a/target-i386/cpu.h
> +++ b/target-i386/cpu.h
> @@ -1249,6 +1249,7 @@ void QEMU_NORETURN raise_interrupt(CPUX86State *nenv,
> int intno, int is_int,
> /* cc_helper.c */
> extern const uint8_t parity_table[256];
> uint32_t cpu_cc_compute_all(CPUX86State *env1, int op);
> +void update_fp_status(CPUX86State *env);
>
> static inline uint32_t cpu_compute_eflags(CPUX86State *env)
> {
> diff --git a/target-i386/fpu_helper.c b/target-i386/fpu_helper.c
> index 1b2900d..a8ffba9 100644
> --- a/target-i386/fpu_helper.c
> +++ b/target-i386/fpu_helper.c
> @@ -537,7 +537,7 @@ uint32_t helper_fnstcw(CPUX86State *env)
> return env->fpuc;
> }
>
> -static void update_fp_status(CPUX86State *env)
> +void update_fp_status(CPUX86State *env)
> {
> int rnd_type;
>
> @@ -1006,6 +1006,7 @@ void helper_fstenv(CPUX86State *env, target_ulong ptr,
> int data32)
> cpu_stw_data(env, ptr + 10, 0);
> cpu_stw_data(env, ptr + 12, 0);
> }
> + update_fp_status(env);
> }
>
> void helper_fldenv(CPUX86State *env, target_ulong ptr, int data32)
> @@ -1055,6 +1056,7 @@ void helper_fsave(CPUX86State *env, target_ulong ptr,
> int data32)
> env->fptags[5] = 1;
> env->fptags[6] = 1;
> env->fptags[7] = 1;
> + update_fp_status(env);
> }
>
> void helper_frstor(CPUX86State *env, target_ulong ptr, int data32)
> @@ -1158,6 +1160,7 @@ void helper_fxrstor(CPUX86State *env, target_ulong ptr,
> int data64)
> }
>
> env->fpuc = cpu_lduw_data(env, ptr);
> + update_fp_status(env);
> fpus = cpu_lduw_data(env, ptr + 2);
> fptag = cpu_lduw_data(env, ptr + 4);
> env->fpstt = (fpus >> 11) & 7;
> diff --git a/target-i386/machine.c b/target-i386/machine.c
> index 16d2f6a..2922c97 100644
> --- a/target-i386/machine.c
> +++ b/target-i386/machine.c
> @@ -315,13 +315,13 @@ static int cpu_post_load(void *opaque, int version_id)
> env->hflags &= ~HF_CPL_MASK;
> env->hflags |= (env->segs[R_SS].flags >> DESC_DPL_SHIFT) & HF_CPL_MASK;
>
> - /* XXX: restore FPU round state */
> env->fpstt = (env->fpus_vmstate >> 11) & 7;
> env->fpus = env->fpus_vmstate & ~0x3800;
> env->fptag_vmstate ^= 0xff;
> for(i = 0; i < 8; i++) {
> env->fptags[i] = (env->fptag_vmstate >> i) & 1;
> }
> + update_fp_status(env);
>
> cpu_breakpoint_remove_all(cs, BP_CPU);
> cpu_watchpoint_remove_all(cs, BP_CPU);
--
Alex Bennée
- [Qemu-devel] [RFC PATCH v3 10/49] rtl8139: adding new fields to vmstate, (continued)
- [Qemu-devel] [RFC PATCH v3 10/49] rtl8139: adding new fields to vmstate, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 11/49] piix: do not raise irq while loading vmstate, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 12/49] mc146818rtc: add missed field to vmstate, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 13/49] pl031: add missed field to vmstate, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 14/49] ide pci: reset status field before loading the vmstate, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 15/49] softmmu: fixing usage of cpu_st/ld* from helpers, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 16/49] target-i386: update fp status fix, Pavel Dovgalyuk, 2014/07/31
- Re: [Qemu-devel] [RFC PATCH v3 16/49] target-i386: update fp status fix,
Alex Bennée <=
- [Qemu-devel] [RFC PATCH v3 17/49] migration: add vmstate for int8 and char arrays, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 18/49] replay: global variables and function stubs, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 19/49] block: add suffix parameter to bdrv_open functions, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 20/49] sysemu: system functions for replay, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 21/49] replay: internal functions for replay log, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 22/49] cpu: invent instruction count for accurate replay, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 23/49] target-arm: instructions counting code for replay, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 24/49] target-i386: instructions counting code for replay, Pavel Dovgalyuk, 2014/07/31
- [Qemu-devel] [RFC PATCH v3 25/49] replay: interrupts and exceptions, Pavel Dovgalyuk, 2014/07/31