Re: [PATCH] f2fs: clean up codes in f2fs_gc related with write_checkpoint

2017-02-23 Thread Chao Yu
3On 2017/2/21 20:43, Yunlong Song wrote: > Signed-off-by: Yunlong Song Reviewed-by: Chao Yu

Re: [PATCH] f2fs: clean up codes in f2fs_gc related with write_checkpoint

2017-02-22 Thread Jaegeuk Kim
On 02/23, Yunlong Song wrote: > Ping... > > Since has_not_enough_free_secs(sbi, 0, 0) must be true if > has_not_enough_free_secs(sbi, sec_freed, 0) is true, > write_checkpoint is sure to execute in both conditions, and segno is > NULL_SEGNO in both conditions. Looks good to me, but applied like

Re: [PATCH] f2fs: clean up codes in f2fs_gc related with write_checkpoint

2017-02-22 Thread Yunlong Song
Ping... Since has_not_enough_free_secs(sbi, 0, 0) must be true if has_not_enough_free_secs(sbi, sec_freed, 0) is true, write_checkpoint is sure to execute in both conditions, and segno is NULL_SEGNO in both conditions. On 2017/2/21 20:43, Yunlong Song wrote: > Signed-off-by: Yunlong Song > ---

[PATCH] f2fs: clean up codes in f2fs_gc related with write_checkpoint

2017-02-21 Thread Yunlong Song
Signed-off-by: Yunlong Song --- fs/f2fs/gc.c | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c index 88e5e7b..e5f2569 100644 --- a/fs/f2fs/gc.c +++ b/fs/f2fs/gc.c @@ -943,23 +943,15 @@ int f2fs_gc(struct f2fs_sb_info *sbi, bool sync,