[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 4/6] Revert "tests: Add parameter to qtest_in
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v8 4/6] Revert "tests: Add parameter to qtest_init_without_qmp_handshake" |
Date: |
Fri, 28 Sep 2018 16:19:06 +0400 |
Hi
On Wed, Sep 5, 2018 at 10:26 AM Peter Xu <address@hidden> wrote:
>
> This reverts commit ddee57e0176f6ab53b13c6c97605b62737a8fd7a.
>
> Meanwhile, revert one line from fa198ad9bdef to make sure
> qtest_init_without_qmp_handshake() will only pass in one parameter.
>
> Reviewed-by: Markus Armbruster <address@hidden>
> Signed-off-by: Peter Xu <address@hidden>
> ---
Reviewed-by: Marc-André Lureau <address@hidden>
> tests/libqtest.c | 10 ++++------
> tests/libqtest.h | 4 +---
> tests/qmp-test.c | 4 ++--
> 3 files changed, 7 insertions(+), 11 deletions(-)
>
> diff --git a/tests/libqtest.c b/tests/libqtest.c
> index ebd92f22f6..3c594abbc2 100644
> --- a/tests/libqtest.c
> +++ b/tests/libqtest.c
> @@ -191,8 +191,7 @@ static const char *qtest_qemu_binary(void)
> return qemu_bin;
> }
>
> -QTestState *qtest_init_without_qmp_handshake(bool use_oob,
> - const char *extra_args)
> +QTestState *qtest_init_without_qmp_handshake(const char *extra_args)
> {
> QTestState *s;
> int sock, qmpsock, i;
> @@ -225,13 +224,12 @@ QTestState *qtest_init_without_qmp_handshake(bool
> use_oob,
> command = g_strdup_printf("exec %s "
> "-qtest unix:%s,nowait "
> "-qtest-log %s "
> - "-chardev socket,path=%s,nowait,id=char0 "
> - "-mon chardev=char0,mode=control%s "
> + "-qmp unix:%s,nowait "
> "-machine accel=qtest "
> "-display none "
> "%s", qemu_binary, socket_path,
> getenv("QTEST_LOG") ? "/dev/fd/2" :
> "/dev/null",
> - qmp_socket_path, "",
> + qmp_socket_path,
> extra_args ?: "");
> execlp("/bin/sh", "sh", "-c", command, NULL);
> exit(1);
> @@ -266,7 +264,7 @@ QTestState *qtest_init_without_qmp_handshake(bool use_oob,
>
> QTestState *qtest_init(const char *extra_args)
> {
> - QTestState *s = qtest_init_without_qmp_handshake(false, extra_args);
> + QTestState *s = qtest_init_without_qmp_handshake(extra_args);
> QDict *greeting;
>
> /* Read the QMP greeting and then do the handshake */
> diff --git a/tests/libqtest.h b/tests/libqtest.h
> index 36d5caecd4..49ffc1ba9f 100644
> --- a/tests/libqtest.h
> +++ b/tests/libqtest.h
> @@ -55,14 +55,12 @@ QTestState *qtest_init(const char *extra_args);
>
> /**
> * qtest_init_without_qmp_handshake:
> - * @use_oob: true to have the server advertise OOB support
> * @extra_args: other arguments to pass to QEMU. CAUTION: these
> * arguments are subject to word splitting and shell evaluation.
> *
> * Returns: #QTestState instance.
> */
> -QTestState *qtest_init_without_qmp_handshake(bool use_oob,
> - const char *extra_args);
> +QTestState *qtest_init_without_qmp_handshake(const char *extra_args);
>
> /**
> * qtest_quit:
> diff --git a/tests/qmp-test.c b/tests/qmp-test.c
> index 5302bd07b9..91a90d1c9d 100644
> --- a/tests/qmp-test.c
> +++ b/tests/qmp-test.c
> @@ -135,7 +135,7 @@ static void test_qmp_protocol(void)
> QList *capabilities;
> QTestState *qts;
>
> - qts = qtest_init_without_qmp_handshake(false, common_args);
> + qts = qtest_init_without_qmp_handshake(common_args);
>
> /* Test greeting */
> resp = qtest_qmp_receive(qts);
> @@ -249,7 +249,7 @@ static void test_qmp_oob(void)
> QList *capabilities;
> QString *qstr;
>
> - qts = qtest_init_without_qmp_handshake(true, common_args);
> + qts = qtest_init_without_qmp_handshake(common_args);
>
> /* Check the greeting message. */
> resp = qtest_qmp_receive(qts);
> --
> 2.17.1
>
>
--
Marc-André Lureau
- [Qemu-devel] [PATCH v8 0/6] monitor: enable OOB by default, Peter Xu, 2018/09/05
- [Qemu-devel] [PATCH v8 1/6] monitor: Suspend monitor instead dropping commands, Peter Xu, 2018/09/05
- [Qemu-devel] [PATCH v8 2/6] monitor: resume the monitor earlier if needed, Peter Xu, 2018/09/05
- [Qemu-devel] [PATCH v8 4/6] Revert "tests: Add parameter to qtest_init_without_qmp_handshake", Peter Xu, 2018/09/05
- Re: [Qemu-devel] [PATCH v8 4/6] Revert "tests: Add parameter to qtest_init_without_qmp_handshake",
Marc-André Lureau <=
- [Qemu-devel] [PATCH v8 5/6] tests: add oob functional test for test-qmp-cmds, Peter Xu, 2018/09/05
- [Qemu-devel] [PATCH v8 6/6] tests: qmp-test: add queue full test, Peter Xu, 2018/09/05
- [Qemu-devel] [PATCH v8 3/6] monitor: remove "x-oob", turn oob on by default, Peter Xu, 2018/09/05
- Re: [Qemu-devel] [PATCH v8 0/6] monitor: enable OOB by default, Peter Xu, 2018/09/28