From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Chinner Subject: Re: [PATCH v3 00/22] loop: Issue O_DIRECT aio using bio_vec Date: Tue, 23 Oct 2012 12:59:11 +1100 Message-ID: <20121023015911.GO4291@dastard> References: <1350918922-6096-1-git-send-email-dave.kleikamp@oracle.com> <20121023000733.GG4291@dastard> <5085EA94.7000901@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Zach Brown , "Maxim V. Patlasov" To: Dave Kleikamp Return-path: Content-Disposition: inline In-Reply-To: <5085EA94.7000901@oracle.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Mon, Oct 22, 2012 at 07:53:40PM -0500, Dave Kleikamp wrote: > On 10/22/2012 07:07 PM, Dave Chinner wrote: > > On Mon, Oct 22, 2012 at 10:15:00AM -0500, Dave Kleikamp wrote: > >> This is the current version of the patchset I presented at the LSF-MM > >> Summit in San Francisco in April. I apologize for letting it go so > >> long before re-submitting. > >> > >> This patchset was begun by Zach Brown and was originally submitted for > >> review in October, 2009. Feedback was positive, and I have picked up > >> where he left off, porting his patches to the latest mainline kernel > >> and adding support more file systems. > >> > >> This patch series adds a kernel interface to fs/aio.c so that kernel code can > >> issue concurrent asynchronous IO to file systems. It adds an aio command and > >> file system methods which specify io memory with pages instead of userspace > >> addresses. > >> > >> This series was written to reduce the current overhead loop imposes by > >> performing synchronus buffered file system IO from a kernel thread. These > >> patches turn loop into a light weight layer that translates bios into iocbs. > > > > I note that there is no support for XFS in this patch set. Is there > > a particular problem that prevents XFS from being converted, or it > > just hasn't been done? > > It just hasn't been done. It wasn't one of the trivial ones so I put it > off at first, and after that, it's an oversight. I'll follow up with an > xfs patch for your review. Thanks Shaggy, that's all I wanted to know. No extra work for me (apart from review and testing) is fine by me. ;) Cheers, Dave. -- Dave Chinner david@fromorbit.com