From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44803) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VZLF0-0004vQ-Je for qemu-devel@nongnu.org; Thu, 24 Oct 2013 09:46:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VZLEs-00042K-0c for qemu-devel@nongnu.org; Thu, 24 Oct 2013 09:46:22 -0400 Received: from mail-wi0-x22a.google.com ([2a00:1450:400c:c05::22a]:46776) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VZLEr-000429-ML for qemu-devel@nongnu.org; Thu, 24 Oct 2013 09:46:13 -0400 Received: by mail-wi0-f170.google.com with SMTP id l12so9863203wiv.1 for ; Thu, 24 Oct 2013 06:46:12 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <526924A1.5010905@redhat.com> Date: Thu, 24 Oct 2013 14:46:09 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: <1382412341-1173-1-git-send-email-lilei@linux.vnet.ibm.com> <1382412341-1173-2-git-send-email-lilei@linux.vnet.ibm.com> In-Reply-To: <1382412341-1173-2-git-send-email-lilei@linux.vnet.ibm.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 01/17] rename is_active to is_block_active List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Lei Li Cc: aarcange@redhat.com, aliguori@us.ibm.com, quintela@redhat.com, qemu-devel@nongnu.org, mrhines@linux.vnet.ibm.com, mdroth@linux.vnet.ibm.com, lagarcia@br.ibm.com, rcj@linux.vnet.ibm.com Il 22/10/2013 04:25, Lei Li ha scritto: > is_active is used to identify block migration, rename to > is_block_active to make it more clear. No, is_active is used to identify whether a set of SaveVMHandlers is active. The default is true, so only block migration is using it. But we could use it in the future for other features (probably using migration capabilities instead of a flag as is the case for block). Paolo > Signed-off-by: Lei Li > --- > block-migration.c | 2 +- > include/migration/vmstate.h | 2 +- > savevm.c | 16 ++++++++-------- > 3 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/block-migration.c b/block-migration.c > index daf9ec1..b637695 100644 > --- a/block-migration.c > +++ b/block-migration.c > @@ -834,7 +834,7 @@ SaveVMHandlers savevm_block_handlers = { > .save_live_pending = block_save_pending, > .load_state = block_load, > .cancel = block_migration_cancel, > - .is_active = block_is_active, > + .is_block_active = block_is_active, > }; > > void blk_mig_init(void) > diff --git a/include/migration/vmstate.h b/include/migration/vmstate.h > index 9d09e60..c634d65 100644 > --- a/include/migration/vmstate.h > +++ b/include/migration/vmstate.h > @@ -42,7 +42,7 @@ typedef struct SaveVMHandlers { > int (*save_live_complete)(QEMUFile *f, void *opaque); > > /* This runs both outside and inside the iothread lock. */ > - bool (*is_active)(void *opaque); > + bool (*is_block_active)(void *opaque); > > /* This runs outside the iothread lock in the migration case, and > * within the lock in the savevm case. The callback had better only > diff --git a/savevm.c b/savevm.c > index 2f631d4..56b8643 100644 > --- a/savevm.c > +++ b/savevm.c > @@ -1867,8 +1867,8 @@ void qemu_savevm_state_begin(QEMUFile *f, > if (!se->ops || !se->ops->save_live_setup) { > continue; > } > - if (se->ops && se->ops->is_active) { > - if (!se->ops->is_active(se->opaque)) { > + if (se->ops && se->ops->is_block_active) { > + if (!se->ops->is_block_active(se->opaque)) { > continue; > } > } > @@ -1907,8 +1907,8 @@ int qemu_savevm_state_iterate(QEMUFile *f) > if (!se->ops || !se->ops->save_live_iterate) { > continue; > } > - if (se->ops && se->ops->is_active) { > - if (!se->ops->is_active(se->opaque)) { > + if (se->ops && se->ops->is_block_active) { > + if (!se->ops->is_block_active(se->opaque)) { > continue; > } > } > @@ -1948,8 +1948,8 @@ void qemu_savevm_state_complete(QEMUFile *f) > if (!se->ops || !se->ops->save_live_complete) { > continue; > } > - if (se->ops && se->ops->is_active) { > - if (!se->ops->is_active(se->opaque)) { > + if (se->ops && se->ops->is_block_active) { > + if (!se->ops->is_block_active(se->opaque)) { > continue; > } > } > @@ -2002,8 +2002,8 @@ uint64_t qemu_savevm_state_pending(QEMUFile *f, uint64_t max_size) > if (!se->ops || !se->ops->save_live_pending) { > continue; > } > - if (se->ops && se->ops->is_active) { > - if (!se->ops->is_active(se->opaque)) { > + if (se->ops && se->ops->is_block_active) { > + if (!se->ops->is_block_active(se->opaque)) { > continue; > } > } >