[PATCH 2/3] tests/qtest/vnc-display-test: Use the 'none' machine

Philippe Mathieu-Daudé posted 3 patches 2 years, 11 months ago
Maintainers: Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
[PATCH 2/3] tests/qtest/vnc-display-test: Use the 'none' machine
Posted by Philippe Mathieu-Daudé 2 years, 11 months ago
If we don't specify any machine, an architecture default
might be picked. But some architectures don't provide any
default, such ARM:

  $ make check-qtest-aarch64
  ...
  19/20 qemu:qtest+qtest-aarch64 / qtest-aarch64/vnc-display-test
  qemu-system-aarch64: No machine specified, and there is no default

Since we don't need any particular machine to run this VNC
test, use the 'none' machine.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 tests/qtest/vnc-display-test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/qtest/vnc-display-test.c b/tests/qtest/vnc-display-test.c
index fd63e3a881..df468c7b22 100644
--- a/tests/qtest/vnc-display-test.c
+++ b/tests/qtest/vnc-display-test.c
@@ -44,7 +44,7 @@ test_setup(Test *test)
 #else
     int pair[2];
 
-    test->qts = qtest_init("-vnc none -name vnc-test");
+    test->qts = qtest_init("-M none -vnc none -name vnc-test");
 
     g_assert_cmpint(qemu_socketpair(AF_UNIX, SOCK_STREAM, 0, pair), ==, 0);
 
-- 
2.38.1


Re: [PATCH 2/3] tests/qtest/vnc-display-test: Use the 'none' machine
Posted by Richard Henderson 2 years, 11 months ago
On 1/19/23 02:05, Philippe Mathieu-Daudé wrote:
> If we don't specify any machine, an architecture default
> might be picked. But some architectures don't provide any
> default, such ARM:
> 
>    $ make check-qtest-aarch64
>    ...
>    19/20 qemu:qtest+qtest-aarch64 / qtest-aarch64/vnc-display-test
>    qemu-system-aarch64: No machine specified, and there is no default
> 
> Since we don't need any particular machine to run this VNC
> test, use the 'none' machine.
> 
> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
> ---
>   tests/qtest/vnc-display-test.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

Re: [PATCH 2/3] tests/qtest/vnc-display-test: Use the 'none' machine
Posted by Fabiano Rosas 2 years, 11 months ago
Philippe Mathieu-Daudé <philmd@linaro.org> writes:

> If we don't specify any machine, an architecture default
> might be picked. But some architectures don't provide any
> default, such ARM:
>
>   $ make check-qtest-aarch64
>   ...
>   19/20 qemu:qtest+qtest-aarch64 / qtest-aarch64/vnc-display-test
>   qemu-system-aarch64: No machine specified, and there is no default
>
> Since we don't need any particular machine to run this VNC
> test, use the 'none' machine.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Reviewed-by: Fabiano Rosas <farosas@suse.de>