[Qemu-devel] [PATCH 33/49] x86: add fptag_vmstate to the state

[Juan Quintela]

It is needed to store fptags

Signed-off-by: Juan Quintela <address@hidden>
---
 target-i386/cpu.h     |    1 +
 target-i386/machine.c |   18 +++++++++---------
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/target-i386/cpu.h b/target-i386/cpu.h
index d5c64e6..18fadf0 100644
--- a/target-i386/cpu.h
+++ b/target-i386/cpu.h
@@ -594,6 +594,7 @@ typedef struct CPUX86State {
     unsigned int fpstt; /* top of stack index */
     uint16_t fpus;
     uint16_t fpus_vmstate;
+    uint16_t fptag_vmstate;
     uint16_t fpuc;
     uint8_t fptags[8];   /* 0 = valid, 1 = empty */
     union {
diff --git a/target-i386/machine.c b/target-i386/machine.c
index 7da5d6a..6309ea7 100644
--- a/target-i386/machine.c
+++ b/target-i386/machine.c
@@ -26,7 +26,7 @@ static void cpu_get_seg(QEMUFile *f, SegmentCache *dt)
 void cpu_save(QEMUFile *f, void *opaque)
 {
     CPUState *env = opaque;
-    uint16_t fptag, fpregs_format;
+    uint16_t fpregs_format;
     int32_t pending_irq;
     int i, bit;

@@ -40,14 +40,14 @@ void cpu_save(QEMUFile *f, void *opaque)

     /* FPU */
     env->fpus_vmstate = (env->fpus & ~0x3800) | (env->fpstt & 0x7) << 11;
-    fptag = 0;
+    env->fptag_vmstate = 0;
     for(i = 0; i < 8; i++) {
-        fptag |= ((!env->fptags[i]) << i);
+        env->fptag_vmstate |= ((!env->fptags[i]) << i);
     }

     qemu_put_be16s(f, &env->fpuc);
     qemu_put_be16s(f, &env->fpus_vmstate);
-    qemu_put_be16s(f, &fptag);
+    qemu_put_be16s(f, &env->fptag_vmstate);

 #ifdef USE_X86LDOUBLE
     fpregs_format = 0;
@@ -196,7 +196,7 @@ int cpu_load(QEMUFile *f, void *opaque, int version_id)
 {
     CPUState *env = opaque;
     int i, guess_mmx;
-    uint16_t fptag, fpregs_format;
+    uint16_t fpregs_format;
     int32_t pending_irq;

     cpu_synchronize_state(env);
@@ -210,13 +210,13 @@ int cpu_load(QEMUFile *f, void *opaque, int version_id)

     qemu_get_be16s(f, &env->fpuc);
     qemu_get_be16s(f, &env->fpus_vmstate);
-    qemu_get_be16s(f, &fptag);
+    qemu_get_be16s(f, &env->fptag_vmstate);
     qemu_get_be16s(f, &fpregs_format);

     /* NOTE: we cannot always restore the FPU state if the image come
        from a host with a different 'USE_X86LDOUBLE' define. We guess
        if we are in an MMX state to restore correctly in that case. */
-    guess_mmx = ((fptag == 0xff) && (env->fpus_vmstate & 0x3800) == 0);
+    guess_mmx = ((env->fptag_vmstate == 0xff) && (env->fpus_vmstate & 0x3800) 
== 0);
     for(i = 0; i < 8; i++) {
         uint64_t mant;
         uint16_t exp;
@@ -261,9 +261,9 @@ int cpu_load(QEMUFile *f, void *opaque, int version_id)
     /* XXX: restore FPU round state */
     env->fpstt = (env->fpus_vmstate >> 11) & 7;
     env->fpus = env->fpus_vmstate & ~0x3800;
-    fptag ^= 0xff;
+    env->fptag_vmstate ^= 0xff;
     for(i = 0; i < 8; i++) {
-        env->fptags[i] = (fptag >> i) & 1;
+        env->fptags[i] = (env->fptag_vmstate >> i) & 1;
     }

     for(i = 0; i < 6; i++)
-- 
1.6.2.5