[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 09/15] tests/qtest: use qos_printf instead of g_test_message
From: |
Alex Bennée |
Subject: |
[PATCH v2 09/15] tests/qtest: use qos_printf instead of g_test_message |
Date: |
Tue, 24 May 2022 16:40:50 +0100 |
The vhost-user tests respawn qos-test as a standalone process. As a
result the gtester framework squashes all messages coming out of it
which make it hard to debug. As the test does not care about asserting
certain messages just convert the tests to use the direct qos_printf.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/qtest/qos-test.c | 5 +++++
tests/qtest/vhost-user-test.c | 13 +++++++------
2 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/tests/qtest/qos-test.c b/tests/qtest/qos-test.c
index 46623da731..fef2471f8f 100644
--- a/tests/qtest/qos-test.c
+++ b/tests/qtest/qos-test.c
@@ -320,6 +320,11 @@ static void walk_path(QOSGraphNode *orig_path, int len)
int main(int argc, char **argv, char** envp)
{
g_test_init(&argc, &argv, NULL);
+
+ if (g_test_subprocess()) {
+ qos_printf("qos_test running single test in subprocess\n");
+ }
+
if (g_test_verbose()) {
qos_printf("ENVIRONMENT VARIABLES: {\n");
for (char **env = envp; *env != 0; env++) {
diff --git a/tests/qtest/vhost-user-test.c b/tests/qtest/vhost-user-test.c
index a2cec87684..90749c147e 100644
--- a/tests/qtest/vhost-user-test.c
+++ b/tests/qtest/vhost-user-test.c
@@ -26,6 +26,7 @@
#include "libqos/virtio-pci.h"
#include "libqos/malloc-pc.h"
+#include "libqos/qgraph_internal.h"
#include "hw/virtio/virtio-net.h"
#include "standard-headers/linux/vhost_types.h"
@@ -316,7 +317,7 @@ static void chr_read(void *opaque, const uint8_t *buf, int
size)
}
if (size != VHOST_USER_HDR_SIZE) {
- g_test_message("Wrong message size received %d", size);
+ qos_printf("%s: Wrong message size received %d\n", __func__, size);
return;
}
@@ -327,8 +328,8 @@ static void chr_read(void *opaque, const uint8_t *buf, int
size)
p += VHOST_USER_HDR_SIZE;
size = qemu_chr_fe_read_all(chr, p, msg.size);
if (size != msg.size) {
- g_test_message("Wrong message size received %d != %d",
- size, msg.size);
+ qos_printf("%s: Wrong message size received %d != %d\n",
+ __func__, size, msg.size);
return;
}
}
@@ -450,7 +451,7 @@ static const char *init_hugepagefs(void)
}
if (access(path, R_OK | W_OK | X_OK)) {
- g_test_message("access on path (%s): %s", path, strerror(errno));
+ qos_printf("access on path (%s): %s", path, strerror(errno));
g_test_fail();
return NULL;
}
@@ -460,13 +461,13 @@ static const char *init_hugepagefs(void)
} while (ret != 0 && errno == EINTR);
if (ret != 0) {
- g_test_message("statfs on path (%s): %s", path, strerror(errno));
+ qos_printf("statfs on path (%s): %s", path, strerror(errno));
g_test_fail();
return NULL;
}
if (fs.f_type != HUGETLBFS_MAGIC) {
- g_test_message("Warning: path not on HugeTLBFS: %s", path);
+ qos_printf("Warning: path not on HugeTLBFS: %s", path);
g_test_fail();
return NULL;
}
--
2.30.2
- Re: [PATCH v2 02/15] include/hw/virtio: more comment for VIRTIO_F_BAD_FEATURE, (continued)
- [PATCH v2 01/15] contrib/vhost-user-blk: fix 32 bit build and enable, Alex Bennée, 2022/05/24
- [PATCH v2 06/15] hw/virtio: add vhost-user-gpio-pci boilerplate, Alex Bennée, 2022/05/24
- [PATCH v2 07/15] tests/qtest: pass stdout/stderr down to subtests, Alex Bennée, 2022/05/24
- [PATCH v2 05/15] hw/virtio: add boilerplate for vhost-user-gpio device, Alex Bennée, 2022/05/24
- [PATCH v2 11/15] tests/qtest: use g_autofree for test_server_create_chr, Alex Bennée, 2022/05/24
- [PATCH v2 10/15] tests/qtest: catch unhandled vhost-user messages, Alex Bennée, 2022/05/24
- [PATCH v2 13/15] tests/qtest: implement stub for VHOST_USER_GET_CONFIG, Alex Bennée, 2022/05/24
- [PATCH v2 09/15] tests/qtest: use qos_printf instead of g_test_message,
Alex Bennée <=
- [PATCH v2 12/15] tests/qtest: plain g_assert for VHOST_USER_F_PROTOCOL_FEATURES, Alex Bennée, 2022/05/24
- [PATCH v2 04/15] include/hw/virtio: document vhost_ack_features, Alex Bennée, 2022/05/24
- [PATCH v2 15/15] tests/qtest: enable tests for virtio-gpio, Alex Bennée, 2022/05/24
- [PATCH v2 14/15] tests/qtest: add a get_features op to vhost-user-test, Alex Bennée, 2022/05/24
- Re: [PATCH v2 00/15] virtio-gpio and various virtio cleanups, Stefan Hajnoczi, 2022/05/25