diff options
author | 2017-05-12 13:51:34 -0700 | |
---|---|---|
committer | 2017-05-23 21:07:15 -0700 | |
commit | cc15620bc826b14006956fd321e026ae96aff53a (patch) | |
tree | 1bb4cbe573d4eadbee4fb62ae9514b0499570a29 /fs/f2fs/segment.c | |
parent | f2fs: split bio cache (diff) | |
download | linux-cc15620bc826b14006956fd321e026ae96aff53a.tar.gz linux-cc15620bc826b14006956fd321e026ae96aff53a.tar.bz2 linux-cc15620bc826b14006956fd321e026ae96aff53a.zip |
f2fs: avoid f2fs_lock_op for IPU writes
Currently, if we do get_node_of_data before f2fs_lock_op, there may be dead lock
as follows, where process A would be in infinite loop, and B will NOT be awaked.
Process A(cp): Process B:
f2fs_lock_all(sbi)
get_dnode_of_data <---- lock dn.node_page
flush_nodes f2fs_lock_op
So, this patch adds f2fs_trylock_op to avoid f2fs_lock_op done by IPU.
Signed-off-by: Hou Pengyang <houpengyang@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs/segment.c')
-rw-r--r-- | fs/f2fs/segment.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index fcada9d03817..3bc36769ec9f 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs/segment.c @@ -312,7 +312,7 @@ static int __commit_inmem_pages(struct inode *inode, fio.page = page; fio.old_blkaddr = NULL_ADDR; fio.encrypted_page = NULL; - fio.need_lock = false, + fio.need_lock = LOCK_DONE; err = do_write_data_page(&fio); if (err) { unlock_page(page); |