[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [kvm-unit-tests PATCH v2 2/4] scripts/arch-run: don't use deprecated
From: |
Auger Eric |
Subject: |
Re: [kvm-unit-tests PATCH v2 2/4] scripts/arch-run: don't use deprecated server/nowait options |
Date: |
Wed, 26 May 2021 17:05:59 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.0 |
Hi Alex,
On 5/25/21 7:26 PM, Alex Bennée wrote:
> The very fact that QEMU drops the deprecation warning while running is
> enough to confuse the its-migration test into failing. The boolean
> options server and wait have accepted the long form options for a long
> time.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> Cc: Shashi Mallela <shashi.mallela@linaro.org>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Eric
> ---
> scripts/arch-run.bash | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/arch-run.bash b/scripts/arch-run.bash
> index 5997e38..70693f2 100644
> --- a/scripts/arch-run.bash
> +++ b/scripts/arch-run.bash
> @@ -122,14 +122,14 @@ run_migration ()
> trap 'kill 0; exit 2' INT TERM
> trap 'rm -f ${migout1} ${migsock} ${qmp1} ${qmp2} ${fifo}' RETURN EXIT
>
> - eval "$@" -chardev socket,id=mon1,path=${qmp1},server,nowait \
> + eval "$@" -chardev socket,id=mon1,path=${qmp1},server=on,wait=off \
> -mon chardev=mon1,mode=control | tee ${migout1} &
>
> # We have to use cat to open the named FIFO, because named FIFO's,
> unlike
> # pipes, will block on open() until the other end is also opened, and
> that
> # totally breaks QEMU...
> mkfifo ${fifo}
> - eval "$@" -chardev socket,id=mon2,path=${qmp2},server,nowait \
> + eval "$@" -chardev socket,id=mon2,path=${qmp2},server=on,wait=off \
> -mon chardev=mon2,mode=control -incoming unix:${migsock} <
> <(cat ${fifo}) &
> incoming_pid=`jobs -l %+ | awk '{print$2}'`
>
>