[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH] qemu-iotests: fix 203 migration completion race
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH] qemu-iotests: fix 203 migration completion race |
Date: |
Mon, 5 Mar 2018 17:04:52 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 2018-03-05 16:59, Stefan Hajnoczi wrote:
> There is a race between the test's 'query-migrate' QMP command after the
> QMP 'STOP' event and completing the migration:
>
> The test case invokes 'query-migrate' upon receiving 'STOP'. At this
> point the migration thread may still be in the process of completing.
> Therefore 'query-migrate' can return 'status': 'active' for a brief
> window of time instead of 'status': 'completed'. This results in
> qemu-iotests 203 hanging.
>
> Solve the race by enabling the 'events' migration capability, which
> causes QEMU to emit migration-specific QMP events that do not suffer
> from this race condition. Wait for the QMP 'MIGRATION' event with
> 'status': 'completed'.
>
> Reported-by: Max Reitz <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> tests/qemu-iotests/203 | 15 +++++++++++----
> tests/qemu-iotests/203.out | 5 +++++
> 2 files changed, 16 insertions(+), 4 deletions(-)
So much for "the ppoll() dungeon"...
Thanks!
Reviewed-by: Max Reitz <address@hidden>
signature.asc
Description: OpenPGP digital signature