Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1336035

[PATCH v3 3/6] ext4: Online defrag not supported with DAX

From Ross Zwisler <ross.zwisler@linux.intel.com>
Newsgroups linux.kernel
Subject [PATCH v3 3/6] ext4: Online defrag not supported with DAX
Date 2016-02-17 04:40 +0100
Message-ID <r3185-vI-1@gated-at.bofh.it> (permalink)
References <r3185-vI-3@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Online defrag operations for ext4 are hard coded to use the page cache.
See ext4_ioctl() -> ext4_move_extents() -> move_extent_per_page()

When combined with DAX I/O, which circumvents the page cache, this can
result in data corruption.  This was observed with xfstests ext4/307 and
ext4/308.

Fix this by only allowing online defrag for non-DAX files.

Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
---
 fs/ext4/ioctl.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/fs/ext4/ioctl.c b/fs/ext4/ioctl.c
index 0f6c369..e32c86f 100644
--- a/fs/ext4/ioctl.c
+++ b/fs/ext4/ioctl.c
@@ -583,6 +583,11 @@ group_extend_out:
 				 "Online defrag not supported with bigalloc");
 			err = -EOPNOTSUPP;
 			goto mext_out;
+		} else if (IS_DAX(inode)) {
+			ext4_msg(sb, KERN_ERR,
+				 "Online defrag not supported with DAX");
+			err = -EOPNOTSUPP;
+			goto mext_out;
 		}
 
 		err = mnt_want_write_file(filp);
-- 
2.5.0

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

[PATCH v3 3/6] ext4: Online defrag not supported with DAX Ross Zwisler <ross.zwisler@linux.intel.com> - 2016-02-17 04:40 +0100
  Re: [PATCH v3 3/6] ext4: Online defrag not supported with DAX Jan Kara <jack@suse.cz> - 2016-02-17 22:40 +0100
  Re: [PATCH v3 3/6] ext4: Online defrag not supported with DAX Ross Zwisler <ross.zwisler@linux.intel.com> - 2016-02-17 23:00 +0100
    Re: [PATCH v3 3/6] ext4: Online defrag not supported with DAX Jan Kara <jack@suse.cz> - 2016-02-17 23:20 +0100
    Re: [PATCH v3 3/6] ext4: Online defrag not supported with DAX Dave Chinner <david@fromorbit.com> - 2016-02-18 01:20 +0100

csiph-web