[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write
From: |
Peter Lieven |
Subject: |
Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes |
Date: |
Tue, 25 Feb 2014 18:03:16 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
Am 25.02.2014 14:41, schrieb Kevin Wolf:
> Am 24.02.2014 um 14:01 hat Peter Lieven geschrieben:
>> On 24.02.2014 11:38, Paolo Bonzini wrote:
>>> Il 24/02/2014 11:26, Peter Lieven ha scritto:
>>>> I personally do not need this for QCOW2 but for iSCSI. Here the
>>>> optimization
>>>> is basically saved bandwidth since a zero write becomes a WRITESAME.
>>> It saves bandwidth, but at the potential cost of extra host CPU
>>> utilization. I would be fine with having this automatically, but
>>> drv->bdrv_co_write_zeroes is not the right check because it is
>>> true for qcow2 and raw formats. Something using bdrv_get_info is
>>> probably better, because it would have fewer or no false
>>> positives.
>>>
>>>> In all cases if unmap=on it would additionally save disk space.
>>> It would also cause worse performance though. I think the automatic
>>> addition BDRV_REQ_MAY_UNMAP is what should be a separate option. Perhaps
>>> you can have a three-state option, detect-zeros=no/yes/unmap.
>> What would be the desired way to store this flag in the BlockDriverState?
> Some new enum field?
Is there an already implemented example where I can copy from? Its quite
diffucult to search through all the involved
functions when doing it for the first time.
Peter
- Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes, (continued)
Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes, Kevin Wolf, 2014/02/24