[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC qom-cpu v2 6/8] target-alpha: Register VMStateDesc
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [RFC qom-cpu v2 6/8] target-alpha: Register VMStateDescription for AlphaCPU |
Date: |
Fri, 22 Feb 2013 10:53:00 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Feb 22, 2013 at 02:22:43PM +0100, Juan Quintela wrote:
> Andreas Färber <address@hidden> wrote:
> > Commit b758aca1f6cdb175634812b79f5560c36c902d00 (target-alpha: Enable
> > the alpha-softmmu target.) introduced cpu_{save,load}() functions but
> > didn't define CPU_SAVE_VERSION, so they were never registered.
> >
> > Drop cpu_{save,load}() and register the VMStateDescription via CPUClass.
> > This operates on the AlphaCPU object instead of CPUAlphaState.
> >
> > Signed-off-by: Andreas Färber <address@hidden>
>
>
> Seeing that we are repeating the code all around. Could we change this
> to something like:
>
> >
> > #define ENV_GET_CPU(e) CPU(alpha_env_get_cpu(e))
> >
> > +#ifdef CONFIG_USER_ONLY
> > +#define vmstate_alpha_cpu vmstate_dummy
> > +#else
> > +extern const struct VMStateDescription vmstate_alpha_cpu;
> > +#endif
> > +
>
> Change this to:
>
> #ifdef CONFIG_USER_ONLY
> #define vmstate_register_cpu(unused, unused)
> #else
> #define vmstate_register_cpu(env, vmstate_cpu) (env->vmsd = vmstate_cpu)
> #endif
I like this approach. But using a macro is going to cause unexpected
"variable is unused" gcc warnings. Can we make it a static inline
function instead?
--
Eduardo
- [Qemu-devel] [PATCH RFC qom-cpu v2 0/8] QOM CPUState VMStateDescriptions, Andreas Färber, 2013/02/18
- [Qemu-devel] [PATCH qom-cpu v2 5/8] target-lm32: Update VMStateDescription to LM32CPU, Andreas Färber, 2013/02/18
- [Qemu-devel] [RFC qom-cpu v2 6/8] target-alpha: Register VMStateDescription for AlphaCPU, Andreas Färber, 2013/02/18
- Re: [Qemu-devel] [RFC qom-cpu v2 6/8] target-alpha: Register VMStateDescription for AlphaCPU, Andreas Färber, 2013/02/22
- Re: [Qemu-devel] [RFC qom-cpu v2 6/8] target-alpha: Register VMStateDescription for AlphaCPU, Juan Quintela, 2013/02/22
[Qemu-devel] [RFC qom-cpu v2 8/8] cpu: Guard cpu_{save, load}() definitions, Andreas Färber, 2013/02/18
[Qemu-devel] [PATCH qom-cpu v2 4/8] target-i386: Update VMStateDescription to X86CPU, Andreas Färber, 2013/02/18
[Qemu-devel] [RFC qom-cpu v2 7/8] target-openrisc: Register VMStateDescription for OpenRISCCPU, Andreas Färber, 2013/02/18