From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56324) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZAE8W-0007jO-M6 for qemu-devel@nongnu.org; Wed, 01 Jul 2015 05:16:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZAE8R-0003vc-Hh for qemu-devel@nongnu.org; Wed, 01 Jul 2015 05:16:56 -0400 Received: from mx1.redhat.com ([209.132.183.28]:47283) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZAE8R-0003vY-AA for qemu-devel@nongnu.org; Wed, 01 Jul 2015 05:16:51 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id 095F7C794F for ; Wed, 1 Jul 2015 09:16:50 +0000 (UTC) Date: Wed, 1 Jul 2015 10:16:47 +0100 From: "Dr. David Alan Gilbert" Message-ID: <20150701091646.GD2137@work-vm> References: <1435740693-10839-1-git-send-email-quintela@redhat.com> <1435740693-10839-5-git-send-email-quintela@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1435740693-10839-5-git-send-email-quintela@redhat.com> Subject: Re: [Qemu-devel] [PATCH 04/12] global_state: Make section optional List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Juan Quintela Cc: amit.shah@redhat.com, qemu-devel@nongnu.org * Juan Quintela (quintela@redhat.com) wrote: > This section would be sent: > > a- for all new machine types > b- for old achine types if section state is different form {running,paused} > that were the only giving us troubles. > > So, in new qemus: it is alwasy there. In old qemus: they are only > there if it an error has happened, basically stoping on target. > > Signed-off-by: Juan Quintela Note you should probably also fix Power (see my 'Fix for section footers for power') Reviewed-by: Dr. David Alan Gilbert > --- > hw/i386/pc_piix.c | 1 + > hw/i386/pc_q35.c | 1 + > include/migration/migration.h | 1 + > migration/migration.c | 30 +++++++++++++++++++++++++++++- > 4 files changed, 32 insertions(+), 1 deletion(-) > > diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c > index e142f75..735fb22 100644 > --- a/hw/i386/pc_piix.c > +++ b/hw/i386/pc_piix.c > @@ -307,6 +307,7 @@ static void pc_init1(MachineState *machine) > static void pc_compat_2_3(MachineState *machine) > { > savevm_skip_section_footers(); > + global_state_set_optional(); > } > > static void pc_compat_2_2(MachineState *machine) > diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c > index 082cd93..26104ca 100644 > --- a/hw/i386/pc_q35.c > +++ b/hw/i386/pc_q35.c > @@ -291,6 +291,7 @@ static void pc_q35_init(MachineState *machine) > static void pc_compat_2_3(MachineState *machine) > { > savevm_skip_section_footers(); > + global_state_set_optional(); > } > > static void pc_compat_2_2(MachineState *machine) > diff --git a/include/migration/migration.h b/include/migration/migration.h > index 1280193..bb53d93 100644 > --- a/include/migration/migration.h > +++ b/include/migration/migration.h > @@ -198,4 +198,5 @@ size_t ram_control_save_page(QEMUFile *f, ram_addr_t block_offset, > void ram_mig_init(void); > void savevm_skip_section_footers(void); > void register_global_state(void); > +void global_state_set_optional(void); > #endif > diff --git a/migration/migration.c b/migration/migration.c > index b4b6d69..0fd5962 100644 > --- a/migration/migration.c > +++ b/migration/migration.c > @@ -100,6 +100,7 @@ void migration_incoming_state_destroy(void) > > > typedef struct { > + bool optional; > uint32_t size; > uint8_t runstate[100]; > } GlobalState; > @@ -121,6 +122,33 @@ static char *global_state_get_runstate(void) > return (char *)global_state.runstate; > } > > +void global_state_set_optional(void) > +{ > + global_state.optional = true; > +} > + > +static bool global_state_needed(void *opaque) > +{ > + GlobalState *s = opaque; > + char *runstate = (char *)s->runstate; > + > + /* If it is not optional, it is mandatory */ > + > + if (s->optional == false) { > + return true; > + } > + > + /* If state is running or paused, it is not needed */ > + > + if (strcmp(runstate, "running") == 0 || > + strcmp(runstate, "paused") == 0) { > + return false; > + } > + > + /* for any other state it is needed */ > + return true; > +} > + > static int global_state_post_load(void *opaque, int version_id) > { > GlobalState *s = opaque; > @@ -152,7 +180,6 @@ static void global_state_pre_save(void *opaque) > > trace_migrate_global_state_pre_save((char *)s->runstate); > s->size = strlen((char *)s->runstate) + 1; > - printf("saved state: %s\n", s->runstate); > } > > static const VMStateDescription vmstate_globalstate = { > @@ -161,6 +188,7 @@ static const VMStateDescription vmstate_globalstate = { > .minimum_version_id = 1, > .post_load = global_state_post_load, > .pre_save = global_state_pre_save, > + .needed = global_state_needed, > .fields = (VMStateField[]) { > VMSTATE_UINT32(size, GlobalState), > VMSTATE_BUFFER(runstate, GlobalState), > -- > 2.4.3 > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK