diff options
| author | Chao Yu <chao2.yu@samsung.com> | 2013-11-22 16:52:50 +0800 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-12-23 10:18:04 +0900 |
| commit | a66c7b2fcfbc9ef4e972f6bc2b63d72d00f23122 (patch) | |
| tree | 81c28b128fc6316e20ac1a8ae697d7b2edc8aa0d | |
| parent | 031fa8cc9ba45c14f440b9cf71d09950fbe5eb9b (diff) | |
| download | blackbird-op-linux-a66c7b2fcfbc9ef4e972f6bc2b63d72d00f23122.tar.gz blackbird-op-linux-a66c7b2fcfbc9ef4e972f6bc2b63d72d00f23122.zip | |
f2fs: remove unneeded code in punch_hole
Because FALLOC_FL_PUNCH_HOLE flag must be ORed with FALLOC_FL_KEEP_SIZE
in fallocate, so we could remove the useless 'keep size' branch code which
will never be excuted in punch_hole.
Signed-off-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Fan Li <fanofcode.li@samsung.com>
[Jaegeuk Kim: remove an unnecessary parameter togather]
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
| -rw-r--r-- | fs/f2fs/file.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index 1cd8e44b637f..2b47adcd852a 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -444,7 +444,7 @@ int truncate_hole(struct inode *inode, pgoff_t pg_start, pgoff_t pg_end) return 0; } -static int punch_hole(struct inode *inode, loff_t offset, loff_t len, int mode) +static int punch_hole(struct inode *inode, loff_t offset, loff_t len) { pgoff_t pg_start, pg_end; loff_t off_start, off_end; @@ -484,12 +484,6 @@ static int punch_hole(struct inode *inode, loff_t offset, loff_t len, int mode) } } - if (!(mode & FALLOC_FL_KEEP_SIZE) && - i_size_read(inode) <= (offset + len)) { - i_size_write(inode, offset); - mark_inode_dirty(inode); - } - return ret; } @@ -552,7 +546,7 @@ static long f2fs_fallocate(struct file *file, int mode, return -EOPNOTSUPP; if (mode & FALLOC_FL_PUNCH_HOLE) - ret = punch_hole(inode, offset, len, mode); + ret = punch_hole(inode, offset, len); else ret = expand_inode_data(inode, offset, len, mode); |
