From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42658) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1db8tX-000114-0b for qemu-devel@nongnu.org; Fri, 28 Jul 2017 13:17:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1db8tP-0001xL-Si for qemu-devel@nongnu.org; Fri, 28 Jul 2017 13:17:43 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49982) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1db8tP-0001uK-Mt for qemu-devel@nongnu.org; Fri, 28 Jul 2017 13:17:39 -0400 Date: Fri, 28 Jul 2017 18:17:31 +0100 From: "Dr. David Alan Gilbert" Message-ID: <20170728171731.GB3008@work-vm> References: <1501249552-788-1-git-send-email-armbru@redhat.com> <1501249552-788-3-git-send-email-armbru@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1501249552-788-3-git-send-email-armbru@redhat.com> Subject: Re: [Qemu-devel] [PATCH 2/3] COLO: Define COLOMode without QAPI List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: qemu-devel@nongnu.org, zhanghailiang * Markus Armbruster (armbru@redhat.com) wrote: > COLOMode is defined in the QAPI schema, but not used there. Of the > stuff QAPI generates for it only the typedef is actually used. Use of > QAPI is pointless and only complicates things, so don't. Hmm, in one of the old COLO worlds I have, there's code to emit an event on exiting from COLO and that event includes the mode it was in. If the intent is to bring that or similar back then it would be worth keeping. Dave > Cc: zhanghailiang > Signed-off-by: Markus Armbruster > --- > include/migration/colo.h | 6 ++++++ > qapi-schema.json | 16 ---------------- > 2 files changed, 6 insertions(+), 16 deletions(-) > > diff --git a/include/migration/colo.h b/include/migration/colo.h > index ff9874e..5d7c500 100644 > --- a/include/migration/colo.h > +++ b/include/migration/colo.h > @@ -26,6 +26,12 @@ void migration_incoming_exit_colo(void); > void *colo_process_incoming_thread(void *opaque); > bool migration_incoming_in_colo_state(void); > > +typedef enum { > + COLO_MODE_UNKNOWN, > + COLO_MODE_PRIMARY, > + COLO_MODE_SECONDARY, > +} COLOMode; > + > COLOMode get_colo_mode(void); > > /* failover */ > diff --git a/qapi-schema.json b/qapi-schema.json > index 9b6f6cb..3f0eb05 100644 > --- a/qapi-schema.json > +++ b/qapi-schema.json > @@ -1304,22 +1304,6 @@ > 'vmstate-loaded' ] } > > ## > -# @COLOMode: > -# > -# The colo mode > -# > -# @unknown: unknown mode > -# > -# @primary: master side > -# > -# @secondary: slave side > -# > -# Since: 2.8 > -## > -{ 'enum': 'COLOMode', > - 'data': [ 'unknown', 'primary', 'secondary'] } > - > -## > # @FailoverStatus: > # > # An enumeration of COLO failover status > -- > 2.7.5 > > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK