[Qemu-devel] [PATCH v5 10/10] target-avr: saving sreg, rampD, rampX, rampY, rampD, eind in HW representation saving cpu features

[Michael Rolnik <mrolnik@gmail.com>]

From: Michael Rolnik <mrolnik@gmail.com>

From: Michael Rolnik <rolnik@amazon.com>

Signed-off-by: Michael Rolnik <mrolnik@gmail.com>
---
 target-avr/machine.c | 105 ++++++++++++++++++++++++++++++++++++++++-----------
 1 file changed, 84 insertions(+), 21 deletions(-)

diff --git a/target-avr/machine.c b/target-avr/machine.c
index 39f1ee6..9659c04 100644
--- a/target-avr/machine.c
+++ b/target-avr/machine.c
@@ -23,31 +23,94 @@
 #include "cpu.h"
 #include "hw/boards.h"
 #include "machine.h"
+#include "migration/qemu-file.h"
+
+static int get_sreg(QEMUFile *f, void *opaque, size_t size)
+{
+    CPUAVRState* env = opaque;
+    uint8_t sreg;
+    
+    qemu_get_8s(f, &sreg);
+    cpu_set_sreg(env, sreg);
+    return 0;
+}
+
+static void put_sreg(QEMUFile *f, void *opaque, size_t size)
+{
+    CPUAVRState* env = opaque;
+    uint8_t sreg = cpu_get_sreg(env);
+
+    qemu_put_8s(f, &sreg);
+}
+
+static const VMStateInfo vmstate_sreg = {
+    .name = "sreg",
+    .get  = get_sreg,
+    .put  = put_sreg,
+};
+
+static int get_segment(QEMUFile *f, void *opaque, size_t size)
+{
+    uint32_t *ramp = opaque;
+    uint8_t temp = *ramp >> 16;
+    
+    qemu_get_8s(f, &temp);
+    return 0;
+}
+
+static void put_segment(QEMUFile *f, void *opaque, size_t size)
+{
+    uint32_t *ramp = opaque;
+    uint8_t temp;
+
+    qemu_put_8s(f, &temp);
+    *ramp = ((uint32_t)temp) << 16;
+}
+
+static const VMStateInfo vmstate_rampD = {
+    .name = "rampD",
+    .get  = get_segment,
+    .put  = put_segment,
+};
+static const VMStateInfo vmstate_rampX = {
+    .name = "rampX",
+    .get  = get_segment,
+    .put  = put_segment,
+};
+static const VMStateInfo vmstate_rampY = {
+    .name = "rampY",
+    .get  = get_segment,
+    .put  = put_segment,
+};
+static const VMStateInfo vmstate_rampZ = {
+    .name = "rampZ",
+    .get  = get_segment,
+    .put  = put_segment,
+};
+static const VMStateInfo vmstate_eind = {
+    .name = "eind",
+    .get  = get_segment,
+    .put  = put_segment,
+};
 
 const VMStateDescription vmstate_avr_cpu = {
     .name               = "cpu",
-    .version_id         = 1,
-    .minimum_version_id = 1,
+    .version_id         = 0,
+    .minimum_version_id = 0,
     .fields = (VMStateField[]) {
-        VMSTATE_UINT32_ARRAY(r, CPUAVRState, 32),
-
-        VMSTATE_UINT32(sregC, CPUAVRState),
-        VMSTATE_UINT32(sregZ, CPUAVRState),
-        VMSTATE_UINT32(sregN, CPUAVRState),
-        VMSTATE_UINT32(sregV, CPUAVRState),
-        VMSTATE_UINT32(sregS, CPUAVRState),
-        VMSTATE_UINT32(sregH, CPUAVRState),
-        VMSTATE_UINT32(sregT, CPUAVRState),
-        VMSTATE_UINT32(sregI, CPUAVRState),
-
-        VMSTATE_UINT32(rampD, CPUAVRState),
-        VMSTATE_UINT32(rampX, CPUAVRState),
-        VMSTATE_UINT32(rampY, CPUAVRState),
-        VMSTATE_UINT32(rampZ, CPUAVRState),
-
-        VMSTATE_UINT32(eind, CPUAVRState),
-        VMSTATE_UINT32(sp, CPUAVRState),
-        VMSTATE_UINT32(pc_w, CPUAVRState),
+        VMSTATE_UINT32(env.features, AVRCPU),
+        VMSTATE_UINT32(env.pc_w, AVRCPU),
+        VMSTATE_UINT32(env.sp, AVRCPU),
+
+        VMSTATE_UINT32_ARRAY(env.r, AVRCPU, 32),
+        VMSTATE_UINT32_ARRAY(env.io, AVRCPU, 64),
+
+        VMSTATE_SINGLE_TEST(env, AVRCPU, NULL, 0, vmstate_sreg, CPUAVRState),
+        VMSTATE_SINGLE_TEST(env.rampD, AVRCPU, NULL, 0, vmstate_rampD, uint32_t),
+        VMSTATE_SINGLE_TEST(env.rampX, AVRCPU, NULL, 0, vmstate_rampX, uint32_t),
+        VMSTATE_SINGLE_TEST(env.rampY, AVRCPU, NULL, 0, vmstate_rampY, uint32_t),
+        VMSTATE_SINGLE_TEST(env.rampZ, AVRCPU, NULL, 0, vmstate_rampZ, uint32_t),
+        VMSTATE_SINGLE_TEST(env.eind, AVRCPU, NULL, 0, vmstate_eind, uint32_t),
 
         VMSTATE_END_OF_LIST()
     }
-- 
2.4.9 (Apple Git-60)