diff options
| author | Dave Chinner <dchinner@redhat.com> | 2012-11-12 11:53:57 (GMT) |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2012-11-14 21:15:08 (GMT) |
| commit | fb59581404ab7ec5075299065c22cb211a9262a9 (patch) | |
| tree | 56a9d9a6cf90ee9b6d753de4401b6a2bf8d02e0f /fs/xfs/xfs_vnodeops.c | |
| parent | 4bc1ea6b8ddd4f2bd78944fbe5a1042ac14b1f5f (diff) | |
| download | linux-fb59581404ab7ec5075299065c22cb211a9262a9.tar.xz | |
xfs: remove xfs_flushinval_pages
It's just a simple wrapper around VFS functionality, and is actually
bugging in that it doesn't remove mappings before invalidating the
page cache. Remove it and replace it with the correct VFS
functionality.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Andrew Dahl <adahl@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_vnodeops.c')
| -rw-r--r-- | fs/xfs/xfs_vnodeops.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/fs/xfs/xfs_vnodeops.c b/fs/xfs/xfs_vnodeops.c index c00326a..81c61fd 100644 --- a/fs/xfs/xfs_vnodeops.c +++ b/fs/xfs/xfs_vnodeops.c @@ -1958,12 +1958,11 @@ xfs_free_file_space( rounding = max_t(uint, 1 << mp->m_sb.sb_blocklog, PAGE_CACHE_SIZE); ioffset = offset & ~(rounding - 1); - - if (VN_CACHED(VFS_I(ip)) != 0) { - error = xfs_flushinval_pages(ip, ioffset, -1, FI_REMAPF_LOCKED); - if (error) - goto out_unlock_iolock; - } + error = -filemap_write_and_wait_range(VFS_I(ip)->i_mapping, + ioffset, -1); + if (error) + goto out_unlock_iolock; + truncate_pagecache_range(VFS_I(ip), ioffset, -1); /* * Need to zero the stuff we're not freeing, on disk. |