xfs
[Top] [All Lists]

Re: [PATCH v6 0/8] block: prepare for multipage bvecs

To: Ming Lei <ming.lei@xxxxxxxxxxxxx>, Jens Axboe <axboe@xxxxxx>, linux-kernel@xxxxxxxxxxxxxxx
Subject: Re: [PATCH v6 0/8] block: prepare for multipage bvecs
From: Jens Axboe <axboe@xxxxxxxxx>
Date: Thu, 9 Jun 2016 10:06:02 -0600
Cc: linux-block@xxxxxxxxxxxxxxx, Christoph Hellwig <hch@xxxxxxxxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxx>, "open list:DRBD DRIVER" <drbd-dev@xxxxxxxxxxxxxxxx>, Jan Kara <jack@xxxxxxx>, Joe Thornber <ejt@xxxxxxxxxx>, Keith Busch <keith.busch@xxxxxxxxx>, Kent Overstreet <kent.overstreet@xxxxxxxxx>, "Kirill A. Shutemov" <kirill.shutemov@xxxxxxxxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, Michal Hocko <mhocko@xxxxxxxx>, Mike Snitzer <snitzer@xxxxxxxxxx>, NeilBrown <neilb@xxxxxxxx>, Sagi Grimberg <sagig@xxxxxxxxxxxx>, Shaohua Li <shli@xxxxxx>, Steven Whitehouse <swhiteho@xxxxxxxxxx>, Tejun Heo <tj@xxxxxxxxxx>, "open list:XFS FILESYSTEM" <xfs@xxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel-dk.20150623.gappssmtp.com; s=20150623; h=subject:to:references:cc:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=gEjnzXH80ke3Z0QlPkKEkAYlOSZ2ssT1AlEiKc/dgLk=; b=S2D/NtJKMi78v5rYNGoaGIp6gYOA/77pr43nNpMHO/5xP1PIrDKifkt4cM6OiulSyH hDBzIjLuxuOOWcpBWoH15h0P5x4+Rr/8U6cXbz459tI0fCdSvuw/lA1aDpzfqJlftGCM eEo6XpBAajGm1N55HL0uYPnNybAmkf+mjXH51zWVGW3CkJuzI/zHH6OlswuNL6uxpLQz 3BwPNcSzWcqiyvHqoNhoBRC2UzAzkd1Ow6B8wjeC2/ZGe839xWl5mY4t5BKFa/gKTrjv r3BKeCGf+kqXnrsW8MIcREYNyY1MjqDB4GreP9q0KbJE07vYKxI4/DT8DqHUFNcEjMFT eM5w==
In-reply-to: <1464615294-9946-1-git-send-email-ming.lei@xxxxxxxxxxxxx>
References: <1464615294-9946-1-git-send-email-ming.lei@xxxxxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0
On 05/30/2016 07:34 AM, Ming Lei wrote:
Hi,

Interests[1] have been shown in multipage bvecs, so this patchset
try to prepare for the support and do two things:

1) the 1st 4 patches use bvec iterator to implement iterate_bvec(),
then we can drop the non-standard way for iterating bvec, which
can be thought as a good cleanup for lib/iov_iter.c

2) remove BIO_MAX_SECTORS and makre BIO_MAX_SIZE as obsolete, and
now there is only one user for each. Once multipage bvecs is introduced,
one bio may hold lots of sectors, and we should always use sort of
BIO_MAX_VECS which should be introduced in future and is similiar
with current BIO_MAX_PAGES.

The only functional change is iterate_bvec():lib/iov_iter.c

xfstests(-a auto) over loop aio is run for ext4/xfs to verify
the change and no regression found with this patchset.

Added patches 1-7, still waiting for an answer on my reply to #8.

--
Jens Axboe

<Prev in Thread] Current Thread [Next in Thread>