Skip to contentSkip to conversation

move MS_NOATIME mirroring inside xfs

PATCH In review Christoph Hellwig 1 participant 2 messages Dec 13, 2005 XFS filesystem

lore
Share & tools
lore
#0 lore raw

PATCH 5/6 move MS_NOATIME mirroring inside xfs

XFS propagates MS_NOATIME through two levels internally but doesn't actually use it. Kill this dead code.

fs/xfs/linux-2.6/xfs_super.c +0 -2
Index: linux-2.6.15-rc5/fs/xfs/linux-2.6/xfs_super.c
===================================================================
@@ -76,8 +76,6 @@
 	strncpy(args->fsname, sb->s_id, MAXNAMELEN);
 
 	/* Copy the already-parsed mount(2) flags we're interested in */
-	if (sb->s_flags & MS_NOATIME)
-		args->flags |= XFSMNT_NOATIME;
 	if (sb->s_flags & MS_DIRSYNC)
 		args->flags |= XFSMNT_DIRSYNC;
 	if (sb->s_flags & MS_SYNCHRONOUS)
fs/xfs/xfs_clnt.h +0 -2
Index: linux-2.6.15-rc5/fs/xfs/xfs_clnt.h
===================================================================
@@ -68,8 +68,6 @@
 						 * enforcement */
 #define XFSMNT_PQUOTAENF	0x00000040	/* IRIX project quota limit
 						 * enforcement */
-#define XFSMNT_NOATIME		0x00000100	/* don't modify access
-						 * times on reads */
 #define XFSMNT_NOALIGN		0x00000200	/* don't allocate at
 						 * stripe boundaries*/
 #define XFSMNT_RETERR		0x00000400	/* return error to user */
fs/xfs/xfs_mount.h +0 -2
Index: linux-2.6.15-rc5/fs/xfs/xfs_mount.h
===================================================================
@@ -387,8 +387,6 @@
 #define XFS_MOUNT_FS_SHUTDOWN	(1ULL << 4)	/* atomic stop of all filesystem
 						   operations, typically for
 						   disk errors in metadata */
-#define XFS_MOUNT_NOATIME	(1ULL << 5)	/* don't modify inode access
-						   times on reads */
 #define XFS_MOUNT_RETERR	(1ULL << 6)     /* return alignment errors to
 						   user */
 #define XFS_MOUNT_NOALIGN	(1ULL << 7)	/* turn off stripe alignment
fs/xfs/xfs_vfsops.c +0 -7
Index: linux-2.6.15-rc5/fs/xfs/xfs_vfsops.c
===================================================================
@@ -258,8 +258,6 @@
 		mp->m_inoadd = XFS_INO64_OFFSET;
 	}
 #endif
-	if (ap->flags & XFSMNT_NOATIME)
-		mp->m_flags |= XFS_MOUNT_NOATIME;
 	if (ap->flags & XFSMNT_RETERR)
 		mp->m_flags |= XFS_MOUNT_RETERR;
 	if (ap->flags & XFSMNT_NOALIGN)
@@ -654,11 +652,6 @@
 	xfs_mount_t	*mp = XFS_BHVTOM(bdp);
 	int		error;
 
-	if (args->flags & XFSMNT_NOATIME)
-		mp->m_flags |= XFS_MOUNT_NOATIME;
-	else
-		mp->m_flags &= ~XFS_MOUNT_NOATIME;
-
 	if (args->flags & XFSMNT_BARRIER)
 		mp->m_flags |= XFS_MOUNT_BARRIER;
 	else
#1 lore raw
Quoted 2 lines · Christoph Hellwig

XFS propagates MS_NOATIME through two levels internally but doesn't actually use it. Kill this dead code.

The subject should read "remove MS_NOATIME mirroring inside xfs" of course..