diff options
author | Chao Yu <yuchao0@huawei.com> | 2017-03-22 07:45:05 +0100 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2017-03-24 20:10:50 +0100 |
commit | 30a61ddf8117c26ac5b295e1233eaa9629a94ca3 (patch) | |
tree | 24bada44d86ff08516389a908570a786d5106dae /fs/f2fs/debug.c | |
parent | f2fs: use set_page_private marcro in f2fs_trace_pid (diff) | |
download | linux-30a61ddf8117c26ac5b295e1233eaa9629a94ca3.tar.xz linux-30a61ddf8117c26ac5b295e1233eaa9629a94ca3.zip |
f2fs: fix race condition in between free nid allocator/initializer
In below concurrent case, allocated nid can be loaded into free nid cache
and be allocated again.
Thread A Thread B
- f2fs_create
- f2fs_new_inode
- alloc_nid
- __insert_nid_to_list(ALLOC_NID_LIST)
- f2fs_balance_fs_bg
- build_free_nids
- __build_free_nids
- scan_nat_page
- add_free_nid
- __lookup_nat_cache
- f2fs_add_link
- init_inode_metadata
- new_inode_page
- new_node_page
- set_node_addr
- alloc_nid_done
- __remove_nid_from_list(ALLOC_NID_LIST)
- __insert_nid_to_list(FREE_NID_LIST)
This patch makes nat cache lookup and free nid list operation being atomical
to avoid this race condition.
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs/debug.c')
0 files changed, 0 insertions, 0 deletions