From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37467) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WxIfC-000142-Nd for qemu-devel@nongnu.org; Wed, 18 Jun 2014 12:24:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WxIf5-00023B-82 for qemu-devel@nongnu.org; Wed, 18 Jun 2014 12:24:42 -0400 Received: from mx1.redhat.com ([209.132.183.28]:14621) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WxIf5-000233-0Y for qemu-devel@nongnu.org; Wed, 18 Jun 2014 12:24:35 -0400 From: Amos Kong Date: Thu, 19 Jun 2014 00:24:13 +0800 Message-Id: <1403108653-26765-5-git-send-email-akong@redhat.com> In-Reply-To: <1403108653-26765-1-git-send-email-akong@redhat.com> References: <1403108653-26765-1-git-send-email-akong@redhat.com> Subject: [Qemu-devel] [PATCH v5 4/4] virtio-blk-test.c: add hotplug subtest List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, stefanha@gmail.com, arei.gonglei@huawei.com, afaerber@suse.de, pbonzini@redhat.com This patch adds a new subtest, it hotplugs 29 * 8 = 232 virtio-blk devices to guest, and try to hot-unplug them. Note: the hot-unplug can't work without cooperation of guest OS. Signed-off-by: Amos Kong --- tests/virtio-blk-test.c | 68 +++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 68 insertions(+) diff --git a/tests/virtio-blk-test.c b/tests/virtio-blk-test.c index 0fdec01..77817ca 100644 --- a/tests/virtio-blk-test.c +++ b/tests/virtio-blk-test.c @@ -7,11 +7,78 @@ * See the COPYING file in the top-level directory. */ +#include #include #include #include "libqtest.h" #include "qemu/osdep.h" + +/* start with no network/block device, slots 3 to 0x1f are free */ +#define MIN_PCI_SLOT 3 +#define MAX_PCI_SLOT 0x1f +#define MAX_PCI_FUNC 7 + +static void test_blk_hotplug(void) +{ + QDict *response; + int i, j; + char drive_id[10]; + char device_id[10]; + char addr[10]; + + qtest_start("-net none"); + + for (i = MIN_PCI_SLOT; i <= MAX_PCI_SLOT; i++) { + for (j = MAX_PCI_FUNC; j >= 0; j--) { + sprintf(drive_id, "drv-%x.%x", i, j); + response = qmp("{ 'execute': 'blockdev-add'," + " 'arguments': {" + " 'options': {" + " 'driver': 'file'," + " 'filename': '/dev/null'," + " 'id': %s" + "} } }", drive_id); + g_assert(response); + g_assert(!qdict_haskey(response, "error")); + QDECREF(response); + + sprintf(device_id, "dev-%x.%x", i, j); + sprintf(addr, "0x%x.%x", i, j); + response = qmp("{'execute': 'device_add'," + " 'arguments': {" + " 'driver': 'virtio-blk-pci'," + " 'drive': %s," + " 'id': %s," + " 'addr': %s," + " 'multifunction': 'on'" + "} }", drive_id, device_id, addr); + g_assert(response); + g_assert(!qdict_haskey(response, "error")); + QDECREF(response); + } + } + + /* hot-unplug doesn't work without cooperation of guest OS */ + for (i = MIN_PCI_SLOT; i <= MAX_PCI_SLOT; i++) { + for (j = MAX_PCI_FUNC; j >= 0; j--) { + if (j == MAX_PCI_FUNC) { + qmp_exec_hmp_cmd("", "drive_del drv-%x.%x", i, 0); + } + sprintf(device_id, "dev-%x.%x", i, j); + response = qmp("{'execute': 'device_del'," + " 'arguments': {" + " 'id': %s" + "} }", device_id); + g_assert(response); + g_assert(!qdict_haskey(response, "error")); + QDECREF(response); + } + } + + qtest_end(); +} + /* Tests only initialization */ static void virtblk_init(void) { @@ -26,6 +93,7 @@ int main(int argc, char **argv) g_test_init(&argc, &argv, NULL); qtest_add_func("/virtio/blk/pci/init", virtblk_init); + qtest_add_func("/virtio/blk/pci/hotplug", test_blk_hotplug); ret = g_test_run(); -- 1.9.3