Re: [PATCH -V7 3/9] vfs: Add name to file handle conversion support

[Andreas Dilger]

On 2010-05-12, at 09:50, Aneesh Kumar K.V wrote:
> +static long do_sys_name_to_handle(struct path *path,
> +			struct file_handle __user *ufh)
> +{
> +	if (handle_size <= f_handle.handle_size) {
> +		/* get the uuid */
> +		retval = sb->s_op->get_fsid(sb, &this_fs_id);
> +		if (!retval) {
> +			/*
> +			 * Now verify whether we get the same vfsmount
> +			 * if we lookup with uuid. In case we end up having
> +			 * same uuid for the multiple file systems. When doing
> +			 * uuid based lookup we would return the first one.So
> +			 * with name_to_handle if we don't find the same
> +			 * vfsmount with lookup return EOPNOTSUPP
> +			 */
> +			mnt = fs_get_vfsmount(current, &this_fs_id);
> +			if (mnt != path->mnt) {
> +				retval = -EOPNOTSUPP;
> +				mntput(mnt);
> +				goto err_free_out;
> +			}

I don't see that this does anything for us except add overhead.  This is no protection against mounting a second filesystem with the same UUID after the handle is returned, since there is no expiration for file handles.

At best I think we could start by changing the list-based UUID lookup with a hash-based one, and when adding a duplicate UUID at mount time start by printing out an error message to the console in case of duplicated UUIDs, and maybe at some point in the future this might cause the mount to fail (though I don't think we can make that decision lightly or quickly).

That moves the overhead to mount time instead of for each name_to_handle() call (which would be brutal for a system with many filesystems mounted).

> +SYSCALL_DEFINE4(name_to_handle_at, int, dfd, const char __user *, name,
> +		struct file_handle __user *, handle, int, flag)
> +{
> +	int follow;
> +	long ret = -EINVAL;
> +	struct path path;
> +
> +	if ((flag & ~AT_SYMLINK_NOFOLLOW) != 0)
> +		goto err_out;

Won't this break the use of AT_FDCWD to get a relative file handle?


Cheers, Andreas
--
Andreas Dilger
Lustre Technical Lead
Oracle Corporation Canada Inc.

--
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/