Re: [PATCH] f2fs: clear get_ssr_cost

On 2017/9/4 9:54, Yunlong Song wrote: > The update_sit_entry provides this: > ... > 1658 if (!f2fs_test_bit(offset, se->ckpt_valid_map)) > 1659 se->ckpt_valid_blocks += del; > ... > As a result, the ckpt_valid_blocks is always larger than valid_blocks. > If not correct, can you provide

Re: [PATCH] f2fs: clear get_ssr_cost

The update_sit_entry provides this: ... 1658 if (!f2fs_test_bit(offset, se->ckpt_valid_map)) 1659 se->ckpt_valid_blocks += del; ... As a result, the ckpt_valid_blocks is always larger than valid_blocks. If not correct, can you provide the case valid_blocks larger than ckpt_valid_bloc

Re: [PATCH] f2fs: clear get_ssr_cost

On 2017/9/1 20:14, Yunlong Song wrote: > se->ckpt_valid_blocks is always larger than se->valid_blocks, so > get_ssr_cost can be cleared. I think this is not correct. Thanks, > > Signed-off-by: Yunlong Song > --- > fs/f2fs/gc.c | 11 +-- > 1 file changed, 1 insertion(+), 10 deletions(-

[PATCH] f2fs: clear get_ssr_cost

se->ckpt_valid_blocks is always larger than se->valid_blocks, so get_ssr_cost can be cleared. Signed-off-by: Yunlong Song --- fs/f2fs/gc.c | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index cd147e7..b226760 100644 --- a/fs/f2fs/gc.c