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:28:47 +0100
Message-ID: <20090402202846.GM28946@ZenIV.linux.org.uk>
References: <1238616394.24360.2.camel@heimdal.trondhjem.org> <20090401202629.GG28946@ZenIV.linux.org.uk> <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>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>,
	linux-fsdevel@vger.kernel.org
To: Linus Torvalds <torvalds@linux-foundation.org>
Return-path: <linux-fsdevel-owner@vger.kernel.org>
Received: from zeniv.linux.org.uk ([195.92.253.2]:58830 "EHLO
	ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1755962AbZDBU2w (ORCPT
	<rfc822;linux-fsdevel@vger.kernel.org>);
	Thu, 2 Apr 2009 16:28:52 -0400
Content-Disposition: inline
In-Reply-To: <alpine.LFD.2.00.0904021312560.4130@localhost.localdomain>
Sender: linux-fsdevel-owner@vger.kernel.org
List-ID: <linux-fsdevel.vger.kernel.org>

On Thu, Apr 02, 2009 at 01:17:47PM -0700, Linus Torvalds wrote:
> 
> 
> On Thu, 2 Apr 2009, Al Viro wrote:
> >
> > *growl*
> > 
> > Right, we are out of clone() flags, thanks to namespace-of-the-week
> > insanity by openvz bunch ;-/
> 
> I really think "clone()" is the wrong kind of granularity, and much too 
> big a hammer.
> 
> You never answered my suggestion about passing in namespace details to the 
> open routines in nameidata. I don't think it actually needs the whole 
> thing, the only thing (I _think_) it cares about ends up being just the 
> root/cwd.

The thing is, it wants more - it wants to mount stuff there.  Which means
that we really want current switched to it, or the normal check_mnt()
logics will screw you.

I don't have too serious objections against having a reference to namespace
in nameidata, but I really doubt that it'll be enough here.