From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Liu Subject: Re: [PATCH v2] vfs: export lseek_execute() to modules Date: Tue, 25 Jun 2013 15:48:55 +0800 Message-ID: <51C94B67.5070705@oracle.com> References: <51C91645.8050502@oracle.com> <20130625071139.GZ4165@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Dave Chinner , andi@firstfloor.org, Andrew Morton , Christoph Hellwig , Chris.mason@fusionio.com, jbacik@fusionio.com, Ben Myers , tytso@mit.edu, hughd@google.com, Mark Fasheh , Joel Becker , sage@inktank.com To: Al Viro Return-path: In-Reply-To: <20130625071139.GZ4165@ZenIV.linux.org.uk> Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On 06/25/2013 03:11 PM, Al Viro wrote: > On Tue, Jun 25, 2013 at 12:02:13PM +0800, Jeff Liu wrote: >> From: Jie Liu >> >> For those file systems(btrfs/ext4/ocfs2/tmpfs) that support >> SEEK_DATA/SEEK_HOLE functions, we end up handling the similar >> matter in lseek_execute() to update the current file offset >> to the desired offset if it is valid, ceph also does the >> simliar things at ceph_llseek(). >> >> To reduce the duplications, this patch make lseek_execute() >> public accessible so that we can call it directly from the >> underlying file systems. > > Umm... I like it, but it needs changes: > * inode argument of lseek_execute() is pointless (and killed > off in vfs.git, actually) > * I'm really not happy about the name of that function. For > a static it's kinda-sort tolerable, but for something global, let > alone exported... > > I've put a modified variant into #for-next; could you check if you are > still OK with it? Yep, I'm ok with vfs_setpos(). Thanks, -Jeff -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org