From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f52.google.com ([209.85.220.52]:36338 "EHLO mail-pa0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753056AbcDHUVb (ORCPT ); Fri, 8 Apr 2016 16:21:31 -0400 Received: by mail-pa0-f52.google.com with SMTP id bx7so64336606pad.3 for ; Fri, 08 Apr 2016 13:21:31 -0700 (PDT) Date: Fri, 8 Apr 2016 13:21:23 -0700 From: Omar Sandoval To: Ian Kent Cc: autofs@vger.kernel.org, linux-fsdevel@vger.kernel.org, kernel-team@fb.com Subject: Re: Autofs and mount namespaces Message-ID: <20160408202123.GA12948@vader.dhcp.thefacebook.com> References: <20160407181911.GA9018@vader> <1460076663.3135.37.camel@themaw.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1460076663.3135.37.camel@themaw.net> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Fri, Apr 08, 2016 at 08:51:03AM +0800, Ian Kent wrote: > On Thu, 2016-04-07 at 11:19 -0700, Omar Sandoval wrote: > > Hi, Ian, > > > > I wanted to bring up the issue of autofs and mount namespaces again, > > which recently resurfaced in the thread here [1]. In that thread, you > > mentioned that you had some patches to make autofs namespace-aware > > that > > you were holding on to. Do you think you could put those up so we can > > all work out the issues you mentioned? > > I can but I haven't tested them at all. > > Possibly I could post them to the autofs list if you want to test and > probably make changes to get them working. > > Would that be ok .... umm, I think I need to post the patch now anyway > as it's the easiest way to demonstrate what I think is a better approach > than the patch below .... > > I'm struggling to get back to work on this but I'm getting there and > hope to return to it soon. Thanks for taking a look. Your patch fixes the case I provided, and it does look much better than my hack :) Like you said in your other email, there are more cases, but this fix is a good start independent of those cases. Were there any other patches that needed testing? -- Omar