diff options
author | Pankaj Gupta <pagupta@redhat.com> | 2019-07-05 16:03:28 +0200 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2019-07-06 00:19:10 +0200 |
commit | b21fec414095d966789581c1466fb2f55de33bfe (patch) | |
tree | 86f35b782b92ed78f5063b8658bbac2966db0318 /fs/xfs/xfs_file.c | |
parent | ext4: disable map_sync for async flush (diff) | |
download | linux-b21fec414095d966789581c1466fb2f55de33bfe.tar.xz linux-b21fec414095d966789581c1466fb2f55de33bfe.zip |
xfs: disable map_sync for async flush
Dont support 'MAP_SYNC' with non-DAX files and DAX files
with asynchronous dax_device. Virtio pmem provides
asynchronous host page cache flush mechanism. We don't
support 'MAP_SYNC' with virtio pmem and xfs.
Signed-off-by: Pankaj Gupta <pagupta@redhat.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'fs/xfs/xfs_file.c')
-rw-r--r-- | fs/xfs/xfs_file.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/fs/xfs/xfs_file.c b/fs/xfs/xfs_file.c index 76748255f843..a06af58af475 100644 --- a/fs/xfs/xfs_file.c +++ b/fs/xfs/xfs_file.c @@ -1203,11 +1203,14 @@ xfs_file_mmap( struct file *filp, struct vm_area_struct *vma) { + struct dax_device *dax_dev; + + dax_dev = xfs_find_daxdev_for_inode(file_inode(filp)); /* - * We don't support synchronous mappings for non-DAX files. At least - * until someone comes with a sensible use case. + * We don't support synchronous mappings for non-DAX files and + * for DAX files if underneath dax_device is not synchronous. */ - if (!IS_DAX(file_inode(filp)) && (vma->vm_flags & VM_SYNC)) + if (!daxdev_mapping_supported(vma, dax_dev)) return -EOPNOTSUPP; file_accessed(filp); |