diff options
author | Dan Williams <dan.j.williams@intel.com> | 2017-12-21 21:18:27 -0500 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2018-01-19 19:50:53 -0500 |
commit | b4b5798cea8f40ab61f3a2c79a26314465dd83e3 (patch) | |
tree | 6af0b815e7de22664f6807867cc0f25c8c7ce0d8 /fs/ext2 | |
parent | 24f3478d664b1eaa6f8860d3aa521aebe51b2a62 (diff) |
ext2: auto disable dax instead of failing mount
Bring the ext2 filesystem in line with xfs that only warns and continues
when the "-o dax" option is specified to mount and the backing device
does not support dax. This is in preparation for removing dax support
from devices that do not enable get_user_pages() operations on dax
mappings. In other words 'gup' support is required and configurations
that were using so called 'page-less' dax will be converted back to
using the page cache.
Removing the broken 'page-less' dax support is a pre-requisite for
removing the "EXPERIMENTAL" warning when mounting a filesystem in dax
mode.
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'fs/ext2')
-rw-r--r-- | fs/ext2/super.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/fs/ext2/super.c b/fs/ext2/super.c index 7646818ab266..38f9222606ee 100644 --- a/fs/ext2/super.c +++ b/fs/ext2/super.c | |||
@@ -959,8 +959,11 @@ static int ext2_fill_super(struct super_block *sb, void *data, int silent) | |||
959 | 959 | ||
960 | if (sbi->s_mount_opt & EXT2_MOUNT_DAX) { | 960 | if (sbi->s_mount_opt & EXT2_MOUNT_DAX) { |
961 | err = bdev_dax_supported(sb, blocksize); | 961 | err = bdev_dax_supported(sb, blocksize); |
962 | if (err) | 962 | if (err) { |
963 | goto failed_mount; | 963 | ext2_msg(sb, KERN_ERR, |
964 | "DAX unsupported by block device. Turning off DAX."); | ||
965 | sbi->s_mount_opt &= ~EXT2_MOUNT_DAX; | ||
966 | } | ||
964 | } | 967 | } |
965 | 968 | ||
966 | /* If the blocksize doesn't match, re-read the thing.. */ | 969 | /* If the blocksize doesn't match, re-read the thing.. */ |