Re: [Patch 2/6] Bind Mount Extensions 0.06

[raven]

On Thu, 24 Feb 2005, Herbert Poetzl wrote:

> On Wed, Feb 23, 2005 at 11:01:05PM +0000, Christoph Hellwig wrote:
> > On Tue, Feb 22, 2005 at 01:11:29PM +0100, Herbert Poetzl wrote:
> > > ;
> > > ; Bind Mount Extensions
> > > ;
> > > ; This part adds the required checks for touch_atime() to allow
> > > ; for vfsmount based NOATIME and NODIRATIME
> > > ; autofs4 update_atime is the only exception (ignored on purpose)
> >
> > and that purpose is?
>
> this is based on a statement from Al Viro:
>
> | autofs4 use - AFAICS there we want atime updated unconditionally,
> | so calling update_atime() (update atime after checking
> | noatime/nodiratime/readonly flags) is wrong.
>
> agreed, maybe a proper fix would be better ...
>
> > Did you discuss this with the autofs maintainers?

I've had a look at the patch and I can't see any problem.

autofs4 doesn't use the inode atime for expire purposes but does update it 
in sync with its dentry info struct field that is used for this purpose.

So AFAICS the atime is an external view of expire status, 
but only when updated within the autofs4 module VFS callbacks.

I haven't yet looked at the v3 (autofs) module.
I'll get back if I see an issue there.

Ian

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/