From: Wen Congyang <wency@cn.fujitsu.com>
To: qemu devel <qemu-devel@nongnu.org>,
Eric Blake <eblake@redhat.com>,
Markus Armbruster <armbru@redhat.com>,
Alberto Garcia <berto@igalia.com>,
Stefan Hajnoczi <stefanha@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>, qemu block <qemu-block@nongnu.org>,
Jiang Yunhong <yunhong.jiang@intel.com>,
Dong Eddie <eddie.dong@intel.com>,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>,
Luiz Capitulino <lcapitulino@redhat.com>,
Gonglei <arei.gonglei@huawei.com>,
Yang Hongyang <yanghy@cn.fujitsu.com>,
zhanghailiang <zhang.zhanghailiang@huawei.com>
Subject: [Qemu-devel] [PATCH v5 4/4] hmp: add monitor command to add/remove a child
Date: Tue, 22 Sep 2015 15:44:22 +0800 [thread overview]
Message-ID: <1442907862-21376-5-git-send-email-wency@cn.fujitsu.com> (raw)
In-Reply-To: <1442907862-21376-1-git-send-email-wency@cn.fujitsu.com>
The new command is blockdev_child_add and blockdev_child_del.
It does the same thing as the QMP command x-blockdev-child-add
and x-blockdev-child-del.
Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
Signed-off-by: Gonglei <arei.gonglei@huawei.com>
Cc: Luiz Capitulino <lcapitulino@redhat.com>
---
hmp-commands.hx | 28 ++++++++++++++++++++++++++++
hmp.c | 20 ++++++++++++++++++++
hmp.h | 2 ++
3 files changed, 50 insertions(+)
diff --git a/hmp-commands.hx b/hmp-commands.hx
index a511004..57adb7d 100644
--- a/hmp-commands.hx
+++ b/hmp-commands.hx
@@ -193,6 +193,34 @@ actions (drive options rerror, werror).
ETEXI
{
+ .name = "blockdev_child_add",
+ .args_type = "id:B,child:B",
+ .params = "parent child",
+ .help = "add a child to a BDS",
+ .mhandler.cmd = hmp_blockdev_child_add,
+ },
+
+STEXI
+@item blockdev_child_add @var{parent} @var{child}
+@findex blockdev_child_add
+Add a child to the block device.
+ETEXI
+
+ {
+ .name = "blockdev_child_del",
+ .args_type = "id:B,child:B",
+ .params = "parent child",
+ .help = "remove a child from a BDS",
+ .mhandler.cmd = hmp_blockdev_child_del,
+ },
+
+STEXI
+@item blockdev_child_del @var{parent} @var{child}
+@findex blockdev_child_del
+Remove a child from the parent device.
+ETEXI
+
+ {
.name = "change",
.args_type = "device:B,target:F,arg:s?",
.params = "device filename [format]",
diff --git a/hmp.c b/hmp.c
index 3f807b7..1dc1bed 100644
--- a/hmp.c
+++ b/hmp.c
@@ -2325,3 +2325,23 @@ void hmp_rocker_of_dpa_groups(Monitor *mon, const QDict *qdict)
qapi_free_RockerOfDpaGroupList(list);
}
+
+void hmp_blockdev_child_add(Monitor *mon, const QDict *qdict)
+{
+ const char *id = qdict_get_str(qdict, "id");
+ const char *child_id = qdict_get_str(qdict, "child");
+ Error *local_err = NULL;
+
+ qmp_x_blockdev_child_add(id, child_id, &local_err);
+ hmp_handle_error(mon, &local_err);
+}
+
+void hmp_blockdev_child_del(Monitor *mon, const QDict *qdict)
+{
+ const char *id = qdict_get_str(qdict, "id");
+ const char *child_id = qdict_get_str(qdict, "child");
+ Error *local_err = NULL;
+
+ qmp_x_blockdev_child_del(id, child_id, &local_err);
+ hmp_handle_error(mon, &local_err);
+}
diff --git a/hmp.h b/hmp.h
index 81656c3..3033648 100644
--- a/hmp.h
+++ b/hmp.h
@@ -130,5 +130,7 @@ void hmp_rocker(Monitor *mon, const QDict *qdict);
void hmp_rocker_ports(Monitor *mon, const QDict *qdict);
void hmp_rocker_of_dpa_flows(Monitor *mon, const QDict *qdict);
void hmp_rocker_of_dpa_groups(Monitor *mon, const QDict *qdict);
+void hmp_blockdev_child_add(Monitor *mon, const QDict *qdict);
+void hmp_blockdev_child_del(Monitor *mon, const QDict *qdict);
#endif
--
2.4.3
next prev parent reply other threads:[~2015-09-22 7:45 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-22 7:44 [Qemu-devel] [PATCH v5 0/4] qapi: child add/delete support Wen Congyang
2015-09-22 7:44 ` [Qemu-devel] [PATCH v5 1/4] Add new block driver interface to add/delete a BDS's child Wen Congyang
2015-10-07 13:35 ` Alberto Garcia
2015-10-08 2:05 ` Wen Congyang
2015-10-07 18:33 ` [Qemu-devel] [Qemu-block] " Max Reitz
2015-10-08 2:06 ` Wen Congyang
2015-10-07 19:00 ` [Qemu-devel] " Dr. David Alan Gilbert
2015-10-08 2:03 ` Wen Congyang
2015-10-08 18:44 ` Dr. David Alan Gilbert
2015-09-22 7:44 ` [Qemu-devel] [PATCH v5 2/4] quorum: implement bdrv_add_child() and bdrv_del_child() Wen Congyang
2015-10-07 14:12 ` Alberto Garcia
2015-10-08 2:10 ` Wen Congyang
2015-10-07 18:51 ` [Qemu-devel] [Qemu-block] " Max Reitz
2015-10-08 8:12 ` Alberto Garcia
2015-10-09 15:51 ` Max Reitz
2015-10-12 11:56 ` Alberto Garcia
2015-09-22 7:44 ` [Qemu-devel] [PATCH v5 3/4] qmp: add monitor command to add/remove a child Wen Congyang
2015-10-07 14:33 ` Alberto Garcia
2015-10-07 19:42 ` [Qemu-devel] [Qemu-block] " Max Reitz
2015-10-08 6:15 ` Markus Armbruster
2015-10-08 8:29 ` Alberto Garcia
2015-10-08 10:03 ` Kevin Wolf
2015-10-08 10:13 ` Alberto Garcia
2015-10-09 16:14 ` Max Reitz
2015-10-08 11:02 ` [Qemu-devel] Dynamic reconfiguration (was: qmp: add monitor command to add/remove a child) Kevin Wolf
2015-10-08 11:10 ` [Qemu-devel] [Qemu-block] " Kevin Wolf
2015-10-21 8:27 ` [Qemu-devel] [Qemu-block] Dynamic reconfiguration Markus Armbruster
2015-10-26 2:04 ` Wen Congyang
2015-10-26 7:24 ` Markus Armbruster
2015-10-26 7:25 ` Wen Congyang
2015-10-09 16:13 ` [Qemu-devel] [Qemu-block] [PATCH v5 3/4] qmp: add monitor command to add/remove a child Max Reitz
2015-10-09 16:42 ` Dr. David Alan Gilbert
2015-10-09 18:24 ` Max Reitz
2015-10-12 8:07 ` Dr. David Alan Gilbert
2015-10-12 8:18 ` Kevin Wolf
2015-10-12 7:58 ` Markus Armbruster
2015-10-12 7:56 ` Markus Armbruster
2015-10-12 16:27 ` Max Reitz
2015-09-22 7:44 ` Wen Congyang [this message]
2015-10-07 14:38 ` [Qemu-devel] [PATCH v5 4/4] hmp: " Alberto Garcia
2015-09-22 11:15 ` [Qemu-devel] [PATCH v5 0/4] qapi: child add/delete support Dr. David Alan Gilbert
2015-09-23 1:08 ` Wen Congyang
2015-09-23 9:21 ` Dr. David Alan Gilbert
2015-09-23 9:30 ` Wen Congyang
2015-10-07 6:40 ` Wen Congyang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1442907862-21376-5-git-send-email-wency@cn.fujitsu.com \
--to=wency@cn.fujitsu.com \
--cc=arei.gonglei@huawei.com \
--cc=armbru@redhat.com \
--cc=berto@igalia.com \
--cc=dgilbert@redhat.com \
--cc=eblake@redhat.com \
--cc=eddie.dong@intel.com \
--cc=kwolf@redhat.com \
--cc=lcapitulino@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
--cc=yanghy@cn.fujitsu.com \
--cc=yunhong.jiang@intel.com \
--cc=zhang.zhanghailiang@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).