From patchwork Thu Mar 26 05:23:36 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ian Kent X-Patchwork-Id: 11459281 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 52293161F for ; Thu, 26 Mar 2020 05:32:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 3B88F207FC for ; Thu, 26 Mar 2020 05:32:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727575AbgCZFcw (ORCPT ); Thu, 26 Mar 2020 01:32:52 -0400 Received: from icp-osb-irony-out5.external.iinet.net.au ([203.59.1.221]:30174 "EHLO icp-osb-irony-out5.external.iinet.net.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726279AbgCZFcw (ORCPT ); Thu, 26 Mar 2020 01:32:52 -0400 X-SMTP-MATCH: 0 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: A2CdAgBuO3xe/4G50HZgBh0BAQEJAREFBQGBaAcBCwEBgieBYRIqhBqPYAaBEjiJZ4ophSkUgWcKAQEBAQEBAQEBGxkBAgQBAYREAoIoJDUIDgIQAQEBBQEBAQEBBQMBbYUKWEIBDAGFFAYjBFIQGA0CGA4CAkcQBhOFfiSuIH8zGgKKToEOKgGMLhp5gQeBRAOBNoFmhDEKgyGCXgSNcIMDhw1FgQCXEIJGlwkdjzsDjCEtpluFeQE3gVhNLgqDJ1AYnH83MIEGAQGEGYklXwEB X-IPAS-Result: A2CdAgBuO3xe/4G50HZgBh0BAQEJAREFBQGBaAcBCwEBgieBYRIqhBqPYAaBEjiJZ4ophSkUgWcKAQEBAQEBAQEBGxkBAgQBAYREAoIoJDUIDgIQAQEBBQEBAQEBBQMBbYUKWEIBDAGFFAYjBFIQGA0CGA4CAkcQBhOFfiSuIH8zGgKKToEOKgGMLhp5gQeBRAOBNoFmhDEKgyGCXgSNcIMDhw1FgQCXEIJGlwkdjzsDjCEtpluFeQE3gVhNLgqDJ1AYnH83MIEGAQGEGYklXwEB X-IronPort-AV: E=Sophos;i="5.72,307,1580745600"; d="scan'208";a="304456596" Received: from unknown (HELO mickey.themaw.net) ([118.208.185.129]) by icp-osb-irony-out5.iinet.net.au with ESMTP; 26 Mar 2020 13:23:36 +0800 Subject: [PATCH 4/4] autofs: add comment about autofs_mountpoint_changed() From: Ian Kent To: Al Viro Cc: autofs mailing list , linux-fsdevel , Kernel Mailing List Date: Thu, 26 Mar 2020 13:23:36 +0800 Message-ID: <158520021604.5325.4342529050022426912.stgit@mickey.themaw.net> In-Reply-To: <158520019862.5325.7856909810909592388.stgit@mickey.themaw.net> References: <158520019862.5325.7856909810909592388.stgit@mickey.themaw.net> User-Agent: StGit/unknown-version MIME-Version: 1.0 Sender: linux-fsdevel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org The function autofs_mountpoint_changed() is unusual, add a comment about two cases for which it is used. Signed-off-by: Ian Kent --- fs/autofs/root.c | 21 ++++++++++++++++++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/fs/autofs/root.c b/fs/autofs/root.c index 308cc49aca1d..a972bbaecb46 100644 --- a/fs/autofs/root.c +++ b/fs/autofs/root.c @@ -280,9 +280,24 @@ static struct dentry *autofs_mountpoint_changed(struct path *path) struct dentry *dentry = path->dentry; struct autofs_sb_info *sbi = autofs_sbi(dentry->d_sb); - /* - * If this is an indirect mount the dentry could have gone away - * as a result of an expire and a new one created. + /* If this is an indirect mount the dentry could have gone away + * and a new one created. + * + * This is unusual and I can't remember the case for which it + * was originally added now. But a example of how this can + * happen is an autofs indirect mount that has the "browse" + * option set and also has the "symlink" option in the autofs + * map entry. In this case the daemon will remove the browse + * directory and create a symlink as the mount (pointing to a + * local path) leaving the struct path stale. + * + * Another not so obvious case is when a mount in an autofs + * indirect mount that uses the "nobrowse" option is being + * expired and the mount has been umounted but the mount point + * directory remains when a stat family system call is made. + * In this case the mount point is removed (by the daemon) and + * a new mount triggered leading to a stale dentry in the struct + * path of the waiting process. */ if (autofs_type_indirect(sbi->type) && d_unhashed(dentry)) { struct dentry *parent = dentry->d_parent;