[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-3.1 v3] iotests: simple mirror test with kvm
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
Re: [Qemu-devel] [PATCH for-3.1 v3] iotests: simple mirror test with kvm on 1G image |
Date: |
Mon, 3 Dec 2018 15:40:38 +0000 |
03.12.2018 18:12, Max Reitz wrote:
> From: Vladimir Sementsov-Ogievskiy <address@hidden>
>
> This test is broken without previous commit fixing dead-lock in mirror.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> Signed-off-by: Max Reitz <address@hidden>
> ---
> Based-on: address@hidden
>
> This is a v3 to the second patch of "[PATCH v2 0/2] mirror dead-lock" by
> Vladimir. Instead of mirroring from a fully allocated image, this
> version mirrors from a preallocation=metadata image, and it mirrors not
> to the real disk, but to a throttle/null-co combination.
>
> The advantages are that it is an indicator of the bug independently of
> disk speed (Vladimir's version always worked on tmpfs for me), and that
> it allocates less than 1 MB of space in contrast to 2 GB.
>
> The disadvantage is that it is not a real-life use case. We may want to
> add Vladimir's test, too, probably in a special group that is not
> executed by default.
>
> Also, this version fixes compatibility with python3 (by using
> iotests.log()), and it kills the VM even on failure (by catching and
> reraising the exception, after the VM has been shut down).
>
> Finally, the test can still be in the quick group, because it turns out
> that bps-total=size is enough to make it fail before the fix (so it runs
> in 1 s if it passes).
works for me, thank you!
--
Best regards,
Vladimir