Re: [PATCH] VFS autmounter support v2

viro@parcelfarce.linux.theplanet.co.uk
Wed, 18 Jun 2003 21:59:45 +0100


On Wed, Jun 18, 2003 at 03:20:15PM +0100, David Howells wrote:
>
> Hi Linus, Al,
>
> I've revised my patch to make sure a process in one namespace doesn't change
> the topology of another namespace (kern_automount() will return an error in
> that case, as does (u)mount). As a bonus, check_mnt() has been simplified to
> take account of the namespace pointer now in vfsmount.

You _still_ don't get it. OK, the last time: kern_automount() will
always do the same thing, no matter which namespace we are it. It
might be OK for AFS, but it's definitely unfit for any other use.

No amount of "use of (u)mount to rearrange topology" will help here -
with your code you have dentry marked, and stepping on it (in any
namespace, in any instance of that fs in a namespace) will always do
the same thing. And that is Wrong(tm).
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/