From mboxrd@z Thu Jan  1 00:00:00 1970
From: Al Viro <viro@ZenIV.linux.org.uk>
Subject: Re: [PATCH 1/4] VFS: Add a VFS helper function
	vfs_remote_path_lookup()
Date: Thu, 2 Apr 2009 21:54:18 +0100
Message-ID: <20090402205418.GN28946@ZenIV.linux.org.uk>
References: <1238628133.24360.51.camel@heimdal.trondhjem.org> <20090401233252.GH28946@ZenIV.linux.org.uk> <1238629407.19782.5.camel@heimdal.trondhjem.org> <20090402191709.GJ28946@ZenIV.linux.org.uk> <1238700874.16087.42.camel@heimdal.trondhjem.org> <20090402195212.GK28946@ZenIV.linux.org.uk> <20090402195759.GL28946@ZenIV.linux.org.uk> <alpine.LFD.2.00.0904021312560.4130@localhost.localdomain> <20090402202846.GM28946@ZenIV.linux.org.uk> <1238705130.16087.58.camel@heimdal.trondhjem.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	linux-fsdevel@vger.kernel.org
To: Trond Myklebust <Trond.Myklebust@netapp.com>
Return-path: <linux-fsdevel-owner@vger.kernel.org>
Received: from zeniv.linux.org.uk ([195.92.253.2]:42290 "EHLO
	ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1756492AbZDBUyX (ORCPT
	<rfc822;linux-fsdevel@vger.kernel.org>);
	Thu, 2 Apr 2009 16:54:23 -0400
Content-Disposition: inline
In-Reply-To: <1238705130.16087.58.camel@heimdal.trondhjem.org>
Sender: linux-fsdevel-owner@vger.kernel.org
List-ID: <linux-fsdevel.vger.kernel.org>

On Thu, Apr 02, 2009 at 04:45:30PM -0400, Trond Myklebust wrote:

> The NFS client is constructing the namespace as it walks the path using
> the ->follow_link() trick to automount new filesystems. As long as we
> can add a mount namespace argument to do_add_mount(), then we should be
> able to make use of a namespace in struct nameidata.

*wince*
Theoretically doable, practically can get ugly.

> The other thing we'd need is a 'root' path argument to ensure that
> __vfs_follow_link() does the right thing when presented with an absolute
> symlink.

The only problem I see with this is that we'll get root vfsmount/dentry
refcount jerked on *all* lookups now, instead of just the absolute
ones...