[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V4 3/7] qmp: Add a command to list the named Blo
From: |
Benoît Canet |
Subject: |
Re: [Qemu-devel] [PATCH V4 3/7] qmp: Add a command to list the named BlockDriverState nodes. |
Date: |
Mon, 9 Dec 2013 16:46:03 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Le Friday 06 Dec 2013 à 08:59:38 (-0700), Eric Blake a écrit :
> On 12/05/2013 10:14 AM, Benoît Canet wrote:
> > Signed-off-by: Benoit Canet <address@hidden>
> > ---
> > block.c | 16 ++++++++++++++++
> > blockdev.c | 5 +++++
> > include/block/block.h | 1 +
> > qapi-schema.json | 11 +++++++++++
> > qmp-commands.hx | 19 +++++++++++++++++++
> > 5 files changed, 52 insertions(+)
>
> > ##
> > +# @query-named-block-nodes
> > +#
> > +# Get the named block driver list
> > +#
> > +# Returns: the list of named nodes names
>
> Sounds funny. Maybe better with just:
>
> Returns: the list of names
>
> > +#
> > +# Since 2.0
> > +##
> > +{ 'command': 'query-named-block-nodes', 'returns': [ 'str' ] }
>
> Also, is list of names sufficient, or are we better off returning a list
> of structs with the details already included? With list of names, the
> management app has to follow up with another QMP command per name if it
> wants to know details about each node, instead of getting it all in one
> command.
Would a list of BlockDeviceInfo be fine ?
Best regards
Benoît
>
> > +SQMP
> > address@hidden
> > +------------------------
> > +
> > +Return a list of string containing the name of the named block driver
> > states
>
> Return a list of strings containing the name of each named block driver node
>
> --
> Eric Blake eblake redhat com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>
[Qemu-devel] [PATCH V4 4/7] qmp: Allow to change password on names block driver states., Benoît Canet, 2013/12/05