From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Kent Subject: Re: [PATCH 1/3] autofs4 - fix device ioctl mount lookup Date: Fri, 06 Sep 2013 16:54:19 +0800 Message-ID: <1378457659.2311.23.camel@perseus.fritz.box> References: <20130904005456.5025.94309.stgit@perseus.fritz.box> <20130904010301.GE13318@ZenIV.linux.org.uk> <20130904020013.GF13318@ZenIV.linux.org.uk> <20130904022617.GG13318@ZenIV.linux.org.uk> <20130904024230.GH13318@ZenIV.linux.org.uk> <1378456682.2311.21.camel@perseus.fritz.box> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Linus Torvalds , linux-fsdevel , rui.xiang@huawei.com, autofs mailing list , Kernel Mailing List , Al Viro To: Jeffrey Layton Return-path: In-Reply-To: <1378456682.2311.21.camel@perseus.fritz.box> Sender: autofs-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org Sorry, I should have added Jeff to the cc for this post. On Fri, 2013-09-06 at 16:38 +0800, Ian Kent wrote: > On Wed, 2013-09-04 at 03:42 +0100, Al Viro wrote: > > On Wed, Sep 04, 2013 at 03:26:17AM +0100, Al Viro wrote: > > > I've applied slightly modified variant of Jeff's "vfs: allow umount to handle > > > mountpoints without revalidating them" (modified by just leaving the > > > struct path filled with mountpoint and leaving the equivalent of follow_mount() > > > to caller) to the local queue and I'm pretty sure that it's what we want > > > here as well. > > > > ... and killed the modifications since the result ends up uglier for > > caller(s) anyway. Reapplied as-is. > > Looks like Jeff's patch has been merged, commit 8033426e6. > > Revalidation isn't the only thing not done on the last component using > Jeff's user_path_umountat() path walk. It also bypasses the managed > dentry code for the last component, which is why it's what I need as > well. > > Encoding umount in the name seems misleading as to what it really does > as would encoding unmanaged or similar since that doesn't properly cover > it either. > > I can rename it in a patch to solve my autofs problem, so how about > something like user_path_simple_last(), other suggestions anyone? > > Ian