ping On 05/10/2017 09:20 PM, Eric Blake wrote: > There are patches floating around to add NBD_CMD_BLOCK_STATUS, > but NBD wants to report status on byte granularity (even if the > reporting will probably be naturally aligned to sectors or even > much higher levels). I've therefore started the task of > converting our block status code to report at a byte granularity > rather than sectors. > > This is part one of that conversion: bdrv_is_allocated(). > Other parts still need a v2, but here's the link to their v1: > tracking dirty bitmaps by bytes: > https://lists.gnu.org/archive/html/qemu-devel/2017-04/msg02163.html > replacing bdrv_get_block_status() with a byte based callback > in all the drivers: > https://lists.gnu.org/archive/html/qemu-devel/2017-04/msg02642.html > > Available as a tag at: > git fetch git://repo.or.cz/qemu/ericb.git nbd-byte-allocated-v2 > > Since v1: > - Add R-b from John as appropriate > - Couple of new patches for cleanups he noticed > - Rebase to Max's block branch > > 001/20:[----] [--] 'blockjob: Track job ratelimits via bytes, not sectors' > 002/20:[----] [--] 'trace: Show blockjob actions via bytes, not sectors' > 003/20:[----] [--] 'stream: Switch stream_populate() to byte-based' > 004/20:[----] [--] 'stream: Switch stream_run() to byte-based' > 005/20:[----] [--] 'commit: Switch commit_populate() to byte-based' > 006/20:[----] [--] 'commit: Switch commit_run() to byte-based' > 007/20:[----] [--] 'mirror: Switch MirrorBlockJob to byte-based' > 008/20:[----] [--] 'mirror: Switch mirror_do_zero_or_discard() to byte-based' > 009/20:[down] 'mirror: Update signature of mirror_clip_sectors()' > 010/20:[0013] [FC] 'mirror: Switch mirror_cow_align() to byte-based' > 011/20:[----] [-C] 'mirror: Switch mirror_do_read() to byte-based' > 012/20:[0014] [FC] 'mirror: Switch mirror_iteration() to byte-based' > 013/20:[----] [--] 'block: Drop unused bdrv_round_sectors_to_clusters()' > 014/20:[----] [--] 'backup: Switch BackupBlockJob to byte-based' > 015/20:[0008] [FC] 'backup: Switch block_backup.h to byte-based' > 016/20:[----] [--] 'backup: Switch backup_do_cow() to byte-based' > 017/20:[----] [--] 'backup: Switch backup_run() to byte-based' > 018/20:[0039] [FC] 'block: Make bdrv_is_allocated() byte-based' > 019/20:[down] 'block: Minimize raw use of bds->total_sectors' > 020/20:[0026] [FC] 'block: Make bdrv_is_allocated_above() byte-based' > > Eric Blake (20): > blockjob: Track job ratelimits via bytes, not sectors > trace: Show blockjob actions via bytes, not sectors > stream: Switch stream_populate() to byte-based > stream: Switch stream_run() to byte-based > commit: Switch commit_populate() to byte-based > commit: Switch commit_run() to byte-based > mirror: Switch MirrorBlockJob to byte-based > mirror: Switch mirror_do_zero_or_discard() to byte-based > mirror: Update signature of mirror_clip_sectors() > mirror: Switch mirror_cow_align() to byte-based > mirror: Switch mirror_do_read() to byte-based > mirror: Switch mirror_iteration() to byte-based > block: Drop unused bdrv_round_sectors_to_clusters() > backup: Switch BackupBlockJob to byte-based > backup: Switch block_backup.h to byte-based > backup: Switch backup_do_cow() to byte-based > backup: Switch backup_run() to byte-based > block: Make bdrv_is_allocated() byte-based > block: Minimize raw use of bds->total_sectors > block: Make bdrv_is_allocated_above() byte-based > > include/block/block.h | 10 +- > include/block/block_backup.h | 11 +- > include/qemu/ratelimit.h | 3 +- > block/backup.c | 130 ++++++++----------- > block/commit.c | 54 ++++---- > block/io.c | 92 +++++++------ > block/mirror.c | 300 > ++++++++++++++++++++++--------------------- > block/replication.c | 29 +++-- > block/stream.c | 35 +++-- > block/vvfat.c | 34 +++-- > migration/block.c | 9 +- > qemu-img.c | 15 ++- > qemu-io-cmds.c | 70 +++++----- > block/trace-events | 14 +- > 14 files changed, 396 insertions(+), 410 deletions(-) >
-- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
