commit 60a902f1c329757705dea416abd741c1f11d3186
author Juan Quintela <quintela@redhat.com> Tue Sep 29 22:48:54 2009 +0200
committer Anthony Liguori <aliguori@us.ibm.com> Mon Oct 05 09:32:41 2009 -0500
tree 94365c90f2bf7dee12f6b528e662a1d091348d45
parent bfc179b60cb74cf33d27a61c14c5b41e6695c3ff

x86: add fpregs_format_vmstate

Don't even ask, being able to load/save between 64<->80bit floats should be forbidden

Signed-off-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

target-i386/cpu.h

diff --git a/target-i386/cpu.h b/target-i386/cpu.h
index 731a22a..f069018 100644
--- a/target-i386/cpu.h
+++ b/target-i386/cpu.h
@@ -596,6 +596,7 @@
     uint16_t fpus;
     uint16_t fpus_vmstate;
     uint16_t fptag_vmstate;
+    uint16_t fpregs_format_vmstate;
     uint16_t fpuc;
     uint8_t fptags[8];   /* 0 = valid, 1 = empty */
     union {

target-i386/machine.c

diff --git a/target-i386/machine.c b/target-i386/machine.c
index c6fd5ba..18347e4 100644
--- a/target-i386/machine.c
+++ b/target-i386/machine.c
@@ -26,7 +26,6 @@
 void cpu_save(QEMUFile *f, void *opaque)
 {
     CPUState *env = opaque;
-    uint16_t fpregs_format;
     int i, bit;
 
     cpu_synchronize_state(env);
@@ -49,11 +48,11 @@
     qemu_put_be16s(f, &env->fptag_vmstate);
 
 #ifdef USE_X86LDOUBLE
-    fpregs_format = 0;
+    env->fpregs_format_vmstate = 0;
 #else
-    fpregs_format = 1;
+    env->fpregs_format_vmstate = 1;
 #endif
-    qemu_put_be16s(f, &fpregs_format);
+    qemu_put_be16s(f, &env->fpregs_format_vmstate);
 
     for(i = 0; i < 8; i++) {
 #ifdef USE_X86LDOUBLE
@@ -196,7 +195,6 @@
 {
     CPUState *env = opaque;
     int i, guess_mmx;
-    uint16_t fpregs_format;
 
     cpu_synchronize_state(env);
     if (version_id < 3 || version_id > CPU_SAVE_VERSION)
@@ -210,7 +208,7 @@
     qemu_get_be16s(f, &env->fpuc);
     qemu_get_be16s(f, &env->fpus_vmstate);
     qemu_get_be16s(f, &env->fptag_vmstate);
-    qemu_get_be16s(f, &fpregs_format);
+    qemu_get_be16s(f, &env->fpregs_format_vmstate);
 
     /* NOTE: we cannot always restore the FPU state if the image come
        from a host with a different 'USE_X86LDOUBLE' define. We guess
@@ -220,7 +218,7 @@
         uint64_t mant;
         uint16_t exp;
 
-        switch(fpregs_format) {
+        switch(env->fpregs_format_vmstate) {
         case 0:
             mant = qemu_get_be64(f);
             exp = qemu_get_be16(f);