From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48818) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bLeNa-0002YF-TY for qemu-devel@nongnu.org; Fri, 08 Jul 2016 18:36:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bLeNW-0006cx-KY for qemu-devel@nongnu.org; Fri, 08 Jul 2016 18:36:13 -0400 Received: from resqmta-po-06v.sys.comcast.net ([2001:558:fe16:19:96:114:154:165]:50145) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bLeNW-0006ch-En for qemu-devel@nongnu.org; Fri, 08 Jul 2016 18:36:10 -0400 From: Eric Blake Date: Fri, 8 Jul 2016 16:35:58 -0600 Message-Id: <1468017364-25980-1-git-send-email-eblake@redhat.com> Subject: [Qemu-devel] [PATCH v2 0/6] Auto-fragment large transactions at the block layer List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: qemu-block@nongnu.org, kwolf@redhat.com, famz@redhat.com We have max_transfer documented in BlockLimits, but while we honor it during pwrite_zeroes, we were blindly ignoring it during pwritev and preadv, leading to multiple drivers having to implement fragmentation themselves. This series moves fragmentation to the block layer, then fixes the NBD and iscsi driver to use it. qcow2 still does self-fragmenting, but that's because of cluster boundaries where it really has to do additional work beyond what the block layer can automatically provide. Prequisite: Kevin's latest block branch PULL request Also available as a tag at: git fetch git://repo.or.cz/qemu/ericb.git nbd-fragment-v2 Changes since v1: - patch 1 - fix valid -O2 compiler complaint [Kevin] - patch 2 [old 3] - move earlier [Kevin] - patch 1, 3 [old 1, 2] - tweak return value - patch 5 - minor formatting tweak [Kevin] 001/6:[0006] [FC] 'block: Fragment reads to max transfer length' 002/6:[----] [--] 'raw_bsd: Don't advertise flags not supported by protocol layer' 003/6:[0004] [FC] 'block: Fragment writes to max transfer length' 004/6:[----] [--] 'nbd: Rely on block layer to break up large requests' 005/6:[0004] [FC] 'nbd: Drop unused offset parameter' 006/6:[----] [--] 'iscsi: Rely on block layer to break up large requests' Eric Blake (6): block: Fragment reads to max transfer length raw_bsd: Don't advertise flags not supported by protocol layer block: Fragment writes to max transfer length nbd: Rely on block layer to break up large requests nbd: Drop unused offset parameter iscsi: Rely on block layer to break up large requests include/block/nbd.h | 1 - nbd/nbd-internal.h | 4 +-- block/io.c | 90 +++++++++++++++++++++++++++++++++++++++-------------- block/iscsi.c | 14 +++------ block/nbd-client.c | 78 +++++++++++++--------------------------------- block/nbd.c | 12 ++----- block/raw_bsd.c | 6 ++-- nbd/common.c | 5 +-- 8 files changed, 103 insertions(+), 107 deletions(-) -- 2.5.5