[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard |
Date: |
Wed, 06 May 2015 10:59:53 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
On 06/05/2015 03:45, Fam Zheng wrote:
>> > This is not enough, you also have to do the discard in block/mirror.c,
>> > otherwise the destination image could even become fully provisioned!
> I wasn't sure what if src and dest have different can_write_zeroes_with_unmap
> value, but your argument is stronger.
>
> I will add discard in mirror. Besides that, do we need to compare the
> can_write_zeroes_with_unmap?
Hmm, if can_write_zeroes_with_unmap is set, it's probably better to
write zeroes instead of discarding, in case the guest is relying on it.
Paolo
- [Qemu-devel] [PATCH 0/4] block: Mirror discarded sectors, Fam Zheng, 2015/05/05
- [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard, Fam Zheng, 2015/05/05
- Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard, Paolo Bonzini, 2015/05/05
- Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard, Fam Zheng, 2015/05/05
- Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard, Fam Zheng, 2015/05/06
- Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard, Paolo Bonzini, 2015/05/06
- Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard, Fam Zheng, 2015/05/11
- Re: [Qemu-devel] [PATCH 1/4] block: Fix dirty bitmap in bdrv_co_discard, Paolo Bonzini, 2015/05/11
[Qemu-devel] [PATCH 2/4] block: Remove bdrv_reset_dirty, Fam Zheng, 2015/05/05
[Qemu-devel] [PATCH 3/4] qemu-iotests: Make block job methods common, Fam Zheng, 2015/05/05
[Qemu-devel] [PATCH 4/4] qemu-iotests: Add test case for mirror with unmap, Fam Zheng, 2015/05/05