[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror
From: |
Max Reitz |
Subject: |
Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror |
Date: |
Wed, 2 Oct 2019 16:57:14 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 |
On 12.09.19 17:13, Vladimir Sementsov-Ogievskiy wrote:
> Prior 9adc1cb49af8d do_sync_target_write had a bug: it reset aligned-up
> region in the dirty bitmap, which means that we may not copy some bytes
> and assume them copied, which actually leads to producing corrupted
> target.
>
> So 9adc1cb49af8d forced dirty bitmap granularity to be
> request_alignment for mirror-top filter, so we are not working with
> unaligned requests. However forcing large alignment obviously decreases
> performance of unaligned requests.
>
> This commit provides another solution for the problem: if unaligned
> padding is already dirty, we can safely ignore it, as
> 1. It's dirty, it will be copied by mirror_iteration anyway
> 2. It's dirty, so skipping it now we don't increase dirtiness of the
> bitmap and therefore don't damage "synchronicity" of the
> write-blocking mirror.
But that’s not what active mirror is for. The point of active mirror is
that it must converge because every guest write will contribute towards
that goal.
If you skip active mirroring for unaligned guest writes, they will not
contribute towards converging, but in fact lead to the opposite.
Max
signature.asc
Description: OpenPGP digital signature
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror,
Max Reitz <=
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Vladimir Sementsov-Ogievskiy, 2019/10/02
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Vladimir Sementsov-Ogievskiy, 2019/10/02
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Max Reitz, 2019/10/02
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Vladimir Sementsov-Ogievskiy, 2019/10/03
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Max Reitz, 2019/10/04
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Vladimir Sementsov-Ogievskiy, 2019/10/04
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Max Reitz, 2019/10/04
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Vladimir Sementsov-Ogievskiy, 2019/10/04
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Max Reitz, 2019/10/04
- Re: [PATCH 3/4] block/mirror: support unaligned write in active mirror, Vladimir Sementsov-Ogievskiy, 2019/10/04