From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:55645) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SzqPG-0004ks-Bp for qemu-devel@nongnu.org; Fri, 10 Aug 2012 10:41:43 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SzqPA-000318-HL for qemu-devel@nongnu.org; Fri, 10 Aug 2012 10:41:42 -0400 Received: from e1.ny.us.ibm.com ([32.97.182.141]:60792) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SzqPA-00030k-C0 for qemu-devel@nongnu.org; Fri, 10 Aug 2012 10:41:36 -0400 Received: from /spool/local by e1.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 10 Aug 2012 10:41:35 -0400 Received: from d01relay07.pok.ibm.com (d01relay07.pok.ibm.com [9.56.227.147]) by d01dlp03.pok.ibm.com (Postfix) with ESMTP id 1D481C9003E for ; Fri, 10 Aug 2012 10:41:32 -0400 (EDT) Received: from d01av01.pok.ibm.com (d01av01.pok.ibm.com [9.56.224.215]) by d01relay07.pok.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id q7AEfTQq26673216 for ; Fri, 10 Aug 2012 10:41:29 -0400 Received: from d01av01.pok.ibm.com (loopback [127.0.0.1]) by d01av01.pok.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id q7AKCKA8025014 for ; Fri, 10 Aug 2012 16:12:22 -0400 From: Anthony Liguori In-Reply-To: <20120727131248.3fdca751@doriath.home> References: <1343396239-19272-1-git-send-email-aliguori@us.ibm.com> <1343396239-19272-4-git-send-email-aliguori@us.ibm.com> <20120727131248.3fdca751@doriath.home> Date: Fri, 10 Aug 2012 09:41:20 -0500 Message-ID: <87ipcq6c8f.fsf@codemonkey.ws> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Subject: Re: [Qemu-devel] [PATCH 3/7] qapi: add query-machines command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Luiz Capitulino Cc: Peter Maydell , libvir-list@redhat.com, qemu-devel@nongnu.org, Markus Armbruster , Alexander Graf , Eric Blake Luiz Capitulino writes: > On Fri, 27 Jul 2012 08:37:15 -0500 > Anthony Liguori wrote: > >> This provides the same output as -M ? but in a structured way. >> >> Signed-off-by: Anthony Liguori >> --- >> qapi-schema.json | 28 ++++++++++++++++++++++++++++ >> qmp-commands.hx | 6 ++++++ >> vl.c | 31 +++++++++++++++++++++++++++++++ >> 3 files changed, 65 insertions(+), 0 deletions(-) >> >> diff --git a/qapi-schema.json b/qapi-schema.json >> index 28e9914..5b47026 100644 >> --- a/qapi-schema.json >> +++ b/qapi-schema.json >> @@ -2200,3 +2200,31 @@ >> # Since: 0.14.0 >> ## >> { 'command': 'closefd', 'data': {'fdname': 'str'} } >> + >> +## >> +# @MachineInfo: >> +# >> +# Information describing a machine. >> +# >> +# @name: the name of the machine >> +# >> +# @alias: #optional an alias for the machine name >> +# >> +# @default: #optional whether the machine is default > > Why is default optional? Brievity. Regards, Anthony Liguori > >> +# >> +# Since: 1.2.0 >> +## >> +{ 'type': 'MachineInfo', >> + 'data': { 'name': 'str', '*alias': 'str', >> + '*is-default': 'bool' } } >> + >> +## >> +# @query-machines: >> +# >> +# Return a list of supported machines >> +# >> +# Returns: a list of MachineInfo >> +# >> +# Since: 1.2.0 >> +## >> +{ 'command': 'query-machines', 'returns': ['MachineInfo'] } >> diff --git a/qmp-commands.hx b/qmp-commands.hx >> index 5c55528..a6f82fc 100644 >> --- a/qmp-commands.hx >> +++ b/qmp-commands.hx >> @@ -2222,3 +2222,9 @@ EQMP >> .mhandler.cmd_new = qmp_marshal_input_device_list_properties, >> }, >> >> + { >> + .name = "query-machines", >> + .args_type = "", >> + .mhandler.cmd_new = qmp_marshal_input_query_machines, >> + }, >> + >> diff --git a/vl.c b/vl.c >> index 8904db1..cd900e0 100644 >> --- a/vl.c >> +++ b/vl.c >> @@ -1209,6 +1209,37 @@ QEMUMachine *find_default_machine(void) >> return NULL; >> } >> >> +MachineInfoList *qmp_query_machines(Error **errp) >> +{ >> + MachineInfoList *mach_list = NULL; >> + QEMUMachine *m; >> + >> + for (m = first_machine; m; m = m->next) { >> + MachineInfoList *entry; >> + MachineInfo *info; >> + >> + info = g_malloc0(sizeof(*info)); >> + if (m->is_default) { >> + info->has_is_default = true; >> + info->is_default = true; >> + } >> + >> + if (m->alias) { >> + info->has_alias = true; >> + info->alias = g_strdup(m->alias); >> + } >> + >> + info->name = g_strdup(m->name); >> + >> + entry = g_malloc0(sizeof(*entry)); >> + entry->value = info; >> + entry->next = mach_list; >> + mach_list = entry; >> + } >> + >> + return mach_list; >> +} >> + >> /***********************************************************/ >> /* main execution loop */ >>