[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH] tests/qtest: pass stdout/stderr down to subtests
From: |
Eric Auger |
Subject: |
Re: [RFC PATCH] tests/qtest: pass stdout/stderr down to subtests |
Date: |
Thu, 14 Apr 2022 19:25:54 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
Hi Alex,
On 4/7/22 5:00 PM, Alex Bennée wrote:
> When trying to work out what the virtio-net-tests where doing it was
> hard because the g_test_trap_subprocess redirects all output to
> /dev/null. Lift this restriction by using the appropriate flags so you
> can see something similar to what the vhost-user-blk tests show when
> running.
>
> While we are at it remove the g_test_verbose() check so we always show
> how the QEMU is run.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/qtest/qos-test.c | 7 +++----
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/tests/qtest/qos-test.c b/tests/qtest/qos-test.c
> index f97d0a08fd..c6c196cc95 100644
> --- a/tests/qtest/qos-test.c
> +++ b/tests/qtest/qos-test.c
> @@ -89,9 +89,7 @@ static void qos_set_machines_devices_available(void)
>
> static void restart_qemu_or_continue(char *path)
> {
> - if (g_test_verbose()) {
> - qos_printf("Run QEMU with: '%s'\n", path);
> - }
> + qos_printf("Run QEMU with: '%s'\n", path);
> /* compares the current command line with the
> * one previously executed: if they are the same,
> * don't restart QEMU, if they differ, stop previous
> @@ -185,7 +183,8 @@ static void run_one_test(const void *arg)
> static void subprocess_run_one_test(const void *arg)
> {
> const gchar *path = arg;
> - g_test_trap_subprocess(path, 0, 0);
> + g_test_trap_subprocess(path, 0,
> + G_TEST_SUBPROCESS_INHERIT_STDOUT |
> G_TEST_SUBPROCESS_INHERIT_STDERR);
While workling on libqos/pci tests on aarch64 I also did that but I
noticed there were a bunch of errors such as:
/aarch64/virt/generic-pcihost/pci-bus-generic/pci-bus/virtio-net-pci/virtio-net/virtio-net-tests/vhost-user/multiqueue:
qemu-system-aarch64: Failed to set msg fds.
qemu-system-aarch64: vhost VQ 0 ring restore failed: -22: Invalid
argument (22)
qemu-system-aarch64: Failed to set msg fds.
qemu-system-aarch64: vhost VQ 1 ring restore failed: -22: Invalid
argument (22)
qemu-system-aarch64: Failed to set msg fds.
qemu-system-aarch64: vhost VQ 2 ring restore failed: -22: Invalid
argument (22)
qemu-system-aarch64: Failed to set msg fds.
qemu-system-aarch64: vhost VQ 3 ring restore failed: -22: Invalid
argument (22)
I see those also when running with x86_64-softmmu/qemu-system-x86_64
(this is no aarch64 specific).
I don't know if it is an issue to get those additional errors?
Thanks
Eric
> g_test_trap_assert_passed();
> }
>
>