lkml.org 
[lkml]   [2017]   [Sep]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 5/7] xfs: introduce xfs_is_dax_state_changing
Date
Pull this code out of xfs_ioctl_setattr_dax_invalidate() as it will be used
in multiple places soon.

Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
---
fs/xfs/xfs_ioctl.c | 34 +++++++++++++++++++++++-----------
1 file changed, 23 insertions(+), 11 deletions(-)

diff --git a/fs/xfs/xfs_ioctl.c b/fs/xfs/xfs_ioctl.c
index 0433aef..386b437 100644
--- a/fs/xfs/xfs_ioctl.c
+++ b/fs/xfs/xfs_ioctl.c
@@ -1020,6 +1020,27 @@ xfs_diflags_to_linux(
#endif
}

+static bool
+xfs_is_dax_state_changing(
+ unsigned int xflags,
+ struct xfs_inode *ip)
+{
+ struct inode *inode = VFS_I(ip);
+
+ /*
+ * If the DAX mount option was used we will update the DAX inode flag
+ * as the user requested but we will continue to use DAX for I/O and
+ * page faults regardless of how the inode flag is set.
+ */
+ if (ip->i_mount->m_flags & XFS_MOUNT_DAX)
+ return false;
+ if ((xflags & FS_XFLAG_DAX) && IS_DAX(inode))
+ return false;
+ if (!(xflags & FS_XFLAG_DAX) && !IS_DAX(inode))
+ return false;
+ return true;
+}
+
static int
xfs_ioctl_setattr_xflags(
struct xfs_trans *tp,
@@ -1105,17 +1126,8 @@ xfs_ioctl_setattr_dax_invalidate(
return -EINVAL;
}

- /*
- * If the DAX state is not changing, we have nothing to do here. If
- * the DAX mount option was used we will update the DAX inode flag as
- * the user requested but we will continue to use DAX for I/O and page
- * faults regardless of how the inode flag is set.
- */
- if (ip->i_mount->m_flags & XFS_MOUNT_DAX)
- return 0;
- if ((fa->fsx_xflags & FS_XFLAG_DAX) && IS_DAX(inode))
- return 0;
- if (!(fa->fsx_xflags & FS_XFLAG_DAX) && !IS_DAX(inode))
+ /* If the DAX state is not changing, we have nothing to do here. */
+ if (!xfs_is_dax_state_changing(fa->fsx_xflags, ip))
return 0;

/* lock, flush and invalidate mapping in preparation for flag change */
--
2.9.5
\
 
 \ /
  Last update: 2017-09-26 01:16    [W:0.454 / U:0.016 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site