[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 04/20] gluster: Switch to .bdrv_co_block_stat
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v6 04/20] gluster: Switch to .bdrv_co_block_status() |
Date: |
Fri, 29 Dec 2017 10:42:43 +0800 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Thu, 12/07 14:30, Eric Blake wrote:
> We are gradually moving away from sector-based interfaces, towards
> byte-based. Update the gluster driver accordingly.
>
> In want_zero mode, we continue to report fine-grained hole
> information (the caller wants as much mapping detail as possible);
> but when not in that mode, the caller prefers larger *pnum and
> merely cares about what offsets are allocated at this layer, rather
> than where the holes live. Since holes still read as zeroes at
> this layer (rather than deferring to a backing layer), we can take
> the shortcut of skipping find_allocation(), and merely state that
> all bytes are allocated.
>
> We can also drop redundant bounds checks that are already
> guaranteed by the block layer.
>
> Signed-off-by: Eric Blake <address@hidden>
>
Reviewed-by: Fam Zheng <address@hidden>
- Re: [Qemu-devel] [PATCH v6 01/20] block: Add .bdrv_co_block_status() callback, (continued)
- [Qemu-devel] [PATCH v6 03/20] file-posix: Switch to .bdrv_co_block_status(), Eric Blake, 2017/12/07
- [Qemu-devel] [PATCH v6 02/20] block: Switch passthrough drivers to .bdrv_co_block_status(), Eric Blake, 2017/12/07
- [Qemu-devel] [PATCH v6 04/20] gluster: Switch to .bdrv_co_block_status(), Eric Blake, 2017/12/07
- [Qemu-devel] [PATCH v6 05/20] iscsi: Switch cluster_sectors to byte-based, Eric Blake, 2017/12/07
- [Qemu-devel] [PATCH v6 07/20] iscsi: Switch to .bdrv_co_block_status(), Eric Blake, 2017/12/07
- [Qemu-devel] [PATCH v6 06/20] iscsi: Switch iscsi_allocmap_update() to byte-based, Eric Blake, 2017/12/07
- [Qemu-devel] [PATCH v6 08/20] null: Switch to .bdrv_co_block_status(), Eric Blake, 2017/12/07
- [Qemu-devel] [PATCH v6 09/20] parallels: Switch to .bdrv_co_block_status(), Eric Blake, 2017/12/07