diff options
author | Jan Kara <jack@suse.cz> | 2009-08-18 12:32:55 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2009-09-14 11:08:17 -0400 |
commit | af0f4414f343429971d33b0dd8dccc85c1f3dcd2 (patch) | |
tree | 1b053812591d19f618240672eb74fd74912c29c1 /fs/xfs | |
parent | d23c937b0f740888765676f6f82f509dbbb2bbad (diff) |
xfs: Convert sync_page_range() to simple filemap_write_and_wait_range()
Christoph Hellwig says that it is enough for XFS to call
filemap_write_and_wait_range() instead of sync_page_range() because we do
all the metadata syncing when forcing the log.
CC: Felix Blyakher <felixb@sgi.com>
CC: xfs@oss.sgi.com
CC: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/xfs')
-rw-r--r-- | fs/xfs/linux-2.6/xfs_lrw.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/xfs/linux-2.6/xfs_lrw.c b/fs/xfs/linux-2.6/xfs_lrw.c index 7078974a6eee..fde63a3c4ecc 100644 --- a/fs/xfs/linux-2.6/xfs_lrw.c +++ b/fs/xfs/linux-2.6/xfs_lrw.c | |||
@@ -817,7 +817,8 @@ write_retry: | |||
817 | xfs_iunlock(xip, iolock); | 817 | xfs_iunlock(xip, iolock); |
818 | if (need_i_mutex) | 818 | if (need_i_mutex) |
819 | mutex_unlock(&inode->i_mutex); | 819 | mutex_unlock(&inode->i_mutex); |
820 | error2 = sync_page_range(inode, mapping, pos, ret); | 820 | error2 = filemap_write_and_wait_range(mapping, pos, |
821 | pos + ret - 1); | ||
821 | if (!error) | 822 | if (!error) |
822 | error = error2; | 823 | error = error2; |
823 | if (need_i_mutex) | 824 | if (need_i_mutex) |