> -----Original Message----- > From: Christoph Hellwig [mailto:[email protected]] > Sent: Friday, October 26, 2018 4:27 PM > To: Paolo Bonzini <[email protected]> > Cc: Christoph Hellwig <[email protected]>; Daniel Verkamp > <[email protected]>; Jens Axboe <[email protected]>; Michael S. Tsirkin > <[email protected]>; [email protected]; linux- > [email protected]; Stefan Hajnoczi <[email protected]>; Liu, Changpeng > <[email protected]> > Subject: Re: [PATCH v8] virtio_blk: add discard and write zeroes support > > On Fri, Oct 26, 2018 at 01:28:54AM +0200, Paolo Bonzini wrote: > > On 15/10/2018 11:27, Christoph Hellwig wrote: > > > There is some issues in this spec. For one using the multiple ranges > > > also for write zeroes is rather inefficient. Write zeroes really should > > > use the same format as read and write. > > > > What makes it inefficient? > > We require a memory allocation for each write zeroes instead of encoding > the lba/len in the command. Make sense to me, but need to change the spec first.
- [PATCH v8] virtio_blk: add discard and write zeroes sup... Daniel Verkamp
- Re: [PATCH v8] virtio_blk: add discard and write z... Michael S. Tsirkin
- Re: [PATCH v8] virtio_blk: add discard and write z... Ming Lei
- Re: [PATCH v8] virtio_blk: add discard and write z... Christoph Hellwig
- Re: [PATCH v8] virtio_blk: add discard and wri... Daniel Verkamp
- RE: [PATCH v8] virtio_blk: add discard and... Liu, Changpeng
- RE: [PATCH v8] virtio_blk: add discard and wri... Liu, Changpeng
- Re: [PATCH v8] virtio_blk: add discard and wri... Paolo Bonzini
- Re: [PATCH v8] virtio_blk: add discard and... Christoph Hellwig
- RE: [PATCH v8] virtio_blk: add discard... Liu, Changpeng
- Re: [PATCH v8] virtio_blk: add discard... Paolo Bonzini
- Re: [PATCH v8] virtio_blk: add discard and write z... Stefan Hajnoczi
- Re: [PATCH v8] virtio_blk: add discard and wri... Michael S. Tsirkin
- Re: [PATCH v8] virtio_blk: add discard and... Stefan Hajnoczi
- Re: [PATCH v8] virtio_blk: add discard... Michael S. Tsirkin
- Re: [PATCH v8] virtio_blk: add di... Daniel Verkamp
