Re: [Qemu-devel] [PATCH qom-cpu v3 3/6] cpu: Register VMStateDescription through CPUState

[Eduardo Habkost]

On Mon, Feb 25, 2013 at 07:22:47PM +0100, Andreas Färber wrote:
> In comparison to DeviceClass::vmsd, CPU VMState is split in two,
> "cpu_common" and "cpu", and uses cpu_index as instance_id instead of -1.
> Therefore add a CPU-specific CPUClass::vmsd field.
> 
> Unlike the legacy CPUArchState registration, rather register CPUState.
> 
> Signed-off-by: Juan Quintela <address@hidden>
> Signed-off-by: Andreas Färber <address@hidden>

Reviewed-by: Eduardo Habkost <address@hidden>


> ---
>  exec.c            |   11 +++++++++--
>  include/qom/cpu.h |    3 +++
>  2 Dateien geändert, 12 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-)
> 
> diff --git a/exec.c b/exec.c
> index a41bcb8..84e43bd 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -219,7 +219,7 @@ void cpu_exec_init_all(void)
>  #endif
>  }
>  
> -#if defined(CPU_SAVE_VERSION) && !defined(CONFIG_USER_ONLY)
> +#if !defined(CONFIG_USER_ONLY)
>  
>  static int cpu_common_post_load(void *opaque, int version_id)
>  {
> @@ -245,6 +245,8 @@ static const VMStateDescription vmstate_cpu_common = {
>          VMSTATE_END_OF_LIST()
>      }
>  };
> +#else
> +#define vmstate_cpu_common vmstate_dummy
>  #endif
>  
>  CPUState *qemu_get_cpu(int index)
> @@ -266,6 +268,7 @@ CPUState *qemu_get_cpu(int index)
>  void cpu_exec_init(CPUArchState *env)
>  {
>      CPUState *cpu = ENV_GET_CPU(env);
> +    CPUClass *cc = CPU_GET_CLASS(cpu);
>      CPUArchState **penv;
>      int cpu_index;
>  
> @@ -290,11 +293,15 @@ void cpu_exec_init(CPUArchState *env)
>  #if defined(CONFIG_USER_ONLY)
>      cpu_list_unlock();
>  #endif
> -#if defined(CPU_SAVE_VERSION) && !defined(CONFIG_USER_ONLY)
>      vmstate_register(NULL, cpu_index, &vmstate_cpu_common, env);
> +#if defined(CPU_SAVE_VERSION) && !defined(CONFIG_USER_ONLY)
>      register_savevm(NULL, "cpu", cpu_index, CPU_SAVE_VERSION,
>                      cpu_save, cpu_load, env);
> +    assert(cc->vmsd == NULL);
>  #endif
> +    if (cc->vmsd != NULL) {
> +        vmstate_register(NULL, cpu_index, cc->vmsd, cpu);
> +    }
>  }
>  
>  #if defined(TARGET_HAS_ICE)
> diff --git a/include/qom/cpu.h b/include/qom/cpu.h
> index ee1a7c8..1106e39 100644
> --- a/include/qom/cpu.h
> +++ b/include/qom/cpu.h
> @@ -44,6 +44,7 @@ typedef struct CPUState CPUState;
>   * @class_by_name: Callback to map -cpu command line model name to an
>   * instantiatable CPU type.
>   * @reset: Callback to reset the #CPUState to its initial state.
> + * @vmsd: State description for migration.
>   *
>   * Represents a CPU family or model.
>   */
> @@ -55,6 +56,8 @@ typedef struct CPUClass {
>      ObjectClass *(*class_by_name)(const char *cpu_model);
>  
>      void (*reset)(CPUState *cpu);
> +
> +    const struct VMStateDescription *vmsd;
>  } CPUClass;
>  
>  struct KVMState;
> -- 
> 1.7.10.4
> 

-- 
Eduardo