diff options
| author | zhengliang <zhengliang6@huawei.com> | 2019-03-04 09:32:25 +0800 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2019-03-05 19:57:56 -0800 |
| commit | a0770e13c8da83bdb64738c0209ab02dd3cfff8b (patch) | |
| tree | 6490c1b76a13fba8587ab7d638440301f7247309 /include | |
| parent | 1ffdc3807589779626924eb600db784d3d943a08 (diff) | |
| download | linux-a0770e13c8da83bdb64738c0209ab02dd3cfff8b.tar.bz2 | |
f2fs: fix to data block override node segment by mistake
v4: Rearrange the previous three versions.
The following scenario could lead to data block override by mistake.
TASK A | TASK kworker | TASK B | TASK C
| | |
open | | |
write | | |
close | | |
| f2fs_write_data_pages | |
| f2fs_write_cache_pages | |
| f2fs_outplace_write_data | |
| f2fs_allocate_data_block (get block in seg S, | |
| S is full, and only | |
| have this valid data | |
| block) | |
| allocate_segment | |
| locate_dirty_segment (mark S as PRE) | |
| f2fs_submit_page_write (submit but is not | |
| written on dev) | |
unlink | | |
iput_final | | |
f2fs_drop_inode | | |
f2fs_truncate | | |
(not evict) | | |
| | write_checkpoint |
| | flush merged bio but not wait file data writeback |
| | set_prefree_as_free (mark S as FREE) |
| | | update NODE/DATA
| | | allocate_segment (select S)
| writeback done | |
So we need to guarantee io complete before truncate inode in f2fs_drop_inode.
Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Zheng Liang <zhengliang6@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions