diff options
| author |   DongDongJu <commisori28@gmail.com> | 2020-03-20 15:01:32 +0900 |
|---|---|---|
| committer |   Jaegeuk Kim <jaegeuk@kernel.org> | 2020-03-30 20:46:23 -0700 |
| commit | ad8d6a02d685ecf046c369d72285a5e69adaf66e (patch) | |
| tree | 991eeb86b5b95374a30e2849cea37fbaf94c1672 /fs/f2fs | |
| parent | f2fs: don't mark compressed inode dirty during f2fs_iget() (diff) | |
| download | wireguard-linux-ad8d6a02d685ecf046c369d72285a5e69adaf66e.tar.xz wireguard-linux-ad8d6a02d685ecf046c369d72285a5e69adaf66e.zip | |
f2fs: delete DIO read lock
This lock can be a contention with multi 4k random read IO with single inode.
example) fio --output=test --name=test --numjobs=60 --filename=/media/samsung960pro/file_test --rw=randread --bs=4k
--direct=1 --time_based --runtime=7 --ioengine=libaio --iodepth=256 --group_reporting --size=10G
With this commit, it remove that possible lock contention.
Signed-off-by: Dongjoo Seo <commisori28@gmail.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/data.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index dbde309349d0..326e6342c578 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -3453,7 +3453,8 @@ static ssize_t f2fs_direct_IO(struct kiocb *iocb, struct iov_iter *iter) err = __blockdev_direct_IO(iocb, inode, inode->i_sb->s_bdev, iter, rw == WRITE ? get_data_block_dio_write : get_data_block_dio, NULL, f2fs_dio_submit_bio, - DIO_LOCKING | DIO_SKIP_HOLES); + rw == WRITE ? DIO_LOCKING | DIO_SKIP_HOLES : + DIO_SKIP_HOLES); if (do_opu) up_read(&fi->i_gc_rwsem[READ]); |