[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Patch v7 3/3] qmp: add monitor command to add/remove a
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [Patch v7 3/3] qmp: add monitor command to add/remove a child |
Date: |
Mon, 23 Nov 2015 09:30:32 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 11/22/2015 11:23 PM, Wen Congyang wrote:
> The new QMP command name is x-blockdev-change. It justs for adding/removing
s/It justs/It's just/
> quorum's child now, and don't support all kinds of children, all kinds of
s/don't/doesn't/
> operations, nor all block drivers. So it is experimental now.
>
> Signed-off-by: Wen Congyang <address@hidden>
> Signed-off-by: zhanghailiang <address@hidden>
> Signed-off-by: Gonglei <address@hidden>
> ---
> +++ b/qapi/block-core.json
> @@ -2400,3 +2400,28 @@
> ##
> { 'command': 'block-set-write-threshold',
> 'data': { 'node-name': 'str', 'write-threshold': 'uint64' } }
> +
> +##
> +# @x-blockdev-change
> +#
> +# Dynamically reconfigure the block driver state graph. It can be used
> +# to add, remove, insert or replace a block driver state. Currently only
> +# the Quorum driver implements this feature to add or remove its child.
> +# This is useful to fix a broken quorum child.
> +#
> +# @operation: the change operation. It can be add, delete.
Documented but not present below. Are you missing a parameter, or
should this line be deleted?
> +#
> +# @parent: the id or name of the node that will be changed.
> +#
> +# @child: #optional the name of the child that will be deleted.
> +#
> +# @node: #optional the name of the node will be added.
> +#
> +# Note: this command is experimental, and its API is not stable.
> +#
> +# Since: 2.6
> +##
> +{ 'command': 'x-blockdev-change',
> + 'data' : { 'parent': 'str',
> + '*child': 'str',
> + '*node': 'str' } }
> +++ b/qmp-commands.hx
> @@ -4285,6 +4285,53 @@ Example:
> EQMP
>
> {
> + .name = "x-blockdev-change",
> + .args_type = "parent:B,child:B?,node:B?",
> + .mhandler.cmd_new = qmp_marshal_x_blockdev_change,
> + },
> +
> +SQMP
> +x-blockdev-change
> +------------
Make the --- divider as long as the text it is underlining.
> +
> +Dynamic reconfigure the block driver state graph. It can be used to
s/Dynamic/Dynamically/
> +add, remove, insert, replace a block driver state. Currently only
s/replace/or replace/
Isn't 'add' and 'insert' the same thing?
> +the Quorum driver implements this feature to add and remove its child.
> +This is useful to fix a broken quorum child.
> +
> +Arguments:
> +- "parent": the id or node name of which node will be changed
> +- "child": the child name which will be delete
s/delete/deleted/; mention that it is optional (if nothing is going to
be delted)
> +- "node": the new node-name which will be added
mention that it is optional (if nothing is going to be added)
> +
> +Note: this command is experimental, and not a stable API. It doesn't
> +support all kinds of operations, all kindes of children, nor all block
s/kindes/kinds/
> +drivers.
> +
> +Example:
> +
> +Add a new quorum's node
s/quorum's node/node to a quorum/
> +-> { "execute": blockdev-add",
> + "arguments": { "options": { "driver": "raw",
> + "node-name": "new_node",
> + "id": "test_new_node",
> + "file": { "driver": "file",
> + "filename": "test.raw" } } } }
> +<- { "return": {} }
> +-> { "execute": "x-blockdev-change",
> + "arguments": { "parent": "disk1",
> + "node": "new_node" } }
> +<- { "return": {} }
> +
> +Delete a quorum's node
> +-> { "execute": "x-blockdev-change",
> + "arguments": { "parent": "disk1",
> + "child": "children.2" } }
> +<- { "return": {} }
> +
> +EQMP
> +
> + {
> .name = "query-named-block-nodes",
> .args_type = "",
> .mhandler.cmd_new = qmp_marshal_query_named_block_nodes,
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature