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 <jeff.liu@xxxxxxxxxx> >> >> 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 from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
