[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 06/11] tests/qtest/qos-test: dump QEMU command if verbose
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v3 06/11] tests/qtest/qos-test: dump QEMU command if verbose |
Date: |
Thu, 8 Oct 2020 14:36:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 02/10/20 18:15, Christian Schoenebeck wrote:
> If qtests are run in verbose mode (i.e. if --verbose CL argument
> was provided) then print the assembled qemu command line for each
> test.
>
> Instead of using g_test_message() rather use printf() in combination
> with g_test_verbose(), to avoid g_test_message() cluttering the
> output.
>
> Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
> ---
> tests/qtest/qos-test.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/tests/qtest/qos-test.c b/tests/qtest/qos-test.c
> index fe240b32a7..b9f0942386 100644
> --- a/tests/qtest/qos-test.c
> +++ b/tests/qtest/qos-test.c
> @@ -89,6 +89,9 @@ static void qos_set_machines_devices_available(void)
>
> static void restart_qemu_or_continue(char *path)
> {
> + if (g_test_verbose()) {
> + 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
>
This need not even be made verbose I think.
Paolo
- [PATCH v3 00/11] 9pfs: add tests using local fs driver, Christian Schoenebeck, 2020/10/02
- [PATCH v3 06/11] tests/qtest/qos-test: dump QEMU command if verbose, Christian Schoenebeck, 2020/10/02
- Re: [PATCH v3 06/11] tests/qtest/qos-test: dump QEMU command if verbose,
Paolo Bonzini <=
- [PATCH v3 11/11] tests/9pfs: add local Tmkdir test, Christian Schoenebeck, 2020/10/02
- [PATCH v3 10/11] tests/9pfs: add virtio_9p_test_path(), Christian Schoenebeck, 2020/10/02
- [PATCH v3 09/11] tests/9pfs: wipe local 9pfs test directory, Christian Schoenebeck, 2020/10/02
- [PATCH v3 04/11] tests/qtest/qos-test: dump qos graph if verbose, Christian Schoenebeck, 2020/10/02
- [PATCH v3 07/11] tests/9pfs: change qtest name prefix to synth, Christian Schoenebeck, 2020/10/02
- [PATCH v3 02/11] libqos/qgraph: add qos_node_create_driver_named(), Christian Schoenebeck, 2020/10/02
- [PATCH v3 03/11] libqos/qgraph: add qos_dump_graph(), Christian Schoenebeck, 2020/10/02