* [Qemu-devel] [PATCH] tests: virtio: separate ccw tests from libqos
@ 2018-08-01 13:27 Paolo Bonzini
2018-08-01 13:34 ` Cornelia Huck
2018-08-01 16:39 ` [Qemu-devel] [qemu-s390x] " Thomas Huth
0 siblings, 2 replies; 4+ messages in thread
From: Paolo Bonzini @ 2018-08-01 13:27 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-s390x, e.emanuelegiuseppe
Because qtest does not support s390 channel I/O, s390 only performs smoke tests on
those few devices that do not have any functional tests. Therefore, every time we
add functional tests for a virtio device, the choice is between removing
those tests from the s390 suite (so that s390 actually _loses_ coverage)
or sprinkling the test with architecture checks.
This patch simply creates a ccw-specific test that only performs smoke tests on
all virtio-ccw devices. If channel I/O support is ever added to qtest and libqos,
then this file can go away. In the meanwhile, it simplifies maintenance and
makes sure that all virtio devices are tested.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
tests/Makefile.include | 5 +-
tests/virtio-ccw-test.c | 118 ++++++++++++++++++++++++++++++++++++++++
2 files changed, 120 insertions(+), 3 deletions(-)
create mode 100644 tests/virtio-ccw-test.c
diff --git a/tests/Makefile.include b/tests/Makefile.include
index a49282704e..cf835f2fd0 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -401,9 +401,7 @@ check-qtest-s390x-$(CONFIG_SLIRP) += tests/test-netfilter$(EXESUF)
check-qtest-s390x-$(CONFIG_POSIX) += tests/test-filter-mirror$(EXESUF)
check-qtest-s390x-$(CONFIG_POSIX) += tests/test-filter-redirector$(EXESUF)
check-qtest-s390x-y += tests/drive_del-test$(EXESUF)
-check-qtest-s390x-y += tests/virtio-balloon-test$(EXESUF)
-check-qtest-s390x-y += tests/virtio-console-test$(EXESUF)
-check-qtest-s390x-y += tests/virtio-serial-test$(EXESUF)
+check-qtest-s390x-y += tests/virtio-ccw-test$(EXESUF)
check-qtest-s390x-y += tests/cpu-plug-test$(EXESUF)
check-qtest-generic-y += tests/machine-none-test$(EXESUF)
@@ -807,6 +805,7 @@ tests/wdt_ib700-test$(EXESUF): tests/wdt_ib700-test.o
tests/tco-test$(EXESUF): tests/tco-test.o $(libqos-pc-obj-y)
tests/virtio-balloon-test$(EXESUF): tests/virtio-balloon-test.o $(libqos-virtio-obj-y)
tests/virtio-blk-test$(EXESUF): tests/virtio-blk-test.o $(libqos-virtio-obj-y)
+tests/virtio-ccw-test$(EXESUF): tests/virtio-ccw-test.o
tests/virtio-net-test$(EXESUF): tests/virtio-net-test.o $(libqos-pc-obj-y) $(libqos-virtio-obj-y)
tests/virtio-rng-test$(EXESUF): tests/virtio-rng-test.o $(libqos-pc-obj-y)
tests/virtio-scsi-test$(EXESUF): tests/virtio-scsi-test.o $(libqos-virtio-obj-y)
diff --git a/tests/virtio-ccw-test.c b/tests/virtio-ccw-test.c
new file mode 100644
index 0000000000..0af3a7bb31
--- /dev/null
+++ b/tests/virtio-ccw-test.c
@@ -0,0 +1,118 @@
+/*
+ * QTest testcase for VirtIO CCW
+ *
+ * Copyright (c) 2014 SUSE LINUX Products GmbH
+ * Copyright (c) 2018 Red Hat, Inc.
+ *
+ * This work is licensed under the terms of the GNU GPL, version 2 or later.
+ * See the COPYING file in the top-level directory.
+ */
+
+/* Until we have a full libqos implementation of virtio-ccw (which requires
+ * also to add support for I/O channels to qtest), we can only do simple
+ * tests that initialize the devices.
+ */
+
+#include "qemu/osdep.h"
+#include "libqtest.h"
+#include "libqos/virtio.h"
+
+static void virtio_balloon_nop(void)
+{
+ global_qtest = qtest_startf("-device virtio-balloon-ccw");
+ qtest_end();
+}
+
+static void virtconsole_nop(void)
+{
+ global_qtest = qtest_startf("-device virtio-serial-ccw,id=vser0 "
+ "-device virtconsole,bus=vser0.0");
+ qtest_end();
+}
+
+static void virtserialport_nop(void)
+{
+ global_qtest = qtest_startf("-device virtio-serial-ccw,id=vser0 "
+ "-device virtserialport,bus=vser0.0");
+ qtest_end();
+}
+
+static void virtio_serial_nop(void)
+{
+ global_qtest = qtest_startf("-device virtio-serial-ccw");
+ qtest_end();
+}
+
+static void virtio_serial_hotplug(void)
+{
+ global_qtest = qtest_startf("-device virtio-serial-ccw");
+ qtest_qmp_device_add("virtserialport", "hp-port", NULL);
+ qtest_qmp_device_del("hp-port");
+ qtest_end();
+}
+
+static void virtio_blk_nop(void)
+{
+ global_qtest = qtest_startf("-drive if=none,id=drv0,file=null-co://,format=raw "
+ "-device virtio-blk-ccw,drive=drv0");
+ qtest_end();
+}
+
+static void virtio_net_nop(void)
+{
+ global_qtest = qtest_startf("-device virtio-net-ccw");
+ qtest_end();
+}
+
+static void virtio_9p_nop(void)
+{
+ global_qtest = qtest_startf("-fsdev synth,id=fsdev0 "
+ "-device virtio-9p-ccw,fsdev=fsdev0,mount_tag=qtest");
+ qtest_end();
+}
+
+static void virtio_rng_nop(void)
+{
+ global_qtest = qtest_startf("-device virtio-rng-ccw");
+ qtest_end();
+}
+
+static void virtio_scsi_nop(void)
+{
+ global_qtest = qtest_startf("-device virtio-scsi-ccw");
+ qtest_end();
+}
+
+static void virtio_scsi_hotplug(void)
+{
+ global_qtest = qtest_startf("-drive if=none,id=drv0,file=null-co://,format=raw "
+ "-drive if=none,id=drv1,file=null-co://,format=raw "
+ "-device virtio-scsi-ccw "
+ "-device scsi-hd,drive=drv0");
+ qtest_qmp_device_add("scsi-hd", "scsihd", "'drive': 'drv1'");
+ qtest_qmp_device_del("scsihd");
+
+ qtest_end();
+}
+
+int main(int argc, char **argv)
+{
+ int ret;
+
+ g_test_init(&argc, &argv, NULL);
+ qtest_add_func("/virtio/balloon/nop", virtio_balloon_nop);
+ qtest_add_func("/virtio/console/nop", virtconsole_nop);
+ qtest_add_func("/virtio/serialport/nop", virtserialport_nop);
+ qtest_add_func("/virtio/serial/nop", virtio_serial_nop);
+ qtest_add_func("/virtio/serial/hotplug", virtio_serial_hotplug);
+ qtest_add_func("/virtio/block/nop", virtio_blk_nop);
+ qtest_add_func("/virtio/net/nop", virtio_net_nop);
+ qtest_add_func("/virtio/9p/nop", virtio_9p_nop);
+ qtest_add_func("/virtio/rng/nop", virtio_rng_nop);
+ qtest_add_func("/virtio/scsi/nop", virtio_scsi_nop);
+ qtest_add_func("/virtio/scsi/hotplug", virtio_scsi_hotplug);
+
+ ret = g_test_run();
+
+ return ret;
+}
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [Qemu-devel] [PATCH] tests: virtio: separate ccw tests from libqos
2018-08-01 13:27 [Qemu-devel] [PATCH] tests: virtio: separate ccw tests from libqos Paolo Bonzini
@ 2018-08-01 13:34 ` Cornelia Huck
2018-08-01 16:39 ` [Qemu-devel] [qemu-s390x] " Thomas Huth
1 sibling, 0 replies; 4+ messages in thread
From: Cornelia Huck @ 2018-08-01 13:34 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: qemu-devel, qemu-s390x, e.emanuelegiuseppe
On Wed, 1 Aug 2018 15:27:33 +0200
Paolo Bonzini <pbonzini@redhat.com> wrote:
> Because qtest does not support s390 channel I/O, s390 only performs smoke tests on
> those few devices that do not have any functional tests. Therefore, every time we
> add functional tests for a virtio device, the choice is between removing
> those tests from the s390 suite (so that s390 actually _loses_ coverage)
> or sprinkling the test with architecture checks.
>
> This patch simply creates a ccw-specific test that only performs smoke tests on
> all virtio-ccw devices. If channel I/O support is ever added to qtest and libqos,
> then this file can go away. In the meanwhile, it simplifies maintenance and
> makes sure that all virtio devices are tested.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> tests/Makefile.include | 5 +-
> tests/virtio-ccw-test.c | 118 ++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 120 insertions(+), 3 deletions(-)
> create mode 100644 tests/virtio-ccw-test.c
I'm not that familiar with qtest/libqos, but that sounds like a
reasonable approach.
Acked-by: Cornelia Huck <cohuck@redhat.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [qemu-s390x] [PATCH] tests: virtio: separate ccw tests from libqos
2018-08-01 13:27 [Qemu-devel] [PATCH] tests: virtio: separate ccw tests from libqos Paolo Bonzini
2018-08-01 13:34 ` Cornelia Huck
@ 2018-08-01 16:39 ` Thomas Huth
2018-08-01 16:46 ` Paolo Bonzini
1 sibling, 1 reply; 4+ messages in thread
From: Thomas Huth @ 2018-08-01 16:39 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: qemu-s390x, e.emanuelegiuseppe
On 08/01/2018 03:27 PM, Paolo Bonzini wrote:
> Because qtest does not support s390 channel I/O, s390 only performs smoke tests on
> those few devices that do not have any functional tests. Therefore, every time we
> add functional tests for a virtio device, the choice is between removing
> those tests from the s390 suite (so that s390 actually _loses_ coverage)
> or sprinkling the test with architecture checks.
>
> This patch simply creates a ccw-specific test that only performs smoke tests on
> all virtio-ccw devices. If channel I/O support is ever added to qtest and libqos,
> then this file can go away. In the meanwhile, it simplifies maintenance and
> makes sure that all virtio devices are tested.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> tests/Makefile.include | 5 +-
> tests/virtio-ccw-test.c | 118 ++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 120 insertions(+), 3 deletions(-)
> create mode 100644 tests/virtio-ccw-test.c
>
> diff --git a/tests/Makefile.include b/tests/Makefile.include
> index a49282704e..cf835f2fd0 100644
> --- a/tests/Makefile.include
> +++ b/tests/Makefile.include
> @@ -401,9 +401,7 @@ check-qtest-s390x-$(CONFIG_SLIRP) += tests/test-netfilter$(EXESUF)
> check-qtest-s390x-$(CONFIG_POSIX) += tests/test-filter-mirror$(EXESUF)
> check-qtest-s390x-$(CONFIG_POSIX) += tests/test-filter-redirector$(EXESUF)
> check-qtest-s390x-y += tests/drive_del-test$(EXESUF)
> -check-qtest-s390x-y += tests/virtio-balloon-test$(EXESUF)
> -check-qtest-s390x-y += tests/virtio-console-test$(EXESUF)
> -check-qtest-s390x-y += tests/virtio-serial-test$(EXESUF)
> +check-qtest-s390x-y += tests/virtio-ccw-test$(EXESUF)
> check-qtest-s390x-y += tests/cpu-plug-test$(EXESUF)
>
> check-qtest-generic-y += tests/machine-none-test$(EXESUF)
> @@ -807,6 +805,7 @@ tests/wdt_ib700-test$(EXESUF): tests/wdt_ib700-test.o
> tests/tco-test$(EXESUF): tests/tco-test.o $(libqos-pc-obj-y)
> tests/virtio-balloon-test$(EXESUF): tests/virtio-balloon-test.o $(libqos-virtio-obj-y)
> tests/virtio-blk-test$(EXESUF): tests/virtio-blk-test.o $(libqos-virtio-obj-y)
> +tests/virtio-ccw-test$(EXESUF): tests/virtio-ccw-test.o
> tests/virtio-net-test$(EXESUF): tests/virtio-net-test.o $(libqos-pc-obj-y) $(libqos-virtio-obj-y)
> tests/virtio-rng-test$(EXESUF): tests/virtio-rng-test.o $(libqos-pc-obj-y)
> tests/virtio-scsi-test$(EXESUF): tests/virtio-scsi-test.o $(libqos-virtio-obj-y)
> diff --git a/tests/virtio-ccw-test.c b/tests/virtio-ccw-test.c
> new file mode 100644
> index 0000000000..0af3a7bb31
> --- /dev/null
> +++ b/tests/virtio-ccw-test.c
> @@ -0,0 +1,118 @@
> +/*
> + * QTest testcase for VirtIO CCW
> + *
> + * Copyright (c) 2014 SUSE LINUX Products GmbH
> + * Copyright (c) 2018 Red Hat, Inc.
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or later.
> + * See the COPYING file in the top-level directory.
> + */
> +
> +/* Until we have a full libqos implementation of virtio-ccw (which requires
> + * also to add support for I/O channels to qtest), we can only do simple
> + * tests that initialize the devices.
> + */
> +
> +#include "qemu/osdep.h"
> +#include "libqtest.h"
> +#include "libqos/virtio.h"
> +
> +static void virtio_balloon_nop(void)
> +{
> + global_qtest = qtest_startf("-device virtio-balloon-ccw");
> + qtest_end();
> +}
> +
> +static void virtconsole_nop(void)
> +{
> + global_qtest = qtest_startf("-device virtio-serial-ccw,id=vser0 "
> + "-device virtconsole,bus=vser0.0");
> + qtest_end();
> +}
> +
> +static void virtserialport_nop(void)
> +{
> + global_qtest = qtest_startf("-device virtio-serial-ccw,id=vser0 "
> + "-device virtserialport,bus=vser0.0");
> + qtest_end();
> +}
> +
> +static void virtio_serial_nop(void)
> +{
> + global_qtest = qtest_startf("-device virtio-serial-ccw");
> + qtest_end();
> +}
> +
> +static void virtio_serial_hotplug(void)
> +{
> + global_qtest = qtest_startf("-device virtio-serial-ccw");
> + qtest_qmp_device_add("virtserialport", "hp-port", NULL);
> + qtest_qmp_device_del("hp-port");
> + qtest_end();
> +}
> +
> +static void virtio_blk_nop(void)
> +{
> + global_qtest = qtest_startf("-drive if=none,id=drv0,file=null-co://,format=raw "
> + "-device virtio-blk-ccw,drive=drv0");
> + qtest_end();
> +}
> +
> +static void virtio_net_nop(void)
> +{
> + global_qtest = qtest_startf("-device virtio-net-ccw");
> + qtest_end();
> +}
> +
> +static void virtio_9p_nop(void)
> +{
> + global_qtest = qtest_startf("-fsdev synth,id=fsdev0 "
> + "-device virtio-9p-ccw,fsdev=fsdev0,mount_tag=qtest");
> + qtest_end();
> +}
> +
> +static void virtio_rng_nop(void)
> +{
> + global_qtest = qtest_startf("-device virtio-rng-ccw");
> + qtest_end();
> +}
> +
> +static void virtio_scsi_nop(void)
> +{
> + global_qtest = qtest_startf("-device virtio-scsi-ccw");
> + qtest_end();
> +}
> +
> +static void virtio_scsi_hotplug(void)
> +{
> + global_qtest = qtest_startf("-drive if=none,id=drv0,file=null-co://,format=raw "
> + "-drive if=none,id=drv1,file=null-co://,format=raw "
> + "-device virtio-scsi-ccw "
> + "-device scsi-hd,drive=drv0");
> + qtest_qmp_device_add("scsi-hd", "scsihd", "'drive': 'drv1'");
> + qtest_qmp_device_del("scsihd");
> +
> + qtest_end();
> +}
> +
> +int main(int argc, char **argv)
> +{
> + int ret;
> +
> + g_test_init(&argc, &argv, NULL);
> + qtest_add_func("/virtio/balloon/nop", virtio_balloon_nop);
> + qtest_add_func("/virtio/console/nop", virtconsole_nop);
> + qtest_add_func("/virtio/serialport/nop", virtserialport_nop);
> + qtest_add_func("/virtio/serial/nop", virtio_serial_nop);
> + qtest_add_func("/virtio/serial/hotplug", virtio_serial_hotplug);
> + qtest_add_func("/virtio/block/nop", virtio_blk_nop);
> + qtest_add_func("/virtio/net/nop", virtio_net_nop);
> + qtest_add_func("/virtio/9p/nop", virtio_9p_nop);
> + qtest_add_func("/virtio/rng/nop", virtio_rng_nop);
> + qtest_add_func("/virtio/scsi/nop", virtio_scsi_nop);
> + qtest_add_func("/virtio/scsi/hotplug", virtio_scsi_hotplug);
> +
> + ret = g_test_run();
> +
> + return ret;
> +}
I think you could omit the virtio_net_nop test here, since we already
check that device in tests/pxe-test.c (and maybe also virtio_blk and
virtio_scsi since they are tested in tests/cdrom-test.c).
Anyway:
Reviewed-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [Qemu-devel] [qemu-s390x] [PATCH] tests: virtio: separate ccw tests from libqos
2018-08-01 16:39 ` [Qemu-devel] [qemu-s390x] " Thomas Huth
@ 2018-08-01 16:46 ` Paolo Bonzini
0 siblings, 0 replies; 4+ messages in thread
From: Paolo Bonzini @ 2018-08-01 16:46 UTC (permalink / raw)
To: Thomas Huth, qemu-devel; +Cc: qemu-s390x, e.emanuelegiuseppe
On 01/08/2018 18:39, Thomas Huth wrote:
> I think you could omit the virtio_net_nop test here, since we already
> check that device in tests/pxe-test.c (and maybe also virtio_blk and
> virtio_scsi since they are tested in tests/cdrom-test.c).
>
> Anyway:
>
> Reviewed-by: Thomas Huth <thuth@redhat.com>
Thanks Conny and Thomas for the quick review.
Emanuele, please add this patch to your qgraph branch! You can use "git
am -s" and cut-and-paste the original message source from thunderbird or
gmail into the Terminal.
Paolo
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-08-01 16:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-08-01 13:27 [Qemu-devel] [PATCH] tests: virtio: separate ccw tests from libqos Paolo Bonzini
2018-08-01 13:34 ` Cornelia Huck
2018-08-01 16:39 ` [Qemu-devel] [qemu-s390x] " Thomas Huth
2018-08-01 16:46 ` Paolo Bonzini
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).