From mboxrd@z Thu Jan 1 00:00:00 1970 From: Subrata Modak Subject: Re: [rfc patch 0/4] splice: cleanups and fixes Date: Sat, 21 Jun 2008 22:50:07 +0530 Message-ID: <1214068808.5607.3.camel@subratamodak.linux.ibm.com> References: <20080621154607.154640724@szeredi.hu> Reply-To: subrata@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: jens.axboe@oracle.com, linux-fsdevel@vger.kernel.org, ltp-list To: Miklos Szeredi Return-path: Received: from e36.co.us.ibm.com ([32.97.110.154]:56062 "EHLO e36.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750989AbYFURUg (ORCPT ); Sat, 21 Jun 2008 13:20:36 -0400 Received: from d03relay02.boulder.ibm.com (d03relay02.boulder.ibm.com [9.17.195.227]) by e36.co.us.ibm.com (8.13.8/8.13.8) with ESMTP id m5LHKUJS027210 for ; Sat, 21 Jun 2008 13:20:30 -0400 Received: from d03av01.boulder.ibm.com (d03av01.boulder.ibm.com [9.17.195.167]) by d03relay02.boulder.ibm.com (8.13.8/8.13.8/NCO v9.0) with ESMTP id m5LHKU67157434 for ; Sat, 21 Jun 2008 11:20:30 -0600 Received: from d03av01.boulder.ibm.com (loopback [127.0.0.1]) by d03av01.boulder.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id m5LHKT51007665 for ; Sat, 21 Jun 2008 11:20:30 -0600 In-Reply-To: <20080621154607.154640724@szeredi.hu> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: Hi Miklos, In the backdrop of the changes in splice(), would you also like to help us in reviewing the existing testcase for splice() in LTP: http://ltp.cvs.sourceforge.net/ltp/ltp/testcases/kernel/syscalls/splice/, It would also be great if you can send us some patch as well in the regard. Regards-- Subrata On Sat, 2008-06-21 at 17:46 +0200, Miklos Szeredi wrote: > Brian Wang reported some problems with NFS export of fuse filesystems, > which turned out to be bad interaction between splice (used by nfsd) > and page cache invalidation. > > I looked at the splice code, and found quite a bit of dead code, > duplication, and unnecessary complication. This patchset attempts to > resolve those, in addition to fixing the invalidation issues. Some > optimizations are lost in the process like the gang page lookup for > the fully cached case, and I can't really tell if these are important > enough to warrant the extra complexity. > > I did minimal testing to verify that splice(2) on regular files still > works. And since generic_file_splice_read() now shares most of its > code with generic_file_aio_read(), there's not much to go wrong in > there. That said, it needs more testing... > > Comments? > > Thanks, > Miklos > -- > > drivers/block/loop.c | 5 > fs/nfsd/vfs.c | 9 - > fs/pipe.c | 58 ------- > fs/splice.c | 371 +++++----------------------------------------- > include/linux/fs.h | 2 > include/linux/pipe_fs_i.h | 36 ---- > kernel/relay.c | 2 > mm/filemap.c | 2 > net/core/skbuff.c | 9 - > 9 files changed, 47 insertions(+), 447 deletions(-) > -- > To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html