On Mon, Mar 19, 2012 at 11:57:29PM +0100, Juan Quintela wrote: > Some cpu's definitions define CPU_SAVE_VERSION, others not, but they have > defined cpu_save/load.
I'm guessing this patch becomes useful later in the series, but as it stands I'm not seeing where this helps with cpus like target-alpha that have cpu_save/load defined but not CPU_SAVE_VERSION, except to save off common cpu fields as opposed to before when we saved nothing. Maybe a add CPU_SAVE_SUPPORTED and check that instead? Or merge this elsewhere in the series? > > Signed-off-by: Juan Quintela <quint...@redhat.com> > --- > exec.c | 6 ++++-- > 1 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/exec.c b/exec.c > index 265e895..9389a61 100644 > --- a/exec.c > +++ b/exec.c > @@ -657,7 +657,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) > { > @@ -724,11 +724,13 @@ void cpu_exec_init(CPUArchState *env) > #if defined(CONFIG_USER_ONLY) > cpu_list_unlock(); > #endif > -#if defined(CPU_SAVE_VERSION) && !defined(CONFIG_USER_ONLY) > +#if !defined(CONFIG_USER_ONLY) > vmstate_register(NULL, cpu_index, &vmstate_cpu_common, env); > +#if defined(CPU_SAVE_VERSION) > register_savevm(NULL, "cpu", cpu_index, CPU_SAVE_VERSION, > cpu_save, cpu_load, env); > #endif > +#endif > } > > /* Allocate a new translation block. Flush the translation buffer if > -- > 1.7.7.6 > >