diff mbox

[v3,09/15] target-mips: update cpu_save/cpu_load to support new registers

Message ID 1414154549-2102-10-git-send-email-leon.alrae@imgtec.com
State New
Headers show

Commit Message

Leon Alrae Oct. 24, 2014, 12:42 p.m. UTC
Signed-off-by: Leon Alrae <leon.alrae@imgtec.com>
---
 target-mips/cpu.h     |  2 +-
 target-mips/machine.c | 26 +++++++++++++++++++++++++-
 2 files changed, 26 insertions(+), 2 deletions(-)

Comments

Yongbok Kim Oct. 29, 2014, 2:02 p.m. UTC | #1
On 24/10/2014 13:42, Leon Alrae wrote:
> Signed-off-by: Leon Alrae <leon.alrae@imgtec.com>
> ---
>   target-mips/cpu.h     |  2 +-
>   target-mips/machine.c | 26 +++++++++++++++++++++++++-
>   2 files changed, 26 insertions(+), 2 deletions(-)
>
> diff --git a/target-mips/cpu.h b/target-mips/cpu.h
> index 4687f4f..c66a725 100644
> --- a/target-mips/cpu.h
> +++ b/target-mips/cpu.h
> @@ -558,7 +558,7 @@ void mips_cpu_list (FILE *f, fprintf_function cpu_fprintf);
>   extern void cpu_wrdsp(uint32_t rs, uint32_t mask_num, CPUMIPSState *env);
>   extern uint32_t cpu_rddsp(uint32_t mask_num, CPUMIPSState *env);
>   
> -#define CPU_SAVE_VERSION 4
> +#define CPU_SAVE_VERSION 5
>   
>   /* MMU modes definitions. We carefully match the indices with our
>      hflags layout. */
> diff --git a/target-mips/machine.c b/target-mips/machine.c
> index 0496faa..0ba7d73 100644
> --- a/target-mips/machine.c
> +++ b/target-mips/machine.c
> @@ -61,7 +61,12 @@ void cpu_save(QEMUFile *f, void *opaque)
>       qemu_put_be32s(f, &env->tlb->nb_tlb);
>       qemu_put_be32s(f, &env->tlb->tlb_in_use);
>       for(i = 0; i < MIPS_TLB_MAX; i++) {
> -        uint16_t flags = ((env->tlb->mmu.r4k.tlb[i].G << 10) |
> +        uint16_t flags = ((env->tlb->mmu.r4k.tlb[i].EHINV << 15) |
> +                          (env->tlb->mmu.r4k.tlb[i].RI1 << 14) |
> +                          (env->tlb->mmu.r4k.tlb[i].RI0 << 13) |
> +                          (env->tlb->mmu.r4k.tlb[i].XI1 << 12) |
> +                          (env->tlb->mmu.r4k.tlb[i].XI0 << 11) |
> +                          (env->tlb->mmu.r4k.tlb[i].G << 10) |
>                             (env->tlb->mmu.r4k.tlb[i].C0 << 7) |
>                             (env->tlb->mmu.r4k.tlb[i].C1 << 4) |
>                             (env->tlb->mmu.r4k.tlb[i].V0 << 3) |
> @@ -111,6 +116,8 @@ void cpu_save(QEMUFile *f, void *opaque)
>       qemu_put_sbe32s(f, &env->CP0_SRSConf4);
>       qemu_put_sbe32s(f, &env->CP0_HWREna);
>       qemu_put_betls(f, &env->CP0_BadVAddr);
> +    qemu_put_be32s(f, &env->CP0_BadInstr);
> +    qemu_put_be32s(f, &env->CP0_BadInstrP);
>       qemu_put_sbe32s(f, &env->CP0_Count);
>       qemu_put_betls(f, &env->CP0_EntryHi);
>       qemu_put_sbe32s(f, &env->CP0_Compare);
> @@ -144,6 +151,9 @@ void cpu_save(QEMUFile *f, void *opaque)
>       qemu_put_sbe32s(f, &env->CP0_DataHi);
>       qemu_put_betls(f, &env->CP0_ErrorEPC);
>       qemu_put_sbe32s(f, &env->CP0_DESAVE);
> +    for (i = 0; i < MIPS_KSCRATCH_NUM; i++) {
> +        qemu_put_betls(f, &env->CP0_KScratch[i]);
> +    }
>   
>       /* Save inactive TC state */
>       for (i = 0; i < MIPS_SHADOW_SET_MAX; i++)
> @@ -232,6 +242,13 @@ int cpu_load(QEMUFile *f, void *opaque, int version_id)
>           env->tlb->mmu.r4k.tlb[i].V1 = (flags >> 2) & 1;
>           env->tlb->mmu.r4k.tlb[i].D0 = (flags >> 1) & 1;
>           env->tlb->mmu.r4k.tlb[i].D1 = (flags >> 0) & 1;
> +        if (version_id >= 5) {
> +            env->tlb->mmu.r4k.tlb[i].EHINV = (flags >> 15) & 1;
> +            env->tlb->mmu.r4k.tlb[i].RI1 = (flags >> 14) & 1;
> +            env->tlb->mmu.r4k.tlb[i].RI0 = (flags >> 13) & 1;
> +            env->tlb->mmu.r4k.tlb[i].XI1 = (flags >> 12) & 1;
> +            env->tlb->mmu.r4k.tlb[i].XI0 = (flags >> 11) & 1;
> +        }
>           qemu_get_betls(f, &env->tlb->mmu.r4k.tlb[i].PFN[0]);
>           qemu_get_betls(f, &env->tlb->mmu.r4k.tlb[i].PFN[1]);
>       }
> @@ -301,6 +318,13 @@ int cpu_load(QEMUFile *f, void *opaque, int version_id)
>       qemu_get_sbe32s(f, &env->CP0_DataHi);
>       qemu_get_betls(f, &env->CP0_ErrorEPC);
>       qemu_get_sbe32s(f, &env->CP0_DESAVE);
> +    if (version_id >= 5) {
> +        qemu_get_be32s(f, &env->CP0_BadInstr);
> +        qemu_get_be32s(f, &env->CP0_BadInstrP);
> +        for (i = 0; i < MIPS_KSCRATCH_NUM; i++) {
> +            qemu_get_betls(f, &env->CP0_KScratch[i]);
> +        }
> +    }
>   
>       /* Load inactive TC state */
>       for (i = 0; i < MIPS_SHADOW_SET_MAX; i++) {

Reviewed-by: Yongbok Kim <yongbok.kim@imgtec.com>

Regards,
Yongbok
diff mbox

Patch

diff --git a/target-mips/cpu.h b/target-mips/cpu.h
index 4687f4f..c66a725 100644
--- a/target-mips/cpu.h
+++ b/target-mips/cpu.h
@@ -558,7 +558,7 @@  void mips_cpu_list (FILE *f, fprintf_function cpu_fprintf);
 extern void cpu_wrdsp(uint32_t rs, uint32_t mask_num, CPUMIPSState *env);
 extern uint32_t cpu_rddsp(uint32_t mask_num, CPUMIPSState *env);
 
-#define CPU_SAVE_VERSION 4
+#define CPU_SAVE_VERSION 5
 
 /* MMU modes definitions. We carefully match the indices with our
    hflags layout. */
diff --git a/target-mips/machine.c b/target-mips/machine.c
index 0496faa..0ba7d73 100644
--- a/target-mips/machine.c
+++ b/target-mips/machine.c
@@ -61,7 +61,12 @@  void cpu_save(QEMUFile *f, void *opaque)
     qemu_put_be32s(f, &env->tlb->nb_tlb);
     qemu_put_be32s(f, &env->tlb->tlb_in_use);
     for(i = 0; i < MIPS_TLB_MAX; i++) {
-        uint16_t flags = ((env->tlb->mmu.r4k.tlb[i].G << 10) |
+        uint16_t flags = ((env->tlb->mmu.r4k.tlb[i].EHINV << 15) |
+                          (env->tlb->mmu.r4k.tlb[i].RI1 << 14) |
+                          (env->tlb->mmu.r4k.tlb[i].RI0 << 13) |
+                          (env->tlb->mmu.r4k.tlb[i].XI1 << 12) |
+                          (env->tlb->mmu.r4k.tlb[i].XI0 << 11) |
+                          (env->tlb->mmu.r4k.tlb[i].G << 10) |
                           (env->tlb->mmu.r4k.tlb[i].C0 << 7) |
                           (env->tlb->mmu.r4k.tlb[i].C1 << 4) |
                           (env->tlb->mmu.r4k.tlb[i].V0 << 3) |
@@ -111,6 +116,8 @@  void cpu_save(QEMUFile *f, void *opaque)
     qemu_put_sbe32s(f, &env->CP0_SRSConf4);
     qemu_put_sbe32s(f, &env->CP0_HWREna);
     qemu_put_betls(f, &env->CP0_BadVAddr);
+    qemu_put_be32s(f, &env->CP0_BadInstr);
+    qemu_put_be32s(f, &env->CP0_BadInstrP);
     qemu_put_sbe32s(f, &env->CP0_Count);
     qemu_put_betls(f, &env->CP0_EntryHi);
     qemu_put_sbe32s(f, &env->CP0_Compare);
@@ -144,6 +151,9 @@  void cpu_save(QEMUFile *f, void *opaque)
     qemu_put_sbe32s(f, &env->CP0_DataHi);
     qemu_put_betls(f, &env->CP0_ErrorEPC);
     qemu_put_sbe32s(f, &env->CP0_DESAVE);
+    for (i = 0; i < MIPS_KSCRATCH_NUM; i++) {
+        qemu_put_betls(f, &env->CP0_KScratch[i]);
+    }
 
     /* Save inactive TC state */
     for (i = 0; i < MIPS_SHADOW_SET_MAX; i++)
@@ -232,6 +242,13 @@  int cpu_load(QEMUFile *f, void *opaque, int version_id)
         env->tlb->mmu.r4k.tlb[i].V1 = (flags >> 2) & 1;
         env->tlb->mmu.r4k.tlb[i].D0 = (flags >> 1) & 1;
         env->tlb->mmu.r4k.tlb[i].D1 = (flags >> 0) & 1;
+        if (version_id >= 5) {
+            env->tlb->mmu.r4k.tlb[i].EHINV = (flags >> 15) & 1;
+            env->tlb->mmu.r4k.tlb[i].RI1 = (flags >> 14) & 1;
+            env->tlb->mmu.r4k.tlb[i].RI0 = (flags >> 13) & 1;
+            env->tlb->mmu.r4k.tlb[i].XI1 = (flags >> 12) & 1;
+            env->tlb->mmu.r4k.tlb[i].XI0 = (flags >> 11) & 1;
+        }
         qemu_get_betls(f, &env->tlb->mmu.r4k.tlb[i].PFN[0]);
         qemu_get_betls(f, &env->tlb->mmu.r4k.tlb[i].PFN[1]);
     }
@@ -301,6 +318,13 @@  int cpu_load(QEMUFile *f, void *opaque, int version_id)
     qemu_get_sbe32s(f, &env->CP0_DataHi);
     qemu_get_betls(f, &env->CP0_ErrorEPC);
     qemu_get_sbe32s(f, &env->CP0_DESAVE);
+    if (version_id >= 5) {
+        qemu_get_be32s(f, &env->CP0_BadInstr);
+        qemu_get_be32s(f, &env->CP0_BadInstrP);
+        for (i = 0; i < MIPS_KSCRATCH_NUM; i++) {
+            qemu_get_betls(f, &env->CP0_KScratch[i]);
+        }
+    }
 
     /* Load inactive TC state */
     for (i = 0; i < MIPS_SHADOW_SET_MAX; i++) {