Commit c192f7a4 authored by Jaegeuk Kim's avatar Jaegeuk Kim

f2fs: find data segments across all the types

Previously, if type is CURSEG_HOT_DATA, we only check CURSEG_HOT_DATA only.
This patch fixes to search all the different types for SSR.
Reviewed-by: default avatarChao Yu <yuchao0@huawei.com>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent d0db7703
...@@ -1528,16 +1528,23 @@ static int get_ssr_segment(struct f2fs_sb_info *sbi, int type) ...@@ -1528,16 +1528,23 @@ static int get_ssr_segment(struct f2fs_sb_info *sbi, int type)
{ {
struct curseg_info *curseg = CURSEG_I(sbi, type); struct curseg_info *curseg = CURSEG_I(sbi, type);
const struct victim_selection *v_ops = DIRTY_I(sbi)->v_ops; const struct victim_selection *v_ops = DIRTY_I(sbi)->v_ops;
int i;
/* need_SSR() already forces to do this */
if (v_ops->get_victim(sbi, &(curseg)->next_segno, BG_GC, type, SSR))
return 1;
if (IS_NODESEG(type)) if (IS_NODESEG(type))
return v_ops->get_victim(sbi, return 0;
&(curseg)->next_segno, BG_GC, type, SSR);
/* For data segments, let's do SSR more intensively */ /* For data segments, let's do SSR more intensively */
for (; type >= CURSEG_HOT_DATA; type--) for (i = CURSEG_HOT_DATA; i <= CURSEG_COLD_DATA; i++) {
if (i == type)
continue;
if (v_ops->get_victim(sbi, &(curseg)->next_segno, if (v_ops->get_victim(sbi, &(curseg)->next_segno,
BG_GC, type, SSR)) BG_GC, i, SSR))
return 1; return 1;
}
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment