@@ -38,6 +38,7 @@
#include "xfs_dir2.h"
#include "xfs_trans_space.h"
#include "xfs_pnfs.h"
+#include "xfs_reflink.h"
#include <linux/capability.h>
#include <linux/xattr.h>
@@ -816,6 +817,14 @@ xfs_setattr_size(
}
/*
+ * CoW the EOF block of the file if it's necessary to avoid
+ * corrupting other files.
+ */
+ error = xfs_reflink_cow_eof_block(ip, newsize);
+ if (error)
+ return error;
+
+ /*
* We are going to log the inode size change in this transaction so
* any previous writes that are beyond the on disk EOF and the new
* EOF that have not been written out need to be written here. If we
@@ -1740,3 +1740,45 @@ out:
trace_xfs_reflink_unshare_error(ip, error, _RET_IP_);
return error;
}
+
+/*
+ * If we're trying to truncate a file whose last block is shared and the new
+ * size isn't aligned to a block boundary, we need to dirty that last block
+ * ahead of the VFS zeroing the page.
+ */
+int
+xfs_reflink_cow_eof_block(
+ struct xfs_inode *ip,
+ xfs_off_t newsize)
+{
+ struct xfs_mount *mp = ip->i_mount;
+ xfs_fileoff_t fbno;
+ xfs_off_t isize;
+ int error;
+
+ if (!xfs_is_reflink_inode(ip) ||
+ (newsize & ((1 << VFS_I(ip)->i_blkbits) - 1)) == 0)
+ return 0;
+
+ /* Try to CoW the shared last block */
+ xfs_ilock(ip, XFS_ILOCK_EXCL);
+ fbno = XFS_B_TO_FSBT(mp, newsize);
+ isize = i_size_read(VFS_I(ip));
+
+ if (newsize > isize)
+ trace_xfs_reflink_cow_eof_block(ip, isize, newsize - isize);
+ else
+ trace_xfs_reflink_cow_eof_block(ip, newsize, isize - newsize);
+
+ error = xfs_reflink_dirty_extents(ip, fbno, fbno + 1, isize);
+ if (error)
+ goto out_unlock;
+ xfs_iunlock(ip, XFS_ILOCK_EXCL);
+
+ return 0;
+
+out_unlock:
+ xfs_iunlock(ip, XFS_ILOCK_EXCL);
+ trace_xfs_reflink_cow_eof_block_error(ip, error, _RET_IP_);
+ return error;
+}
@@ -50,6 +50,7 @@ extern int xfs_reflink_remap_range(struct xfs_inode *src, xfs_off_t srcoff,
unsigned int flags);
extern int xfs_reflink_unshare(struct xfs_inode *ip, xfs_off_t offset,
xfs_off_t len);
+extern int xfs_reflink_cow_eof_block(struct xfs_inode *ip, xfs_off_t newsize);
/* xfs_aops.c */
extern int xfs_map_cow_blocks(struct inode *inode, xfs_off_t offset,
When shrinking a file, the VFS zeroes everything in the associated page between the new EOF and the previous EOF to avoid leaking data. If this block is shared we need to CoW it before the VFS does its zeroing to avoid corrupting the other files. Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> --- fs/xfs/xfs_iops.c | 9 +++++++++ fs/xfs/xfs_reflink.c | 42 ++++++++++++++++++++++++++++++++++++++++++ fs/xfs/xfs_reflink.h | 1 + 3 files changed, 52 insertions(+) -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html