[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: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes |
Date: |
Tue, 25 Feb 2014 14:41:57 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
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?
Kevin
- 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, Paolo Bonzini, 2014/02/24
- Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes, Fam Zheng, 2014/02/24
- Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes, Kevin Wolf, 2014/02/24
- Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes, Paolo Bonzini, 2014/02/24
- Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes, Kevin Wolf, 2014/02/24
Re: [Qemu-devel] [RFC PATCH] block: optimize zero writes with bdrv_write_zeroes, Kevin Wolf, 2014/02/24