summaryrefslogtreecommitdiff
path: root/fs/xfs/libxfs/xfs_inode_fork.c
diff options
context:
space:
mode:
authorDave Chinner <dchinner@redhat.com>2014-08-04 13:44:08 +1000
committerDave Chinner <david@fromorbit.com>2014-08-04 13:44:08 +1000
commit4ef897a27543b513351262881660147366c042a1 (patch)
tree8b1bf48a105a81a61a3a1299b0cf49fd5508ef41 /fs/xfs/libxfs/xfs_inode_fork.c
parent812176832169c77b4bacddd01edc3e55340263fd (diff)
xfs: flush both inodes in xfs_swap_extents
We need to treat both inodes identically from a page cache point of view when prepareing them for extent swapping. We don't do this right now - we assume that one of the inodes empty, because that's what xfs_fsr currently does. Remove this assumption from the code. While factoring out the flushing and related checks, move the transactions reservation to immeidately after the flushes so that we don't need to pick up and then drop the ilock to do the transaction reservation. There are no issues with aborting the transaction it if the checks fail before we join the inodes to the transaction and dirty them, so this is a safe change to make. Signed-off-by: Dave Chinner <dchinner@redhat.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs/libxfs/xfs_inode_fork.c')
0 files changed, 0 insertions, 0 deletions