[virtio-dev] [PATCH] virtio-blk: add DISCARD support to virtio-blk driver

Liu, Changpeng
Tue, 28 Mar 2017 01:52:02 -0000

> -----Original Message-----
> From: virtio-dev at lists.oasis-open.org [mailto:virtio-dev at
lists.oasis-open.org] On
> Behalf Of Paolo Bonzini
> Sent: Monday, March 27, 2017 7:34 PM
> To: Liu, Changpeng <changpeng.liu at intel.com>; virtio-dev at
lists.oasis-open.org;
> virtualization at lists.linux-foundation.org; linux-kernel at
vger.kernel.org; hch at lst.de
> Cc: qemu-devel at nongnu.org
> Subject: Re: [virtio-dev] [PATCH] virtio-blk: add DISCARD support to
virtio-blk driver
>
>
>
> On 28/03/2017 10:39, Changpeng Liu wrote:
> > +	if (virtio_has_feature(vdev, VIRTIO_BLK_F_DISCARD)) {
> > +		q->limits.discard_zeroes_data = 0;
>
> Maybe you could use another feature bit to populate discard_zeroes_data.
>
> Paolo
>
Sounds good to me, Christoph Hellwig mentioned this field will be removed in
next release, just removed this line makes clear.
> > +		q->limits.discard_alignment = blk_size;
> > +		q->limits.discard_granularity = blk_size;
> > +		blk_queue_max_discard_sectors(q, UINT_MAX);
> > +		blk_queue_max_discard_segments(q, 1);
> > +		queue_flag_set_unlocked(QUEUE_FLAG_DISCARD, q);
> > +	}
> > +
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: virtio-dev-unsubscribe at lists.oasis-open.org
> For additional commands, e-mail: virtio-dev-help at lists.oasis-open.org