From: Christian Schoenebeck <qemu_oss@crudebyte.com>
To: qemu-devel@nongnu.org
Cc: Thomas Huth <thuth@redhat.com>,
Laurent Vivier <lvivier@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
Emanuele Giuseppe Esposito <e.emanuelegiuseppe@gmail.com>,
Greg Kurz <groug@kaod.org>
Subject: [PATCH 08/12] tests/9pfs: refactor test names and test devices
Date: Sun, 27 Sep 2020 12:40:34 +0200 [thread overview]
Message-ID: <265f5d9a0fb10ce5e782455839d1baf678dbac48.1601203436.git.qemu_oss@crudebyte.com> (raw)
In-Reply-To: <cover.1601203436.git.qemu_oss@crudebyte.com>
Rename all 9pfs tests and devices they create for running their tests
from 'virtio*' -> 'virtio*-synth'.
In order for the tests still to work after this renaming, use the
newly added function qos_node_create_driver_named() instead of
qos_node_create_driver(). That new function allows to assign a name
to a device that differs from the actual QEMU driver it's using.
Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
---
tests/qtest/libqos/virtio-9p.c | 29 ++++++++++++++-------------
tests/qtest/virtio-9p-test.c | 36 +++++++++++++++++-----------------
2 files changed, 33 insertions(+), 32 deletions(-)
diff --git a/tests/qtest/libqos/virtio-9p.c b/tests/qtest/libqos/virtio-9p.c
index 9f099737f9..1bda5403ff 100644
--- a/tests/qtest/libqos/virtio-9p.c
+++ b/tests/qtest/libqos/virtio-9p.c
@@ -62,10 +62,10 @@ static void virtio_9p_device_start_hw(QOSGraphObject *obj)
static void *virtio_9p_get_driver(QVirtio9P *v_9p,
const char *interface)
{
- if (!g_strcmp0(interface, "virtio-9p")) {
+ if (!g_strcmp0(interface, "virtio-9p-synth")) {
return v_9p;
}
- if (!g_strcmp0(interface, "virtio")) {
+ if (!g_strcmp0(interface, "virtio-synth")) {
return v_9p->vdev;
}
@@ -159,22 +159,23 @@ static void virtio_9p_register_nodes(void)
.before_cmd_line = "-fsdev synth,id=fsdev0",
};
- /* virtio-9p-device */
+ /* virtio-9p-device-synth */
opts.extra_device_opts = str_simple,
- qos_node_create_driver("virtio-9p-device", virtio_9p_device_create);
- qos_node_consumes("virtio-9p-device", "virtio-bus", &opts);
- qos_node_produces("virtio-9p-device", "virtio");
- qos_node_produces("virtio-9p-device", "virtio-9p");
+ qos_node_create_driver_named("virtio-9p-device-synth", "virtio-9p-device",
+ virtio_9p_device_create);
+ qos_node_consumes("virtio-9p-device-synth", "virtio-bus", &opts);
+ qos_node_produces("virtio-9p-device-synth", "virtio-synth");
+ qos_node_produces("virtio-9p-device-synth", "virtio-9p-synth");
- /* virtio-9p-pci */
+ /* virtio-9p-pci-synth */
opts.extra_device_opts = str_addr;
add_qpci_address(&opts, &addr);
- qos_node_create_driver("virtio-9p-pci", virtio_9p_pci_create);
- qos_node_consumes("virtio-9p-pci", "pci-bus", &opts);
- qos_node_produces("virtio-9p-pci", "pci-device");
- qos_node_produces("virtio-9p-pci", "virtio");
- qos_node_produces("virtio-9p-pci", "virtio-9p");
-
+ qos_node_create_driver_named("virtio-9p-pci-synth", "virtio-9p-pci",
+ virtio_9p_pci_create);
+ qos_node_consumes("virtio-9p-pci-synth", "pci-bus", &opts);
+ qos_node_produces("virtio-9p-pci-synth", "pci-device");
+ qos_node_produces("virtio-9p-pci-synth", "virtio-synth");
+ qos_node_produces("virtio-9p-pci-synth", "virtio-9p-synth");
}
libqos_init(virtio_9p_register_nodes);
diff --git a/tests/qtest/virtio-9p-test.c b/tests/qtest/virtio-9p-test.c
index f7505396f3..d46d675309 100644
--- a/tests/qtest/virtio-9p-test.c
+++ b/tests/qtest/virtio-9p-test.c
@@ -897,27 +897,27 @@ static void fs_readdir_split_512(void *obj, void *data,
static void register_virtio_9p_test(void)
{
- qos_add_test("config", "virtio-9p", pci_config, NULL);
- qos_add_test("fs/version/basic", "virtio-9p", fs_version, NULL);
- qos_add_test("fs/attach/basic", "virtio-9p", fs_attach, NULL);
- qos_add_test("fs/walk/basic", "virtio-9p", fs_walk, NULL);
- qos_add_test("fs/walk/no_slash", "virtio-9p", fs_walk_no_slash,
+ /* selects the 9pfs 'synth' filesystem driver for the respective test */
+ const char *synth_driver = "virtio-9p-synth";
+
+ qos_add_test("config", synth_driver, pci_config, NULL);
+ qos_add_test("fs/version/basic", synth_driver, fs_version, NULL);
+ qos_add_test("fs/attach/basic", synth_driver, fs_attach, NULL);
+ qos_add_test("fs/walk/basic", synth_driver, fs_walk, NULL);
+ qos_add_test("fs/walk/no_slash", synth_driver, fs_walk_no_slash, NULL);
+ qos_add_test("fs/walk/dotdot_from_root", synth_driver, fs_walk_dotdot,
NULL);
- qos_add_test("fs/walk/dotdot_from_root", "virtio-9p",
- fs_walk_dotdot, NULL);
- qos_add_test("fs/lopen/basic", "virtio-9p", fs_lopen, NULL);
- qos_add_test("fs/write/basic", "virtio-9p", fs_write, NULL);
- qos_add_test("fs/flush/success", "virtio-9p", fs_flush_success,
+ qos_add_test("fs/lopen/basic", synth_driver, fs_lopen, NULL);
+ qos_add_test("fs/write/basic", synth_driver, fs_write, NULL);
+ qos_add_test("fs/flush/success", synth_driver, fs_flush_success, NULL);
+ qos_add_test("fs/flush/ignored", synth_driver, fs_flush_ignored, NULL);
+ qos_add_test("fs/readdir/basic", synth_driver, fs_readdir, NULL);
+ qos_add_test("fs/readdir/split_512", synth_driver, fs_readdir_split_512,
NULL);
- qos_add_test("fs/flush/ignored", "virtio-9p", fs_flush_ignored,
+ qos_add_test("fs/readdir/split_256", synth_driver, fs_readdir_split_256,
+ NULL);
+ qos_add_test("fs/readdir/split_128", synth_driver, fs_readdir_split_128,
NULL);
- qos_add_test("fs/readdir/basic", "virtio-9p", fs_readdir, NULL);
- qos_add_test("fs/readdir/split_512", "virtio-9p",
- fs_readdir_split_512, NULL);
- qos_add_test("fs/readdir/split_256", "virtio-9p",
- fs_readdir_split_256, NULL);
- qos_add_test("fs/readdir/split_128", "virtio-9p",
- fs_readdir_split_128, NULL);
}
libqos_init(register_virtio_9p_test);
--
2.20.1
next prev parent reply other threads:[~2020-09-27 12:21 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-27 10:43 [PATCH 00/12] 9pfs: add tests using local fs driver Christian Schoenebeck
2020-09-27 10:38 ` [PATCH 01/12] tests/qtest/qgraph: add qemu_name to QOSGraphNode Christian Schoenebeck
2020-09-27 10:39 ` [PATCH 02/12] tests/qtest/qgraph: add qos_node_create_driver_named() Christian Schoenebeck
2020-09-27 10:39 ` [PATCH 03/12] tests/qtest/qos: add qos_dump_graph() Christian Schoenebeck
2020-09-27 10:39 ` [PATCH 04/12] tests/qtest/qos-test: new QTEST_DUMP_GRAPH environment variable Christian Schoenebeck
2020-09-27 10:39 ` [PATCH 05/12] tests/qtest/qos-test: add QTEST_DUMP_ENV " Christian Schoenebeck
2020-09-27 10:40 ` [PATCH 06/12] tests/qtest/qos-test: add environment variable QTEST_DEBUG Christian Schoenebeck
2020-09-28 8:31 ` Paolo Bonzini
2020-09-28 12:11 ` Christian Schoenebeck
2020-09-27 10:40 ` [PATCH 07/12] test/9pfs: change export tag name to qtest-synth Christian Schoenebeck
2020-09-27 10:40 ` Christian Schoenebeck [this message]
2020-09-28 8:36 ` [PATCH 08/12] tests/9pfs: refactor test names and test devices Paolo Bonzini
2020-09-28 8:37 ` Paolo Bonzini
2020-09-28 11:56 ` Christian Schoenebeck
2020-09-28 12:42 ` Paolo Bonzini
2020-09-28 13:35 ` Christian Schoenebeck
2020-09-28 16:38 ` Paolo Bonzini
2020-10-01 11:34 ` Christian Schoenebeck
2020-10-01 11:56 ` Paolo Bonzini
2020-10-01 12:15 ` Christian Schoenebeck
2020-10-01 14:04 ` Paolo Bonzini
2020-10-01 15:26 ` Christian Schoenebeck
2020-09-27 10:40 ` [PATCH 09/12] tests/9pfs: introduce local tests Christian Schoenebeck
2020-09-27 10:41 ` [PATCH 10/12] tests/9pfs: wipe local 9pfs test directory Christian Schoenebeck
2020-09-27 10:41 ` [PATCH 11/12] tests/9pfs: add virtio_9p_test_path() Christian Schoenebeck
2020-09-27 10:41 ` [PATCH 12/12] tests/9pfs: add local Tmkdir test Christian Schoenebeck
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=265f5d9a0fb10ce5e782455839d1baf678dbac48.1601203436.git.qemu_oss@crudebyte.com \
--to=qemu_oss@crudebyte.com \
--cc=e.emanuelegiuseppe@gmail.com \
--cc=groug@kaod.org \
--cc=lvivier@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.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).