From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Kleikamp Subject: Re: [PATCH v3 00/22] loop: Issue O_DIRECT aio using bio_vec Date: Mon, 22 Oct 2012 19:53:40 -0500 Message-ID: <5085EA94.7000901@oracle.com> References: <1350918922-6096-1-git-send-email-dave.kleikamp@oracle.com> <20121023000733.GG4291@dastard> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Zach Brown , "Maxim V. Patlasov" To: Dave Chinner Return-path: In-Reply-To: <20121023000733.GG4291@dastard> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org 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 > > Cheers, > > Dave. >