Re: [OpenAFS-devel] Linux atime nits

[Marc Aurele La France]

On Wed, 6 Jan 2010, Simon Wilkinson wrote:

Marc Aurele La France wrote:

src/afs/LINUX*/osi_file.c (all versions) should set the inode's S_NOATIME, instead of MS_NOATIME, as the latter is a super block flag, which in recent kernels collides with another inode flag. This means preventing atime updates for cache files isn't currently effective.

The split between MS_NOATIME (as a super block flag) and S_NOATIME (as an inode flag) seems to have happened with 2.4.0.

I've put a patch to address this into gerrit as
http://gerrit.openafs.org/1070

Also, given AFS itself doesn't appear to track access times, shouldn't src/afs/LINUX*/osi_vfsops.c (again, all versions) set MS_NOATIME in its super block?

Yes, I think it probably should. A patch for this is in gerrit as
http://gerrit.openafs.org/1071

Any chance of incorporating these into stable?

Thanks.

Marc.

+----------------------------------+----------------------------------+
|  Marc Aurele La France           |  work:   1-780-492-9310          |
|  Academic Information and        |  fax:    1-780-492-1729          |
|    Communications Technologies   |  email:  t...@ualberta.ca         |
|  352 General Services Building   +----------------------------------+
|  University of Alberta           |                                  |
|  Edmonton, Alberta               |    Standard disclaimers apply    |
|  T6G 2H1                         |                                  |
|  CANADA                          |                                  |
+----------------------------------+----------------------------------+
XFree86 developer and VP.  ATI driver and X server internals.
_______________________________________________
OpenAFS-devel mailing list
OpenAFS-devel@openafs.org
https://lists.openafs.org/mailman/listinfo/openafs-devel