[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 05/10] migration: move global_state.optional
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v5 05/10] migration: move global_state.optional out |
Date: |
Mon, 26 Jun 2017 10:51:19 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Jun 23, 2017 at 07:19:01PM -0300, Eduardo Habkost wrote:
> On Fri, Jun 23, 2017 at 12:46:41PM +0800, Peter Xu wrote:
> > Put it into MigrationState then we can use the properties to specify
> > whether to enable storing global state.
> >
> > Removing global_state_set_optional() since now we can use HW_COMPAT_2_3
> > for x86/power, and AccelClass.global_props for Xen.
> >
> > Reviewed-by: Juan Quintela <address@hidden>
> > Signed-off-by: Peter Xu <address@hidden>
> > ---
> > hw/i386/pc_piix.c | 1 -
> > hw/ppc/spapr.c | 1 -
> > hw/xen/xen-common.c | 11 ++++++++++-
> > include/hw/compat.h | 4 ++++
> > include/migration/global_state.h | 1 -
> > migration/global_state.c | 9 ++-------
> > migration/migration.c | 7 +++++++
> > migration/migration.h | 6 ++++++
> > 8 files changed, 29 insertions(+), 11 deletions(-)
> >
> [...]
> > diff --git a/hw/xen/xen-common.c b/hw/xen/xen-common.c
> > index d3fa705..9163a0a 100644
> > --- a/hw/xen/xen-common.c
> > +++ b/hw/xen/xen-common.c
> > @@ -139,19 +139,28 @@ static int xen_init(MachineState *ms)
> > }
> > qemu_add_vm_change_state_handler(xen_change_state_handler, NULL);
> >
> > - global_state_set_optional();
> > savevm_skip_configuration();
> > savevm_skip_section_footers();
> >
> > return 0;
> > }
> >
> > +GlobalProperty xen_compat_props[] = {
>
> This can be static.
Yep.
>
> > + {
> > + .driver = "migration",
> > + .property = "store-global-state",
> > + .value = "off",
> > + },
> > + { .driver = NULL, .property = NULL, .value = NULL },
>
> Normally this is written as:
>
> { /* end of list */ },
Let me switch.
>
> > +};
> > +
> [...]
>
> I didn't review the rest yet, but it seems to be good.
Thanks!
--
Peter Xu
- Re: [Qemu-devel] [PATCH v5 02/10] accel: introduce AccelClass.global_props, (continued)
- [Qemu-devel] [PATCH v5 05/10] migration: move global_state.optional out, Peter Xu, 2017/06/23
- [Qemu-devel] [PATCH v5 06/10] migration: move only_migratable to MigrationState, Peter Xu, 2017/06/23
- [Qemu-devel] [PATCH v5 07/10] migration: move skip_configuration out, Peter Xu, 2017/06/23
- [Qemu-devel] [PATCH v5 08/10] migration: move skip_section_footers, Peter Xu, 2017/06/23
- [Qemu-devel] [PATCH v5 09/10] migration: merge enforce_config_section somewhat, Peter Xu, 2017/06/23
- [Qemu-devel] [PATCH v5 10/10] migration: hmp: dump globals, Peter Xu, 2017/06/23