Commit c8dc3047 authored by Chao Yu's avatar Chao Yu Committed by Jaegeuk Kim

f2fs: fix to unmap pages from userspace process in punch_hole()

We need to unmap pages from userspace process before removing pagecache
in punch_hole() like we did in f2fs_setattr().

Similar change:
commit 5e44f8c3 ("ext4: hole-punch use truncate_pagecache_range")

Fixes: fbfa2cc5 ("f2fs: add file operations")
Signed-off-by: default avatarChao Yu <chao@kernel.org>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent adf9ea89
...@@ -1107,7 +1107,6 @@ static int punch_hole(struct inode *inode, loff_t offset, loff_t len) ...@@ -1107,7 +1107,6 @@ static int punch_hole(struct inode *inode, loff_t offset, loff_t len)
} }
if (pg_start < pg_end) { if (pg_start < pg_end) {
struct address_space *mapping = inode->i_mapping;
loff_t blk_start, blk_end; loff_t blk_start, blk_end;
struct f2fs_sb_info *sbi = F2FS_I_SB(inode); struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
...@@ -1119,8 +1118,7 @@ static int punch_hole(struct inode *inode, loff_t offset, loff_t len) ...@@ -1119,8 +1118,7 @@ static int punch_hole(struct inode *inode, loff_t offset, loff_t len)
down_write(&F2FS_I(inode)->i_gc_rwsem[WRITE]); down_write(&F2FS_I(inode)->i_gc_rwsem[WRITE]);
down_write(&F2FS_I(inode)->i_mmap_sem); down_write(&F2FS_I(inode)->i_mmap_sem);
truncate_inode_pages_range(mapping, blk_start, truncate_pagecache_range(inode, blk_start, blk_end - 1);
blk_end - 1);
f2fs_lock_op(sbi); f2fs_lock_op(sbi);
ret = f2fs_truncate_hole(inode, pg_start, pg_end); ret = f2fs_truncate_hole(inode, pg_start, pg_end);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment