From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Nate Diller" Subject: Re: [PATCH -mm 3/10][RFC] aio: use iov_length instead of ki_left Date: Mon, 15 Jan 2007 21:37:31 -0800 Message-ID: <5c49b0ed0701152137s52e8f2c8sc93ea3a073e17e1c@mail.gmail.com> References: <20070116015450.9764.37697.patchbomb.py@nate-64.agami.com> <20070116015450.9764.52713.patchbomb.py@nate-64.agami.com> <20070116021438.GA15774@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: To: "Christoph Hellwig" , "Nate Diller" , "Nate Diller" , "Andrew Morton" , "Alan Cox" , "Trond Myklebust" , "Benjamin LaHaise" , "Alexander Viro" , "Suparna Bhattacharya" , "Kenneth W Chen" , "David Brownell" , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, netdev@vger.kernel.org, ocfs2-devel@oss.oracle.com, linux-aio@kvack.org, xfs-masters@oss.sgi.com In-Reply-To: <20070116021438.GA15774@infradead.org> Content-Disposition: inline Sender: netdev-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On 1/15/07, Christoph Hellwig wrote: > On Mon, Jan 15, 2007 at 05:54:50PM -0800, Nate Diller wrote: > > Convert code using iocb->ki_left to use the more generic iov_length() call. > > No way. We need to reduce the numer of iovec traversals, not adding > more of them. ok, I can work on a version of this that uses struct iodesc. Maybe something like this? struct iodesc { struct iovec *iov; unsigned long nr_segs; size_t nbytes; }; I suppose it's worth doing the iodesc thing along with this patchset anyway, since it'll avoid an extra round of interface churn. NATE