]> git.proxmox.com Git - mirror_qemu.git/commitdiff
mips_fulong2e: Don't register "cpu" VMState twice
authorJuan Quintela <quintela@redhat.com>
Sun, 13 May 2012 17:32:54 +0000 (19:32 +0200)
committerAndreas Färber <afaerber@suse.de>
Sun, 13 May 2012 17:58:23 +0000 (19:58 +0200)
We have the following simplified callgraph in mips_fulong2e_init():

  cpu_init() => cpu_mips_init()
    object_new()
      mips_cpu_initfn()
        cpu_exec_init()
          register_savevm(NULL, "cpu", cpu_index, CPU_SAVE_VERSION,
                          cpu_save, cpu_load, env)
  register_savevm(NULL, "cpu", 0, 3, cpu_save, cpu_load, env)

CPU_SAVE_VERSION is defined as 3 in target-mips/cpu.h.
fulong2e instantiates one CPU, so its cpu_index is 0.
Thus the two are fully identical.

Therefore just remove the second call in fulong2e.

Signed-off-by: Juan Quintela <quintela@redhat.com>
[AF: Extend explanation in commit message]
Signed-off-by: Andreas Färber <afaerber@suse.de>
hw/mips_fulong2e.c

index 37dc711e08e1f3370b68d2d7b7f7fe4268a84c2c..1a8df10429a268cb31c69d0e72291d059050e8b1 100644 (file)
@@ -284,7 +284,6 @@ static void mips_fulong2e_init(ram_addr_t ram_size, const char *boot_device,
         exit(1);
     }
 
-    register_savevm(NULL, "cpu", 0, 3, cpu_save, cpu_load, env);
     qemu_register_reset(main_cpu_reset, env);
 
     /* fulong 2e has 256M ram. */