[lkml]   [2011]   [Sep]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] vfs: automount should ignore LOOKUP_FOLLOW
    On Tue, 2011-09-06 at 10:09 +0200, Miklos Szeredi wrote:
    > Ian Kent <> writes:
    > > On Mon, 2011-09-05 at 19:02 +0200, Miklos Szeredi wrote:
    > >>
    > >> If automounting on lstat(2) is the correct behavior (is it? why?) then at
    > >> least it should be enabled by a global switch or mount option, IMO.
    > >
    > > Ideally we wouldn't need to take special precautions for these
    > > operations at all but we can't, especially for GUI environments that
    > > constantly scan file systems on mount/umount activity.
    > >
    > > Historically for autofs, neither stat(2) or lstat(2) would trigger a
    > > mount. With the current implementation stat(2) now does but lstat(2)
    > > doesn't which is a step in the right direction IMHO. So, I recommend we
    > > continue to encourage user space to make the needed changes so we
    > > continue to move in the right direction, and yes, I acknowledge it is a
    > > pain but it'll never get done otherwise.
    > I'm not quite convinced. What's the advantage of triggering automount
    > on stat(2)?

    You get the information of the directory of the mounted fs and for many
    peoples purposes automounting should be transparent so that would be

    > Has anybody complained that stat(2) on the mountpoint doesn't cause an
    > automount?

    Yes, based on the reasoning above.

    I have plead my case on that a number of times and because I have had no
    choice but to continue with it the users have no choice but to accept
    it. For these people I recommend not using the "browse" option with
    autofs mount maps.

    For autofs this is only a problem for mount maps that use "browse"
    option. Basically ones where the mount point directory already exists
    and is later mounted upon. When the dentry doesn't exist exist a mount
    needs to be performed, at least it is needed to maintain previous
    semantics. Other users of automounting are usually mounting on
    directories that already exist, such as NFS crossing mount point
    directories. I'm pretty sure the automount should be transparent for
    them and similarly for CIFS DFS mounts.


     \ /
      Last update: 2011-09-06 16:41    [W:0.024 / U:16.804 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site