Re: [Qemu-block] [PATCH v2 00/17] block: Convert .bdrv_read/write to .bdrv_preadv/pwritev

[Fam Zheng]

On Thu, 04/28 15:16, Kevin Wolf wrote:
> This series introduces a new BlockDriver interface, which will hopefully be 
> the
> final one, or at least good enough for another few years: 
> .bdrv_preadv/pwritev.
> It is based on coroutines, vectored, has flags and uses a byte granularity.
> This is now the preferred interface for new drivers.
> 
> All drivers still using the legacy interface .bdrv_read/write are converted to
> the new interface and the emulation code we had for the old interface is
> removed. Most of the drivers become zero-copy with these patches as they are
> vectored now; only vvfat continues to allocate a bounce buffer.
> 
> The less obscure formats (vmdk, vdi, vpc) also natively support byte-aligned
> requests now. The block layer is still enforcing a minimal alignment of 512, 
> so
> this isn't actually used yet, but in a next step, we can lift this restriction
> for drivers that implement .bdrv_preadv/pwritev.

Code working on byte granularity is consistent and much nicer to read!

Though I'm not 100% sure about the compressed write change in VMDK, but there
is apparently nothing wrong, too. So:

Reviewed-by: Fam Zheng <address@hidden>