[src/trunk]: src/sys/fs/udf Access time should only be set on accessing the d...

[reinoud <reinoud%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/382a1a17919e
branches:  trunk
changeset: 331055:382a1a17919e
user:      reinoud <reinoud%NetBSD.org@localhost>
date:      Tue Jul 29 11:10:12 2014 +0000

description:
Access time should only be set on accessing the data or messing with the
contents, not on inode access rights and ownership changes. Should address
PR kern/49033 for UDF.

Test results now come clean for bugs related to this issue in the ATF.

diffstat:

 sys/fs/udf/udf_vnops.c |  8 ++------
 1 files changed, 2 insertions(+), 6 deletions(-)

diffs (36 lines):

diff -r fc6f6334800c -r 382a1a17919e sys/fs/udf/udf_vnops.c
--- a/sys/fs/udf/udf_vnops.c    Tue Jul 29 09:15:48 2014 +0000
+++ b/sys/fs/udf/udf_vnops.c    Tue Jul 29 11:10:12 2014 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: udf_vnops.c,v 1.92 2014/07/25 08:20:52 dholland Exp $ */
+/* $NetBSD: udf_vnops.c,v 1.93 2014/07/29 11:10:12 reinoud Exp $ */
 
 /*
  * Copyright (c) 2006, 2008 Reinoud Zandijk
@@ -32,7 +32,7 @@
 
 #include <sys/cdefs.h>
 #ifndef lint
-__KERNEL_RCSID(0, "$NetBSD: udf_vnops.c,v 1.92 2014/07/25 08:20:52 dholland Exp $");
+__KERNEL_RCSID(0, "$NetBSD: udf_vnops.c,v 1.93 2014/07/29 11:10:12 reinoud Exp $");
 #endif /* not lint */
 
 
@@ -1020,8 +1020,6 @@
 
        /* mark node changed */
        udf_node->i_flags |= IN_CHANGE;
-       if (vp->v_mount->mnt_flag & MNT_RELATIME)
-               udf_node->i_flags |= IN_ACCESS;
 
        return 0;
 }
@@ -1059,8 +1057,6 @@
 
        /* mark node changed */
        udf_node->i_flags |= IN_CHANGE;
-       if (vp->v_mount->mnt_flag & MNT_RELATIME)
-               udf_node->i_flags |= IN_ACCESS;
 
        return 0;
 }