diff options
| author | Chao Yu | 2018-04-04 17:35:13 +0800 |
|---|---|---|
| committer | Jaegeuk Kim | 2018-05-31 11:31:45 -0700 |
| commit | 47cca3d62af20e1343cbd0a08b1f45e83892d82b (patch) | |
| tree | 1157f1c1146086effe880de333473d1f189f5b5f /fs/f2fs | |
| parent | 8045249829c9db0e664b3fa2f9c9b004089a6156 (diff) | |
f2fs: remove redundant block plug
For buffered IO, we don't need to use block plug to cache bio,
for direct IO, generic f2fs_direct_IO has already added block
plug, so let's remove redundant one in .write_iter.
As Yunlei described in his patch:
-f2fs_file_write_iter
-blk_start_plug
-__generic_file_write_iter
...
-do_blockdev_direct_IO
-blk_start_plug
...
-blk_finish_plug
...
-blk_finish_plug
which may conduct performance decrease in our platform
Signed-off-by: Yunlei He <heyunlei@huawei.com>
Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/file.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index ff6a59c1a3a3..f968c0e3b3e3 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -2886,7 +2886,6 @@ static ssize_t f2fs_file_write_iter(struct kiocb *iocb, struct iov_iter *from) { struct file *file = iocb->ki_filp; struct inode *inode = file_inode(file); - struct blk_plug plug; ssize_t ret; if (unlikely(f2fs_cp_error(F2FS_I_SB(inode)))) @@ -2931,9 +2930,7 @@ static ssize_t f2fs_file_write_iter(struct kiocb *iocb, struct iov_iter *from) return err; } } - blk_start_plug(&plug); ret = __generic_file_write_iter(iocb, from); - blk_finish_plug(&plug); clear_inode_flag(inode, FI_NO_PREALLOC); /* if we couldn't write data, we should deallocate blocks. */ |