[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 00/17] block: Convert .bdrv_read/write to .bd
From: |
Fam Zheng |
Subject: |
Re: [Qemu-block] [PATCH v2 00/17] block: Convert .bdrv_read/write to .bdrv_preadv/pwritev |
Date: |
Fri, 29 Apr 2016 11:29:48 +0800 |
User-agent: |
Mutt/1.6.0 (2016-04-01) |
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>
- [Qemu-block] [PATCH v2 12/17] vmdk: Implement .bdrv_co_preadv() interface, (continued)
Re: [Qemu-block] [PATCH v2 00/17] block: Convert .bdrv_read/write to .bdrv_preadv/pwritev,
Fam Zheng <=
Re: [Qemu-block] [PATCH v2 00/17] block: Convert .bdrv_read/write to .bdrv_preadv/pwritev, Kevin Wolf, 2016/04/29