qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 2/2] iotests: Test commit job start with concurr


From: Max Reitz
Subject: Re: [Qemu-devel] [PATCH 2/2] iotests: Test commit job start with concurrent I/O
Date: Wed, 22 May 2019 13:48:31 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1

On 21.05.19 21:16, Kevin Wolf wrote:
> This tests that concurrent requests are correctly drained before making
> graph modifications instead of running into assertions in
> bdrv_replace_node().
> 
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
>  tests/qemu-iotests/255        | 83 +++++++++++++++++++++++++++++++++++
>  tests/qemu-iotests/255.out    | 16 +++++++
>  tests/qemu-iotests/group      |  1 +
>  tests/qemu-iotests/iotests.py | 10 ++++-
>  4 files changed, 109 insertions(+), 1 deletion(-)
>  create mode 100755 tests/qemu-iotests/255
>  create mode 100644 tests/qemu-iotests/255.out
Reviewed-by: Max Reitz <address@hidden>

(By the way, because I see throttle on mirror, here, too: My problem
involves a concurrent bdrv_drain_all(), that’s the slight difference.
But your patch 1 works well for me.)

(Max)

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]