From nobody Sun Nov 9 22:30:05 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1551981683980598.1227025406484; Thu, 7 Mar 2019 10:01:23 -0800 (PST) Received: from localhost ([127.0.0.1]:56528 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1h1xKa-0003KV-TW for importer@patchew.org; Thu, 07 Mar 2019 13:01:20 -0500 Received: from eggs.gnu.org ([209.51.188.92]:39874) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1h1wrA-0003ei-Nm for qemu-devel@nongnu.org; Thu, 07 Mar 2019 12:30:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1h1wr9-0008Qv-As for qemu-devel@nongnu.org; Thu, 07 Mar 2019 12:30:56 -0500 Received: from mail-wm1-x342.google.com ([2a00:1450:4864:20::342]:56016) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1h1wr9-0008QE-2Z for qemu-devel@nongnu.org; Thu, 07 Mar 2019 12:30:55 -0500 Received: by mail-wm1-x342.google.com with SMTP id q187so10151339wme.5 for ; Thu, 07 Mar 2019 09:30:54 -0800 (PST) Received: from 640k.localdomain ([93.56.166.5]) by smtp.gmail.com with ESMTPSA id d206sm9979753wmc.11.2019.03.07.09.30.51 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 07 Mar 2019 09:30:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=HkGgy5COX3XUihodCLAlptHaI6AHdslJQuMj/d4DtZg=; b=cXCpwEye1bhH8vsiw4PShRLNsVNVuisq1aej5saiQ8mT5EVznis1rzAZYnTb3aeC7r M9+nbQhSJ5X9lTh2ULhsUWJu5cIAaLv4gxTyjB3q1dxJINSiICEpJDfngioAzoH6S5/9 p2+JASC2wXQVx9Pls9bOs0m9U1QBaa+kedTPPKhI+tEJhgJ2J2+pvtn6XfUN6wOKGJEd qly1ckdWD2PhlbrPH7qi+8oaoBgfns/a1f7I5YiUx3jXCXtSLpY6YnVJASokNjMPI2kt gdTJc1BcRj2Jv+GnYpbrVAhRAaS4tHDCra5jX9oy8neo5pBm29vZvF9u67a0n6lSRmlm 4zjg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references; bh=HkGgy5COX3XUihodCLAlptHaI6AHdslJQuMj/d4DtZg=; b=X3CtOAnLg6gIOE8omqoUgWvTTQvGasgLlfx1X78ei/Gzyh58iA0PST4ESO+wrwmx5c 9mqd01xesytfvP4Hlge3FQi9B/0Z1zSsuPZkFSlNaxnhrToHUm9NN39eQJMxzj2YEl0D 30/F8bpq2DkFV4xCI+r3IQLtzunKGANn57rSaQ1YwyfICxdJ2cFFsG0N6z+dUOsO0Lg7 2rJJVRTx2urUySuVr+yiwitTUBNch8nHc2vjTIR4wtuqBSooa+RQtrwWU9wbYCRXaAIa lsyjKHtiD5r8Vffoj/PoVtnzPu2QRnIfbOkiNc/EDP+aN/HvDXsZEB2gs9jA88M3MG/c HRyg== X-Gm-Message-State: APjAAAUBl4xwduBEAv0UnwOALorJK8aVUF4vEDLN4QymX3OjThNDcOGF bd12MXjHeIs1j1LucUY2rgg5O7+b X-Google-Smtp-Source: APXvYqyLLAKePin5P6HrBcoPKvSNiO1l93OLaatyrMH/DUVR5DDc3LbL+oq+IyOxeByJggM3HAz5yg== X-Received: by 2002:a1c:6555:: with SMTP id z82mr6178224wmb.125.1551979853660; Thu, 07 Mar 2019 09:30:53 -0800 (PST) From: Paolo Bonzini To: qemu-devel@nongnu.org Date: Thu, 7 Mar 2019 18:29:49 +0100 Message-Id: <1551979804-6060-43-git-send-email-pbonzini@redhat.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1551979804-6060-1-git-send-email-pbonzini@redhat.com> References: <1551979804-6060-1-git-send-email-pbonzini@redhat.com> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2a00:1450:4864:20::342 Subject: [Qemu-devel] [PULL 42/57] qos-test: virtio-scsi test node X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: thuth@redhat.com, Emanuele Giuseppe Esposito , lviver@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Emanuele Giuseppe Esposito Convert tests/virtio-scsi-test in qgraph test node, virtio-scsi-test. This test consumes a virtio-scsi interface and checks that its function return the expected values. Some functions are implemented only for virtio-scsi-pci, so they don't consume virtio-scsi, but virtio-scsi-pci Note that this test does not allocate any virtio-scsi structure, it's all done by the qtest walking graph mechanism Signed-off-by: Emanuele Giuseppe Esposito Signed-off-by: Paolo Bonzini --- tests/Makefile.include | 4 +- tests/virtio-scsi-test.c | 143 +++++++++++++++++++------------------------= ---- 2 files changed, 60 insertions(+), 87 deletions(-) diff --git a/tests/Makefile.include b/tests/Makefile.include index 48b7cf0..09483d0 100644 --- a/tests/Makefile.include +++ b/tests/Makefile.include @@ -153,8 +153,6 @@ check-qtest-generic-y +=3D tests/cdrom-test$(EXESUF) =20 check-qtest-ipack-y +=3D tests/ipoctal232-test$(EXESUF) =20 -check-qtest-virtio-y +=3D tests/virtio-scsi-test$(EXESUF) - check-qtest-pci-y +=3D tests/e1000-test$(EXESUF) check-qtest-pci-$(CONFIG_RTL8139_PCI) +=3D tests/rtl8139-test$(EXESUF) check-qtest-pci-$(CONFIG_PCNET_PCI) +=3D tests/pcnet-test$(EXESUF) @@ -766,6 +764,7 @@ qos-test-obj-$(CONFIG_VIRTFS) +=3D tests/virtio-9p-test= .o qos-test-obj-y +=3D tests/virtio-blk-test.o qos-test-obj-y +=3D tests/virtio-net-test.o qos-test-obj-y +=3D tests/virtio-rng-test.o +qos-test-obj-y +=3D tests/virtio-scsi-test.o qos-test-obj-y +=3D tests/virtio-serial-test.o =20 check-unit-y +=3D tests/test-qgraph$(EXESUF) @@ -813,7 +812,6 @@ tests/ne2000-test$(EXESUF): tests/ne2000-test.o tests/wdt_ib700-test$(EXESUF): tests/wdt_ib700-test.o tests/tco-test$(EXESUF): tests/tco-test.o $(libqos-pc-obj-y) tests/virtio-ccw-test$(EXESUF): tests/virtio-ccw-test.o -tests/virtio-scsi-test$(EXESUF): tests/virtio-scsi-test.o $(libqos-virtio-= obj-y) tests/tpci200-test$(EXESUF): tests/tpci200-test.o tests/display-vga-test$(EXESUF): tests/display-vga-test.o tests/ipoctal232-test$(EXESUF): tests/ipoctal232-test.o diff --git a/tests/virtio-scsi-test.c b/tests/virtio-scsi-test.c index 9298aa8..162b31c 100644 --- a/tests/virtio-scsi-test.c +++ b/tests/virtio-scsi-test.c @@ -18,6 +18,8 @@ #include "standard-headers/linux/virtio_ids.h" #include "standard-headers/linux/virtio_pci.h" #include "standard-headers/linux/virtio_scsi.h" +#include "libqos/virtio-scsi.h" +#include "libqos/qgraph.h" =20 #define PCI_SLOT 0x02 #define PCI_FN 0x00 @@ -27,55 +29,28 @@ =20 typedef struct { QVirtioDevice *dev; - QOSState *qs; int num_queues; QVirtQueue *vq[MAX_NUM_QUEUES + 2]; -} QVirtIOSCSI; +} QVirtioSCSIQueues; =20 -static QOSState *qvirtio_scsi_start(const char *extra_opts) -{ - QOSState *qs; - const char *arch =3D qtest_get_arch(); - const char *cmd =3D "-drive id=3Ddrv0,if=3Dnone,file=3Dnull-co://,form= at=3Draw " - "-device virtio-scsi-pci,id=3Dvs0 " - "-device scsi-hd,bus=3Dvs0.0,drive=3Ddrv0 %s"; - - if (strcmp(arch, "i386") =3D=3D 0 || strcmp(arch, "x86_64") =3D=3D 0) { - qs =3D qtest_pc_boot(cmd, extra_opts ? : ""); - } else if (strcmp(arch, "ppc64") =3D=3D 0) { - qs =3D qtest_spapr_boot(cmd, extra_opts ? : ""); - } else { - g_printerr("virtio-scsi tests are only available on x86 or ppc64\n= "); - exit(EXIT_FAILURE); - } - global_qtest =3D qs->qts; - return qs; -} - -static void qvirtio_scsi_stop(QOSState *qs) -{ - qtest_shutdown(qs); -} +static QGuestAllocator *alloc; =20 -static void qvirtio_scsi_pci_free(QVirtIOSCSI *vs) +static void qvirtio_scsi_pci_free(QVirtioSCSIQueues *vs) { int i; =20 for (i =3D 0; i < vs->num_queues + 2; i++) { - qvirtqueue_cleanup(vs->dev->bus, vs->vq[i], &vs->qs->alloc); + qvirtqueue_cleanup(vs->dev->bus, vs->vq[i], alloc); } - qvirtio_pci_device_disable(container_of(vs->dev, QVirtioPCIDevice, vde= v)); - qvirtio_pci_device_free((QVirtioPCIDevice *)vs->dev); - qvirtio_scsi_stop(vs->qs); g_free(vs); } =20 -static uint64_t qvirtio_scsi_alloc(QVirtIOSCSI *vs, size_t alloc_size, +static uint64_t qvirtio_scsi_alloc(QVirtioSCSIQueues *vs, size_t alloc_siz= e, const void *data) { uint64_t addr; =20 - addr =3D guest_alloc(&vs->qs->alloc, alloc_size); + addr =3D guest_alloc(alloc, alloc_size); if (data) { memwrite(addr, data, alloc_size); } @@ -83,7 +58,8 @@ static uint64_t qvirtio_scsi_alloc(QVirtIOSCSI *vs, size_= t alloc_size, return addr; } =20 -static uint8_t virtio_scsi_do_command(QVirtIOSCSI *vs, const uint8_t *cdb, +static uint8_t virtio_scsi_do_command(QVirtioSCSIQueues *vs, + const uint8_t *cdb, const uint8_t *data_in, size_t data_in_len, uint8_t *data_out, size_t data_out_l= en, @@ -133,40 +109,28 @@ static uint8_t virtio_scsi_do_command(QVirtIOSCSI *vs= , const uint8_t *cdb, memread(resp_addr, resp_out, sizeof(*resp_out)); } =20 - guest_free(&vs->qs->alloc, req_addr); - guest_free(&vs->qs->alloc, resp_addr); - guest_free(&vs->qs->alloc, data_in_addr); - guest_free(&vs->qs->alloc, data_out_addr); + guest_free(alloc, req_addr); + guest_free(alloc, resp_addr); + guest_free(alloc, data_in_addr); + guest_free(alloc, data_out_addr); return response; } =20 -static QVirtIOSCSI *qvirtio_scsi_pci_init(int slot) +static QVirtioSCSIQueues *qvirtio_scsi_init(QVirtioDevice *dev) { + QVirtioSCSIQueues *vs; const uint8_t test_unit_ready_cdb[VIRTIO_SCSI_CDB_SIZE] =3D {}; - QVirtIOSCSI *vs; - QVirtioPCIDevice *dev; struct virtio_scsi_cmd_resp resp; int i; =20 - vs =3D g_new0(QVirtIOSCSI, 1); - - vs->qs =3D qvirtio_scsi_start("-drive file=3Dblkdebug::null-co://," - "if=3Dnone,id=3Ddr1,format=3Draw,file.alig= n=3D4k " - "-device scsi-hd,drive=3Ddr1,lun=3D0,scsi-= id=3D1"); - dev =3D qvirtio_pci_device_find(vs->qs->pcibus, VIRTIO_ID_SCSI); - vs->dev =3D (QVirtioDevice *)dev; - g_assert(dev !=3D NULL); - g_assert_cmphex(vs->dev->device_type, =3D=3D, VIRTIO_ID_SCSI); - - qvirtio_pci_device_enable(dev); - qvirtio_start_device(vs->dev); - - vs->num_queues =3D qvirtio_config_readl(vs->dev, 0); + vs =3D g_new0(QVirtioSCSIQueues, 1); + vs->dev =3D dev; + vs->num_queues =3D qvirtio_config_readl(dev, 0); =20 g_assert_cmpint(vs->num_queues, <, MAX_NUM_QUEUES); =20 for (i =3D 0; i < vs->num_queues + 2; i++) { - vs->vq[i] =3D qvirtqueue_setup(vs->dev, &vs->qs->alloc, i); + vs->vq[i] =3D qvirtqueue_setup(dev, alloc, i); } =20 /* Clear the POWER ON OCCURRED unit attention */ @@ -182,30 +146,18 @@ static QVirtIOSCSI *qvirtio_scsi_pci_init(int slot) return vs; } =20 -/* Tests only initialization so far. TODO: Replace with functional tests */ -static void pci_nop(void) -{ - QOSState *qs; - - qs =3D qvirtio_scsi_start(NULL); - qvirtio_scsi_stop(qs); -} - -static void hotplug(void) +static void hotplug(void *obj, void *data, QGuestAllocator *alloc) { - QOSState *qs; - - qs =3D qvirtio_scsi_start( - "-drive id=3Ddrv1,if=3Dnone,file=3Dnull-co://,format=3Draw"); qtest_qmp_device_add("scsi-hd", "scsihd", "{'drive': 'drv1'}"); qtest_qmp_device_del("scsihd"); - qvirtio_scsi_stop(qs); } =20 /* Test WRITE SAME with the lba not aligned */ -static void test_unaligned_write_same(void) +static void test_unaligned_write_same(void *obj, void *data, + QGuestAllocator *t_alloc) { - QVirtIOSCSI *vs; + QVirtioSCSI *scsi =3D obj; + QVirtioSCSIQueues *vs; uint8_t buf1[512] =3D { 0 }; uint8_t buf2[512] =3D { 1 }; const uint8_t write_same_cdb_1[VIRTIO_SCSI_CDB_SIZE] =3D { @@ -218,27 +170,50 @@ static void test_unaligned_write_same(void) 0x41, 0x01, 0x00, 0x00, 0x00, 0x01, 0x00, 0x33, 0x00, 0x00 }; =20 - vs =3D qvirtio_scsi_pci_init(PCI_SLOT); + alloc =3D t_alloc; + vs =3D qvirtio_scsi_init(scsi->vdev); =20 g_assert_cmphex(0, =3D=3D, - virtio_scsi_do_command(vs, write_same_cdb_1, NULL, 0, buf1, 512, N= ULL)); + virtio_scsi_do_command(vs, write_same_cdb_1, NULL, 0, buf1, 512, + NULL)); =20 g_assert_cmphex(0, =3D=3D, - virtio_scsi_do_command(vs, write_same_cdb_2, NULL, 0, buf2, 512, N= ULL)); + virtio_scsi_do_command(vs, write_same_cdb_2, NULL, 0, buf2, 512, + NULL)); =20 g_assert_cmphex(0, =3D=3D, - virtio_scsi_do_command(vs, write_same_cdb_ndob, NULL, 0, NULL, 0, = NULL)); + virtio_scsi_do_command(vs, write_same_cdb_ndob, NULL, 0, NULL, 0, + NULL)); =20 qvirtio_scsi_pci_free(vs); } =20 -int main(int argc, char **argv) +static void *virtio_scsi_hotplug_setup(GString *cmd_line, void *arg) { - g_test_init(&argc, &argv, NULL); - qtest_add_func("/virtio/scsi/pci/nop", pci_nop); - qtest_add_func("/virtio/scsi/pci/hotplug", hotplug); - qtest_add_func("/virtio/scsi/pci/scsi-disk/unaligned-write-same", - test_unaligned_write_same); + g_string_append(cmd_line, + " -drive id=3Ddrv1,if=3Dnone,file=3Dnull-co://,format= =3Draw"); + return arg; +} =20 - return g_test_run(); +static void *virtio_scsi_setup(GString *cmd_line, void *arg) +{ + g_string_append(cmd_line, + " -drive file=3Dblkdebug::null-co://," + "if=3Dnone,id=3Ddr1,format=3Draw,file.align=3D4k " + "-device scsi-hd,drive=3Ddr1,lun=3D0,scsi-id=3D1"); + return arg; } + +static void register_virtio_scsi_test(void) +{ + QOSGraphTestOptions opts =3D { }; + + opts.before =3D virtio_scsi_hotplug_setup; + qos_add_test("hotplug", "virtio-scsi", hotplug, &opts); + + opts.before =3D virtio_scsi_setup; + qos_add_test("unaligned-write-same", "virtio-scsi", + test_unaligned_write_same, &opts); +} + +libqos_init(register_virtio_scsi_test); --=20 1.8.3.1