[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 08/12] tests/9pfs: refactor test names and test devices
From: |
Christian Schoenebeck |
Subject: |
[PATCH 08/12] tests/9pfs: refactor test names and test devices |
Date: |
Sun, 27 Sep 2020 12:40:34 +0200 |
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
- [PATCH 12/12] tests/9pfs: add local Tmkdir test, (continued)
- [PATCH 12/12] tests/9pfs: add local Tmkdir test, Christian Schoenebeck, 2020/09/27
- [PATCH 06/12] tests/qtest/qos-test: add environment variable QTEST_DEBUG, Christian Schoenebeck, 2020/09/27
- [PATCH 07/12] test/9pfs: change export tag name to qtest-synth, Christian Schoenebeck, 2020/09/27
- [PATCH 04/12] tests/qtest/qos-test: new QTEST_DUMP_GRAPH environment variable, Christian Schoenebeck, 2020/09/27
- [PATCH 02/12] tests/qtest/qgraph: add qos_node_create_driver_named(), Christian Schoenebeck, 2020/09/27
- [PATCH 01/12] tests/qtest/qgraph: add qemu_name to QOSGraphNode, Christian Schoenebeck, 2020/09/27
- [PATCH 09/12] tests/9pfs: introduce local tests, Christian Schoenebeck, 2020/09/27
- [PATCH 03/12] tests/qtest/qos: add qos_dump_graph(), Christian Schoenebeck, 2020/09/27
- [PATCH 08/12] tests/9pfs: refactor test names and test devices,
Christian Schoenebeck <=
[PATCH 10/12] tests/9pfs: wipe local 9pfs test directory, Christian Schoenebeck, 2020/09/27
[PATCH 11/12] tests/9pfs: add virtio_9p_test_path(), Christian Schoenebeck, 2020/09/27