[PATCH v2] python, tests: do not use short-form boolean options

Paolo Bonzini posted 1 patch 3 years, 3 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201120154051.2495221-1-pbonzini@redhat.com
Maintainers: Thomas Huth <thuth@redhat.com>, Cleber Rosa <crosa@redhat.com>, Laurent Vivier <lvivier@redhat.com>, John Snow <jsnow@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Eduardo Habkost <ehabkost@redhat.com>
python/qemu/machine.py               | 2 +-
tests/qtest/pflash-cfi02-test.c      | 4 ++--
tests/qtest/test-filter-redirector.c | 8 ++++----
tests/qtest/vhost-user-test.c        | 8 ++++----
tests/test-char.c                    | 8 ++++----
5 files changed, 15 insertions(+), 15 deletions(-)
[PATCH v2] python, tests: do not use short-form boolean options
Posted by Paolo Bonzini 3 years, 3 months ago
They are going to be deprecated, avoid warnings on stdout while the
tests run.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 python/qemu/machine.py               | 2 +-
 tests/qtest/pflash-cfi02-test.c      | 4 ++--
 tests/qtest/test-filter-redirector.c | 8 ++++----
 tests/qtest/vhost-user-test.c        | 8 ++++----
 tests/test-char.c                    | 8 ++++----
 5 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/python/qemu/machine.py b/python/qemu/machine.py
index 6420f01bed..6216530b0d 100644
--- a/python/qemu/machine.py
+++ b/python/qemu/machine.py
@@ -292,7 +292,7 @@ class QEMUMachine:
         for _ in range(self._console_index):
             args.extend(['-serial', 'null'])
         if self._console_set:
-            chardev = ('socket,id=console,path=%s,server,nowait' %
+            chardev = ('socket,id=console,path=%s,server=on,wait=off' %
                        self._console_address)
             args.extend(['-chardev', chardev])
             if self._console_device_type is None:
diff --git a/tests/qtest/pflash-cfi02-test.c b/tests/qtest/pflash-cfi02-test.c
index afb702b565..60db81a3a2 100644
--- a/tests/qtest/pflash-cfi02-test.c
+++ b/tests/qtest/pflash-cfi02-test.c
@@ -261,7 +261,7 @@ static void test_geometry(const void *opaque)
     const FlashConfig *config = opaque;
     QTestState *qtest;
     qtest = qtest_initf("-M musicpal"
-                        " -drive if=pflash,file=%s,format=raw,copy-on-read"
+                        " -drive if=pflash,file=%s,format=raw,copy-on-read=on"
                         /* Device geometry properties. */
                         " -global driver=cfi.pflash02,"
                         "property=num-blocks0,value=%d"
@@ -581,7 +581,7 @@ static void test_cfi_in_autoselect(const void *opaque)
     const FlashConfig *config = opaque;
     QTestState *qtest;
     qtest = qtest_initf("-M musicpal"
-                        " -drive if=pflash,file=%s,format=raw,copy-on-read",
+                        " -drive if=pflash,file=%s,format=raw,copy-on-read=on",
                         image_path);
     FlashConfig explicit_config = expand_config_defaults(config);
     explicit_config.qtest = qtest;
diff --git a/tests/qtest/test-filter-redirector.c b/tests/qtest/test-filter-redirector.c
index 829db8c2ea..4269b2cdd9 100644
--- a/tests/qtest/test-filter-redirector.c
+++ b/tests/qtest/test-filter-redirector.c
@@ -95,8 +95,8 @@ static void test_redirector_tx(void)
     qts = qtest_initf(
         "-netdev socket,id=qtest-bn0,fd=%d "
         "-device %s,netdev=qtest-bn0,id=qtest-e0 "
-        "-chardev socket,id=redirector0,path=%s,server,nowait "
-        "-chardev socket,id=redirector1,path=%s,server,nowait "
+        "-chardev socket,id=redirector0,path=%s,server=on,wait=off "
+        "-chardev socket,id=redirector1,path=%s,server=on,wait=off "
         "-chardev socket,id=redirector2,path=%s "
         "-object filter-redirector,id=qtest-f0,netdev=qtest-bn0,"
         "queue=tx,outdev=redirector0 "
@@ -165,8 +165,8 @@ static void test_redirector_rx(void)
     qts = qtest_initf(
         "-netdev socket,id=qtest-bn0,fd=%d "
         "-device %s,netdev=qtest-bn0,id=qtest-e0 "
-        "-chardev socket,id=redirector0,path=%s,server,nowait "
-        "-chardev socket,id=redirector1,path=%s,server,nowait "
+        "-chardev socket,id=redirector0,path=%s,server=on,wait=off "
+        "-chardev socket,id=redirector1,path=%s,server=on,wait=off "
         "-chardev socket,id=redirector2,path=%s "
         "-object filter-redirector,id=qtest-f0,netdev=qtest-bn0,"
         "queue=rx,indev=redirector0 "
diff --git a/tests/qtest/vhost-user-test.c b/tests/qtest/vhost-user-test.c
index 3df5322614..1a5f5313ff 100644
--- a/tests/qtest/vhost-user-test.c
+++ b/tests/qtest/vhost-user-test.c
@@ -537,7 +537,7 @@ static void test_server_create_chr(TestServer *server, const gchar *opt)
 
 static void test_server_listen(TestServer *server)
 {
-    test_server_create_chr(server, ",server,nowait");
+    test_server_create_chr(server, ",server=on,wait=off");
 }
 
 static void test_server_free(TestServer *server)
@@ -846,7 +846,7 @@ static void *vhost_user_test_setup_reconnect(GString *cmd_line, void *arg)
 
     g_thread_new("connect", connect_thread, s);
     append_mem_opts(s, cmd_line, 256, TEST_MEMFD_AUTO);
-    s->vu_ops->append_opts(s, cmd_line, ",server");
+    s->vu_ops->append_opts(s, cmd_line, ",server=on");
 
     g_test_queue_destroy(vhost_user_test_cleanup, s);
 
@@ -883,7 +883,7 @@ static void *vhost_user_test_setup_connect_fail(GString *cmd_line, void *arg)
 
     g_thread_new("connect", connect_thread, s);
     append_mem_opts(s, cmd_line, 256, TEST_MEMFD_AUTO);
-    s->vu_ops->append_opts(s, cmd_line, ",server");
+    s->vu_ops->append_opts(s, cmd_line, ",server=on");
 
     g_test_queue_destroy(vhost_user_test_cleanup, s);
 
@@ -898,7 +898,7 @@ static void *vhost_user_test_setup_flags_mismatch(GString *cmd_line, void *arg)
 
     g_thread_new("connect", connect_thread, s);
     append_mem_opts(s, cmd_line, 256, TEST_MEMFD_AUTO);
-    s->vu_ops->append_opts(s, cmd_line, ",server");
+    s->vu_ops->append_opts(s, cmd_line, ",server=on");
 
     g_test_queue_destroy(vhost_user_test_cleanup, s);
 
diff --git a/tests/test-char.c b/tests/test-char.c
index 9196e566e9..953e0d1c1f 100644
--- a/tests/test-char.c
+++ b/tests/test-char.c
@@ -413,7 +413,7 @@ static void char_websock_test(void)
     CharBackend client_be;
     Chardev *chr_client;
     Chardev *chr = qemu_chr_new("server",
-                                "websocket:127.0.0.1:0,server,nowait", NULL);
+                                "websocket:127.0.0.1:0,server=on,wait=off", NULL);
     const char handshake[] = "GET / HTTP/1.1\r\n"
                              "Upgrade: websocket\r\n"
                              "Connection: Upgrade\r\n"
@@ -696,7 +696,7 @@ char_socket_addr_to_opt_str(SocketAddress *addr, bool fd_pass,
         fd = ioc->fd;
         ioc->fd = -1;
         optstr = g_strdup_printf("socket,id=cdev0,fd=%d%s",
-                                 fd, is_listen ? ",server,nowait" : "");
+                                 fd, is_listen ? ",server=on,wait=off" : "");
         object_unref(OBJECT(ioc));
         return optstr;
     } else {
@@ -706,13 +706,13 @@ char_socket_addr_to_opt_str(SocketAddress *addr, bool fd_pass,
                                    addr->u.inet.host,
                                    addr->u.inet.port,
                                    reconnect ? reconnect : "",
-                                   is_listen ? ",server,nowait" : "");
+                                   is_listen ? ",server=on,wait=off" : "");
 
         case SOCKET_ADDRESS_TYPE_UNIX:
             return g_strdup_printf("socket,id=cdev0,path=%s%s%s",
                                    addr->u.q_unix.path,
                                    reconnect ? reconnect : "",
-                                   is_listen ? ",server,nowait" : "");
+                                   is_listen ? ",server=on,wait=off" : "");
 
         default:
             g_assert_not_reached();
-- 
2.26.2


Re: [PATCH v2] python, tests: do not use short-form boolean options
Posted by John Snow 3 years, 3 months ago
On 11/20/20 10:40 AM, Paolo Bonzini wrote:
> They are going to be deprecated, avoid warnings on stdout while the
> tests run.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>   python/qemu/machine.py               | 2 +-

Acked-by: John Snow <jsnow@redhat.com>

>   tests/qtest/pflash-cfi02-test.c      | 4 ++--
>   tests/qtest/test-filter-redirector.c | 8 ++++----
>   tests/qtest/vhost-user-test.c        | 8 ++++----
>   tests/test-char.c                    | 8 ++++----
>   5 files changed, 15 insertions(+), 15 deletions(-)
> 
> diff --git a/python/qemu/machine.py b/python/qemu/machine.py
> index 6420f01bed..6216530b0d 100644
> --- a/python/qemu/machine.py
> +++ b/python/qemu/machine.py
> @@ -292,7 +292,7 @@ class QEMUMachine:
>           for _ in range(self._console_index):
>               args.extend(['-serial', 'null'])
>           if self._console_set:
> -            chardev = ('socket,id=console,path=%s,server,nowait' %
> +            chardev = ('socket,id=console,path=%s,server=on,wait=off' %
>                          self._console_address)
>               args.extend(['-chardev', chardev])
>               if self._console_device_type is None:
> diff --git a/tests/qtest/pflash-cfi02-test.c b/tests/qtest/pflash-cfi02-test.c
> index afb702b565..60db81a3a2 100644
> --- a/tests/qtest/pflash-cfi02-test.c
> +++ b/tests/qtest/pflash-cfi02-test.c
> @@ -261,7 +261,7 @@ static void test_geometry(const void *opaque)
>       const FlashConfig *config = opaque;
>       QTestState *qtest;
>       qtest = qtest_initf("-M musicpal"
> -                        " -drive if=pflash,file=%s,format=raw,copy-on-read"
> +                        " -drive if=pflash,file=%s,format=raw,copy-on-read=on"
>                           /* Device geometry properties. */
>                           " -global driver=cfi.pflash02,"
>                           "property=num-blocks0,value=%d"
> @@ -581,7 +581,7 @@ static void test_cfi_in_autoselect(const void *opaque)
>       const FlashConfig *config = opaque;
>       QTestState *qtest;
>       qtest = qtest_initf("-M musicpal"
> -                        " -drive if=pflash,file=%s,format=raw,copy-on-read",
> +                        " -drive if=pflash,file=%s,format=raw,copy-on-read=on",
>                           image_path);
>       FlashConfig explicit_config = expand_config_defaults(config);
>       explicit_config.qtest = qtest;
> diff --git a/tests/qtest/test-filter-redirector.c b/tests/qtest/test-filter-redirector.c
> index 829db8c2ea..4269b2cdd9 100644
> --- a/tests/qtest/test-filter-redirector.c
> +++ b/tests/qtest/test-filter-redirector.c
> @@ -95,8 +95,8 @@ static void test_redirector_tx(void)
>       qts = qtest_initf(
>           "-netdev socket,id=qtest-bn0,fd=%d "
>           "-device %s,netdev=qtest-bn0,id=qtest-e0 "
> -        "-chardev socket,id=redirector0,path=%s,server,nowait "
> -        "-chardev socket,id=redirector1,path=%s,server,nowait "
> +        "-chardev socket,id=redirector0,path=%s,server=on,wait=off "
> +        "-chardev socket,id=redirector1,path=%s,server=on,wait=off "
>           "-chardev socket,id=redirector2,path=%s "
>           "-object filter-redirector,id=qtest-f0,netdev=qtest-bn0,"
>           "queue=tx,outdev=redirector0 "
> @@ -165,8 +165,8 @@ static void test_redirector_rx(void)
>       qts = qtest_initf(
>           "-netdev socket,id=qtest-bn0,fd=%d "
>           "-device %s,netdev=qtest-bn0,id=qtest-e0 "
> -        "-chardev socket,id=redirector0,path=%s,server,nowait "
> -        "-chardev socket,id=redirector1,path=%s,server,nowait "
> +        "-chardev socket,id=redirector0,path=%s,server=on,wait=off "
> +        "-chardev socket,id=redirector1,path=%s,server=on,wait=off "
>           "-chardev socket,id=redirector2,path=%s "
>           "-object filter-redirector,id=qtest-f0,netdev=qtest-bn0,"
>           "queue=rx,indev=redirector0 "
> diff --git a/tests/qtest/vhost-user-test.c b/tests/qtest/vhost-user-test.c
> index 3df5322614..1a5f5313ff 100644
> --- a/tests/qtest/vhost-user-test.c
> +++ b/tests/qtest/vhost-user-test.c
> @@ -537,7 +537,7 @@ static void test_server_create_chr(TestServer *server, const gchar *opt)
>   
>   static void test_server_listen(TestServer *server)
>   {
> -    test_server_create_chr(server, ",server,nowait");
> +    test_server_create_chr(server, ",server=on,wait=off");
>   }
>   
>   static void test_server_free(TestServer *server)
> @@ -846,7 +846,7 @@ static void *vhost_user_test_setup_reconnect(GString *cmd_line, void *arg)
>   
>       g_thread_new("connect", connect_thread, s);
>       append_mem_opts(s, cmd_line, 256, TEST_MEMFD_AUTO);
> -    s->vu_ops->append_opts(s, cmd_line, ",server");
> +    s->vu_ops->append_opts(s, cmd_line, ",server=on");
>   
>       g_test_queue_destroy(vhost_user_test_cleanup, s);
>   
> @@ -883,7 +883,7 @@ static void *vhost_user_test_setup_connect_fail(GString *cmd_line, void *arg)
>   
>       g_thread_new("connect", connect_thread, s);
>       append_mem_opts(s, cmd_line, 256, TEST_MEMFD_AUTO);
> -    s->vu_ops->append_opts(s, cmd_line, ",server");
> +    s->vu_ops->append_opts(s, cmd_line, ",server=on");
>   
>       g_test_queue_destroy(vhost_user_test_cleanup, s);
>   
> @@ -898,7 +898,7 @@ static void *vhost_user_test_setup_flags_mismatch(GString *cmd_line, void *arg)
>   
>       g_thread_new("connect", connect_thread, s);
>       append_mem_opts(s, cmd_line, 256, TEST_MEMFD_AUTO);
> -    s->vu_ops->append_opts(s, cmd_line, ",server");
> +    s->vu_ops->append_opts(s, cmd_line, ",server=on");
>   
>       g_test_queue_destroy(vhost_user_test_cleanup, s);
>   
> diff --git a/tests/test-char.c b/tests/test-char.c
> index 9196e566e9..953e0d1c1f 100644
> --- a/tests/test-char.c
> +++ b/tests/test-char.c
> @@ -413,7 +413,7 @@ static void char_websock_test(void)
>       CharBackend client_be;
>       Chardev *chr_client;
>       Chardev *chr = qemu_chr_new("server",
> -                                "websocket:127.0.0.1:0,server,nowait", NULL);
> +                                "websocket:127.0.0.1:0,server=on,wait=off", NULL);
>       const char handshake[] = "GET / HTTP/1.1\r\n"
>                                "Upgrade: websocket\r\n"
>                                "Connection: Upgrade\r\n"
> @@ -696,7 +696,7 @@ char_socket_addr_to_opt_str(SocketAddress *addr, bool fd_pass,
>           fd = ioc->fd;
>           ioc->fd = -1;
>           optstr = g_strdup_printf("socket,id=cdev0,fd=%d%s",
> -                                 fd, is_listen ? ",server,nowait" : "");
> +                                 fd, is_listen ? ",server=on,wait=off" : "");
>           object_unref(OBJECT(ioc));
>           return optstr;
>       } else {
> @@ -706,13 +706,13 @@ char_socket_addr_to_opt_str(SocketAddress *addr, bool fd_pass,
>                                      addr->u.inet.host,
>                                      addr->u.inet.port,
>                                      reconnect ? reconnect : "",
> -                                   is_listen ? ",server,nowait" : "");
> +                                   is_listen ? ",server=on,wait=off" : "");
>   
>           case SOCKET_ADDRESS_TYPE_UNIX:
>               return g_strdup_printf("socket,id=cdev0,path=%s%s%s",
>                                      addr->u.q_unix.path,
>                                      reconnect ? reconnect : "",
> -                                   is_listen ? ",server,nowait" : "");
> +                                   is_listen ? ",server=on,wait=off" : "");
>   
>           default:
>               g_assert_not_reached();
>