From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Nbzlh-0003eL-KY for qemu-devel@nongnu.org; Mon, 01 Feb 2010 12:08:57 -0500 Received: from [199.232.76.173] (port=45204 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Nbzlh-0003e6-7P for qemu-devel@nongnu.org; Mon, 01 Feb 2010 12:08:57 -0500 Received: from Debian-exim by monty-python.gnu.org with spam-scanned (Exim 4.60) (envelope-from ) id 1Nbzlf-0005De-Bj for qemu-devel@nongnu.org; Mon, 01 Feb 2010 12:08:56 -0500 Received: from mx1.redhat.com ([209.132.183.28]:2395) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Nbzle-0005DQ-Nl for qemu-devel@nongnu.org; Mon, 01 Feb 2010 12:08:55 -0500 Received: from int-mx08.intmail.prod.int.phx2.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id o11H8r2u019751 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Mon, 1 Feb 2010 12:08:53 -0500 From: Markus Armbruster Subject: Re: [Qemu-devel] [PATCH 1/8] QMP: Initial mode-oriented support References: <1264686180-29845-1-git-send-email-lcapitulino@redhat.com> <1264686180-29845-2-git-send-email-lcapitulino@redhat.com> Date: Mon, 01 Feb 2010 18:08:50 +0100 In-Reply-To: <1264686180-29845-2-git-send-email-lcapitulino@redhat.com> (Luiz Capitulino's message of "Thu, 28 Jan 2010 11:42:53 -0200") Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Luiz Capitulino Cc: qemu-devel@nongnu.org Luiz Capitulino writes: > In order to have feature negotiation in QMP, the Monitor > has to be modified to support different modes of operation. > > We need two modes: > > o Handshake: where features are negotiated, only commands > which deal with protocol configuration are allowed > > o Operational: regular Monitor operations, all handlers > (except the protocol configuration ones) are allowed > > This commit does the following: > > 1. Adds the QMPMode data type to MonitorControl and sets > QMODE_HANDSHAKE as the default one > > 2. Grant permission to 'query-version' and 'query-commands' > to run on handshake mode > > Note, however, that these changes are not visable yet and > thus QMP's behavior is still the same. > > Signed-off-by: Luiz Capitulino > --- > monitor.c | 14 ++++++++++++++ > 1 files changed, 14 insertions(+), 0 deletions(-) > > diff --git a/monitor.c b/monitor.c > index fb7c572..baae9d0 100644 > --- a/monitor.c > +++ b/monitor.c > @@ -82,11 +82,16 @@ struct MonitorCompletionData { > void (*user_print)(Monitor *mon, const QObject *data); > }; > > +/* Handler flags */ > +#define HANDLER_HANDSHAKE 0x01 /* allowed to run on handshake mode */ > +#define HANDLER_HANDSHAKE_ONLY 0x02 /* can ONLY run on handshake mode */ > + A command's flags encode the set of modes in which this command may be used. The obvious way to encode a set of modes is to have an enable bit for each mode. Your choice of flags has the advantage that the value zero, which doesn't need an initializer, means "operational mode only". Fine with me. > typedef struct mon_cmd_t { > const char *name; > const char *args_type; > const char *params; > const char *help; > + unsigned int flags; > void (*user_print)(Monitor *mon, const QObject *data); > union { > void (*info)(Monitor *mon); > @@ -108,8 +113,14 @@ struct mon_fd_t { > QLIST_ENTRY(mon_fd_t) next; > }; > > +typedef enum QMPMode { > + QMODE_OPERATIONAL, > + QMODE_HANDSHAKE, > +} QMPMode; > + > typedef struct MonitorControl { > QObject *id; > + QMPMode mode; > int print_enabled; > JSONMessageParser parser; > } MonitorControl; > @@ -2471,6 +2482,7 @@ static const mon_cmd_t info_cmds[] = { > .args_type = "", > .params = "", > .help = "show the version of QEMU", > + .flags = HANDLER_HANDSHAKE, > .user_print = do_info_version_print, > .mhandler.info_new = do_info_version, > }, > @@ -2479,6 +2491,7 @@ static const mon_cmd_t info_cmds[] = { > .args_type = "", > .params = "", > .help = "list QMP available commands", > + .flags = HANDLER_HANDSHAKE, > .user_print = monitor_user_noop, > .mhandler.info_new = do_info_commands, > }, > @@ -4293,6 +4306,7 @@ static void monitor_control_event(void *opaque, int event) > QObject *data; > Monitor *mon = opaque; > > + mon->mc->mode = QMODE_HANDSHAKE; > json_message_parser_init(&mon->mc->parser, handle_qmp_command); > > data = qobject_from_jsonf("{ 'QMP': { 'capabilities': [] } }");