]> git.ipfire.org Git - people/ms/linux.git/commit - fs/ext4/ioctl.c
ext4: prevent data corruption with journaling + DAX
authorRoss Zwisler <ross.zwisler@linux.intel.com>
Thu, 12 Oct 2017 15:54:08 +0000 (11:54 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Thu, 12 Oct 2017 15:54:08 +0000 (11:54 -0400)
commite9072d859df3e0f2c3ba450f0d1739595c2d5d13
tree8e91f9ce1bd1fdaf439d246e9d8045fd418a61d2
parent559db4c6d784ceedc2a5418ced4d357cb843e221
ext4: prevent data corruption with journaling + DAX

The current code has the potential for data corruption when changing an
inode's journaling mode, as that can result in a subsequent unsafe change
in S_DAX.

I've captured an instance of this data corruption in the following fstest:

https://patchwork.kernel.org/patch/9948377/

Prevent this data corruption from happening by disallowing changes to the
journaling mode if the '-o dax' mount option was used.  This means that for
a given filesystem we could have a mix of inodes using either DAX or
data journaling, but whatever state the inodes are in will be held for the
duration of the mount.

Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Reviewed-by: Jan Kara <jack@suse.cz>
Cc: stable@vger.kernel.org
fs/ext4/inode.c
fs/ext4/ioctl.c