On 2017/2/24 3:03, Jaegeuk Kim wrote: > Hi Pengyang, > > On 02/23, Hou Pengyang wrote: >> Signed-off-by: Hou Pengyang <houpengy...@huawei.com> >> --- >> fs/f2fs/gc.c | 13 +++++++++++-- >> 1 file changed, 11 insertions(+), 2 deletions(-) >> >> diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c >> index 6c996e3..eaf9e68 100644 >> --- a/fs/f2fs/gc.c >> +++ b/fs/f2fs/gc.c >> @@ -293,13 +293,22 @@ static int get_victim_by_default(struct f2fs_sb_info >> *sbi, >> unsigned int secno, last_victim; >> unsigned int last_segment = MAIN_SEGS(sbi); >> unsigned int nsearched = 0; >> + int nr_dirty; >> >> mutex_lock(&dirty_i->seglist_lock); >> >> p.alloc_mode = alloc_mode; >> - select_policy(sbi, gc_type, type, &p); >> - >> p.min_segno = NULL_SEGNO; >> + >> + if (alloc_mode == SSR) >> + nr_dirty = dirty_i->nr_dirty[type]; >> + else >> + nr_dirty = dirty_i->nr_dirty[DIRTY]; >> + >> + if (!nr_dirty) >> + goto out; >> + >> + select_policy(sbi, gc_type, type, &p); >> p.min_cost = get_max_cost(sbi, &p); >> >> if (p.max_search == 0) > > Here, max_search will return if there is no candidate.
Got it :) Thanks, > > Thanks, > >> -- >> 2.10.1 >> >> >> ------------------------------------------------------------------------------ >> Check out the vibrant tech community on one of the world's most >> engaging tech sites, SlashDot.org! http://sdm.link/slashdot >> _______________________________________________ >> Linux-f2fs-devel mailing list >> Linux-f2fs-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel > > . > ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Linux-f2fs-devel mailing list Linux-f2fs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel