Re: [PATCH -V21 00/12] Generic name to handle and open by handlesyscalls

From: J. Bruce Fields
Date: Mon Oct 11 2010 - 19:35:08 EST


On Tue, Oct 05, 2010 at 03:52:08PM +0530, Aneesh Kumar K.V wrote:
> Hi,
>
> The below set of patches implement open by handle support using exportfs
> operations. This allows user space application to map a file name to file
> handle and later open the file using handle. This should be usable
> for userspace NFS [1] and 9P server [2]. XFS already support this with the ioctls
> XFS_IOC_PATH_TO_HANDLE and XFS_IOC_OPEN_BY_HANDLE.
>
> [1] http://nfs-ganesha.sourceforge.net/
> [2] http://thread.gmane.org/gmane.comp.emulators.qemu/68992
>
> git repo for the patchset at:
> git://git.kernel.org/pub/scm/linux/kernel/git/kvaneesh/linux-open-handle.git open-by-handle
>
> Test case can be found at
> http://git.kernel.org/?p=fs/ext2/kvaneesh/handle-test.git
> git://git.kernel.org/pub/scm/fs/ext2/kvaneesh/handle-test.git
>
> Changes from V20:
> a) Use better variable names
> b) change handle_size type to __u32
> c) max handle size is now nfsv4 handle size. Considering file system would
> want to support a handle that can fit into nfsv4 handle size limiting
> max handle size to 128 rather than 4096 is the right thing.

Looks OK to me.

I haven't read the later patches carefully, but in general an interface
like this seems to me a useful thing to have.

--b.

> d) I didn't add Ceph changes in the series. I am expecting the changes
> will go via Ceph tree.
>
> -aneesh
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/