Re: [PATCH] VFS: Suppress automount on [l]stat, [l]getxattr, etc.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 22, 2011 at 12:20 PM, Trond Myklebust
<Trond.Myklebust@xxxxxxxxxx> wrote:
>
> We need to audit not only NFS, but a bunch of stuff in the VFS.

Why? What has *anything*else* got to do with fixing a NFSv4 bug?

IOW, why the hell would we hold up one bugfix just because you might
want to look at something else too?

> For instance, what is the correct behaviour if I do a bind mount with
> the source being an automount directory? Currently, that doesn't set
> LOOKUP_DIRECTORY (they cant, since file may also be a mountpoint) and so
> the current behaviour means that they will fail to automount.

Umm. If there is any question at all what the correct behavior might
be, then pretty clearly the correct behavior is to let the user
decide.

And as pointed out, that's one of the *advantages* of
LOOKUP_DIRECTORY. Because it lets users decide.

Exatly because it gets automatically set for the 'slash at end of
name' case. So suddenly you can literally let the user say one or the
other by just letting the user add or remove the slash: "/mountpoint"
is not automount-followed, while "/mountpoint/" is. Nice, obvious, and
consistent interface. Which is exactly what we'd be looking for.

                 Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux