* [PATCH v2] python, tests: do not use short-form boolean options
@ 2020-11-20 15:40 Paolo Bonzini
2020-12-07 19:19 ` John Snow
0 siblings, 1 reply; 2+ messages in thread
From: Paolo Bonzini @ 2020-11-20 15:40 UTC (permalink / raw)
To: qemu-devel; +Cc: armbru
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
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] python, tests: do not use short-form boolean options
2020-11-20 15:40 [PATCH v2] python, tests: do not use short-form boolean options Paolo Bonzini
@ 2020-12-07 19:19 ` John Snow
0 siblings, 0 replies; 2+ messages in thread
From: John Snow @ 2020-12-07 19:19 UTC (permalink / raw)
To: Paolo Bonzini, qemu-devel; +Cc: armbru
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();
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-12-07 19:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-20 15:40 [PATCH v2] python, tests: do not use short-form boolean options Paolo Bonzini
2020-12-07 19:19 ` John Snow
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).