BlockLimits is currently an ugly mix of byte limits vs. sector limits. Unify it. Fix some bugs I found in bdrv_aligned_preadv() while at it.
Prequisite: Kevin's ongoing work to migrate bdrv_aligned_preadv() to be byte-based (commit 3de06b2 on his vmstate branch at the time of this email, but that gets rebased): https://lists.gnu.org/archive/html/qemu-devel/2016-06/msg02832.html Trivial contextual conflict in nbd.h with the pull request Paolo will soon be posting (both series add a #define near the same line; resolution is to add both): https://lists.gnu.org/archive/html/qemu-devel/2016-06/msg03333.html Also available as a tag at: git fetch git://repo.or.cz/qemu/ericb.git nbd-limits-v2 Since v1: - drop things already done in Kevin's work - rebase - split out lots of cleanup work to bdrv_refresh_limits() so that qemu-iotests does not gain any problems on 77 001/17:[down] 'block: Tighter assertions on bdrv_aligned_pwritev()' 002/17:[down] 'block: Document supported flags during bdrv_aligned_preadv()' 003/17:[down] 'block: Fix harmless off-by-one in bdrv_aligned_preadv()' 004/17:[down] 'nbd: Allow larger requests' 005/17:[down] 'nbd: Advertise realistic limits to block layer' 006/17:[down] 'iscsi: Advertise realistic limits to block layer' 007/17:[down] 'block: Give nonzero result to blk_get_max_transfer_length()' 008/17:[down] 'blkdebug: Set request_alignment during .bdrv_refresh_limits()' 009/17:[down] 'iscsi: Set request_alignment during .bdrv_refresh_limits()' 010/17:[down] 'qcow2: Set request_alignment during .bdrv_refresh_limits()' 011/17:[down] 'raw-win32: Set request_alignment during .bdrv_refresh_limits()' 012/17:[down] 'block: Set request_alignment during .bdrv_refresh_limits()' 013/17:[down] 'block: Set default request_alignment during bdrv_refresh_limits()' 014/17:[0061] [FC] 'block: Switch transfer length bounds to byte-based' 015/17:[0008] [FC] 'block: Switch discard length bounds to byte-based' 016/17:[down] 'block: Split bdrv_merge_limits() from bdrv_refresh_limits()' 017/17:[0044] [FC] 'block: Move request_alignment into BlockLimit' Eric Blake (17): block: Tighter assertions on bdrv_aligned_pwritev() block: Document supported flags during bdrv_aligned_preadv() block: Fix harmless off-by-one in bdrv_aligned_preadv() nbd: Allow larger requests nbd: Advertise realistic limits to block layer iscsi: Advertise realistic limits to block layer block: Give nonzero result to blk_get_max_transfer_length() blkdebug: Set request_alignment during .bdrv_refresh_limits() iscsi: Set request_alignment during .bdrv_refresh_limits() qcow2: Set request_alignment during .bdrv_refresh_limits() raw-win32: Set request_alignment during .bdrv_refresh_limits() block: Set request_alignment during .bdrv_refresh_limits() block: Set default request_alignment during bdrv_refresh_limits() block: Switch transfer length bounds to byte-based block: Switch discard length bounds to byte-based block: Split bdrv_merge_limits() from bdrv_refresh_limits() block: Move request_alignment into BlockLimit include/block/block.h | 1 + include/block/block_int.h | 48 ++++++++++++++------- include/block/nbd.h | 1 + include/qemu/typedefs.h | 1 + include/sysemu/block-backend.h | 2 +- block.c | 3 +- block/blkdebug.c | 18 ++++++-- block/block-backend.c | 9 ++-- block/bochs.c | 7 ++- block/cloop.c | 7 ++- block/dmg.c | 7 ++- block/io.c | 96 +++++++++++++++++++++++------------------- block/iscsi.c | 40 +++++++++--------- block/nbd-client.c | 4 -- block/nbd.c | 4 +- block/qcow2.c | 7 +-- block/raw-posix.c | 19 +++++---- block/raw-win32.c | 10 ++--- block/raw_bsd.c | 4 +- block/vvfat.c | 7 ++- hw/block/virtio-blk.c | 10 ++--- hw/scsi/scsi-generic.c | 15 ++++--- qemu-img.c | 9 ++-- 23 files changed, 195 insertions(+), 134 deletions(-) -- 2.5.5
