mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: viro@parcelfarce.linux.theplanet.co.uk
To: David Howells <dhowells@warthog.cambridge.redhat.com>
Cc: Linus Torvalds <torvalds@transmeta.com>,
	David Howells <dhowells@redhat.com>,
	Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] VFS autmounter support
Date: Wed, 18 Jun 2003 09:36:10 +0100	[thread overview]
Message-ID: <20030618083610.GB6754@parcelfarce.linux.theplanet.co.uk> (raw)
In-Reply-To: <18851.1055922933@warthog.warthog>

On Wed, Jun 18, 2003 at 08:55:33AM +0100, David Howells wrote:

> > We need a light-weight automount.  No arguments here.  But it should
> > be per-namespace - i.e. "I want to have <foo> mounted on /usr/barf on
> > demand and I have no intention to screw somebody else - somebody who
> > might have the same directory seen as /usr/local/debian/barf and want
> > <blah> mounted on demand there".
> 
> I don't have that intention of mucking someone else up either... But consider:
> what happens when a namespace is copied? All the automounter directories for
> autofs/amd/AFS are copied too... With the way I've come up with, this is
> irrelevant; the in-VFS automount will still work exactly the way it did until
> it is removed from that namespace. This is the way _I_ would expect things to
> work.

With your patch automount will happen on every reference to dentry.
Regardless of the namespace we are in.  
 
> I don't see that your argument is a problem... anyone who wants something
> different in their namespace is going to have to change things anyway.

And how would they do it?  You get mount triggered by stepping on dentry.
Period.  Your kern_automount() will then create a new vfsmount and slap
it on the tree.  Again, regardless of the namespace we are in / vfsmount
we are looking at / etc.

I have two namespaces.  One of them has filesystem A mounted on /usr/include.
Another - on /usr/local/include.  The first one wants /usr/include/foo1 trigger
mounting B and /usr/include/foo2 trigger mounting C.  The second one wants
/usr/local/include/foo1 trigger mounting D and /usr/local/include/foo2 not
trigger anything.

Namespaces are completely unrelated - I have them set for two different
users that happen to need some common files, but otherwise have very
different environments.  Could you describe what that "having to change
things" would involve?

  reply	other threads:[~2003-06-18  8:22 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-06-17 14:55 David Howells
2003-06-17 16:08 ` H. Peter Anvin
2003-06-17 18:07   ` David Howells
2003-06-17 18:19     ` H. Peter Anvin
2003-06-18  8:37       ` David Howells
2003-06-18  8:48         ` viro
2003-06-18 17:16         ` H. Peter Anvin
2003-06-19  7:20           ` David Howells
2003-06-19  7:30             ` H. Peter Anvin
2003-06-24 15:23               ` David Howells
2003-06-18  4:55 ` Linus Torvalds
2003-06-18  5:10   ` viro
2003-06-18  7:55     ` David Howells
2003-06-18  8:36       ` viro [this message]
2003-06-18 11:01         ` David Howells

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20030618083610.GB6754@parcelfarce.linux.theplanet.co.uk \
    --to=viro@parcelfarce.linux.theplanet.co.uk \
    --cc=dhowells@redhat.com \
    --cc=dhowells@warthog.cambridge.redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@transmeta.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®