Module Name:    src
Committed By:   reinoud
Date:           Tue Jul 29 11:10:12 UTC 2014

Modified Files:
        src/sys/fs/udf: udf_vnops.c

Log Message:
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.


To generate a diff of this commit:
cvs rdiff -u -r1.92 -r1.93 src/sys/fs/udf/udf_vnops.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/fs/udf/udf_vnops.c
diff -u src/sys/fs/udf/udf_vnops.c:1.92 src/sys/fs/udf/udf_vnops.c:1.93
--- src/sys/fs/udf/udf_vnops.c:1.92	Fri Jul 25 08:20:52 2014
+++ src/sys/fs/udf/udf_vnops.c	Tue Jul 29 11:10:12 2014
@@ -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 @@ udf_chown(struct vnode *vp, uid_t new_ui
 
 	/* 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 @@ udf_chmod(struct vnode *vp, mode_t mode,
 
 	/* 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;
 }

Reply via email to