]> git.baikalelectronics.ru Git - kernel.git/commit
f2fs: fix to avoid touching checkpointed data in get_victim()
authorChao Yu <yuchao0@huawei.com>
Wed, 24 Mar 2021 03:18:28 +0000 (11:18 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Fri, 26 Mar 2021 17:27:01 +0000 (10:27 -0700)
commit1dbda25a6cc9ce96d61bb2dd0c5867aec70f3f9c
treeb9db9f5471ba68730cf3e03f9cf6af0dbfe5b0ca
parent23accf82616d0130f64adcfeb4c099c8e7ce04f4
f2fs: fix to avoid touching checkpointed data in get_victim()

In CP disabling mode, there are two issues when using LFS or SSR | AT_SSR
mode to select victim:

1. LFS is set to find source section during GC, the victim should have
no checkpointed data, since after GC, section could not be set free for
reuse.

Previously, we only check valid chpt blocks in current segment rather
than section, fix it.

2. SSR | AT_SSR are set to find target segment for writes which can be
fully filled by checkpointed and newly written blocks, we should never
select such segment, otherwise it can cause panic or data corruption
during allocation, potential case is described as below:

 a) target segment has 'n' (n < 512) ckpt valid blocks
 b) GC migrates 'n' valid blocks to other segment (segment is still
    in dirty list)
 c) GC migrates '512 - n' blocks to target segment (segment has 'n'
    cp_vblocks and '512 - n' vblocks)
 d) If GC selects target segment via {AT,}SSR allocator, however there
    is no free space in targe segment.

Fixes: 08f7d04e15b9 ("f2fs: checkpoint disabling")
Fixes: c0102ba4fd6b ("f2fs: support age threshold based garbage collection")
Signed-off-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/f2fs.h
fs/f2fs/gc.c
fs/f2fs/segment.c
fs/f2fs/segment.h