mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Ian Kent <raven@themaw.net>
To: Miklos Szeredi <miklos@szeredi.hu>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>,
	autofs@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	linux-kernel@vger.kernel.org, sukadev@linux.vnet.ibm.com,
	serge.hallyn@canonical.com
Subject: Re: [PATCH 0/2] struct pid-ify autofs4
Date: Thu, 25 Oct 2012 08:25:28 +0800	[thread overview]
Message-ID: <1351124728.3116.8.camel@perseus.themaw.net> (raw)
In-Reply-To: <87625z531j.fsf@tucsk.pomaz.szeredi.hu>

On Wed, 2012-10-24 at 16:59 +0200, Miklos Szeredi wrote:
> > Ian Kent <raven@themaw.net> writes:
> > 
> > > Yeah, the problem with that is that "autofs doesn't work if containers
> > > are used" is ill defined since there are use cases where it does, I
> > > believe. At the very least, ill defined in my view of things.
> 
> Customer says:
> 
>  "There is no interaction between host and the conatainer.  The host use
>   only his own automount and each containers used automount in their
>   container."

That sounds like a sensible requirement to me.

> 
> I think it's a pretty clearly defined use case.  And one which automount
> could easily support since the only requirement is that all namespaces
> are treated equally.

Yep. A problem might be dealing with mounts cloned from the parent
namespace at container creation. Ideally they wouldn't be duplicated so
they wouldn't need to be cleaned up (perhaps that's justified given the
requirement above).

Another thought is, what would happen on just cloning a namespace, not
necessarily as a container (is that even a sensible question)? The user
may actually want the mounts in this case, and can we even tell the
difference at namespace creation?

> 
> But I agree that adding safeguards against cases which don't have such
> easily defined semantics (such as triggers from several different
> namespaces).

Look forward to it.

> 
> I'll post updated patches.
> 
> Thanks,
> Miklos
> 



      reply	other threads:[~2012-10-25  0:32 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-19 15:29 Miklos Szeredi
2012-09-19 15:29 ` [PATCH 1/2] Replace pid_t in autofs4 with struct pid reference Miklos Szeredi
2012-09-19 15:29 ` [PATCH 2/2] autofs4: store struct pids in autofs_waitqs Miklos Szeredi
2012-09-21 15:44 ` [PATCH 0/2] struct pid-ify autofs4 Miklos Szeredi
2012-09-24  0:38   ` Ian Kent
2012-09-24 13:34     ` Miklos Szeredi
2012-09-25  1:46       ` Ian Kent
2012-09-25  2:56         ` Eric W. Biederman
2012-10-11  3:34           ` Ian Kent
2012-10-24 14:59             ` Miklos Szeredi
2012-10-25  0:25               ` Ian Kent [this message]

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=1351124728.3116.8.camel@perseus.themaw.net \
    --to=raven@themaw.net \
    --cc=autofs@vger.kernel.org \
    --cc=ebiederm@xmission.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=miklos@szeredi.hu \
    --cc=serge.hallyn@canonical.com \
    --cc=sukadev@linux.vnet.ibm.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®