Modify the large section so that only segments
with valid blocks are moved.
Remove Migration Window Granularity by searching
based on segment numbers instead of window search

Signed-off-by: yohan.joung <yohan.jo...@sk.com>
---
 Documentation/ABI/testing/sysfs-fs-f2fs |  8 --------
 fs/f2fs/gc.c                            | 15 +++++++++------
 fs/f2fs/gc.h                            |  2 +-
 fs/f2fs/super.c                         |  5 ++---
 fs/f2fs/sysfs.c                         |  7 -------
 5 files changed, 12 insertions(+), 25 deletions(-)

diff --git a/Documentation/ABI/testing/sysfs-fs-f2fs 
b/Documentation/ABI/testing/sysfs-fs-f2fs
index 59adb7dc6f9e..b65033730a2c 100644
--- a/Documentation/ABI/testing/sysfs-fs-f2fs
+++ b/Documentation/ABI/testing/sysfs-fs-f2fs
@@ -788,14 +788,6 @@ Description:       The zone UFS we are currently using 
consists of two parts:
                blkzone_alloc_policy = 2  Prioritize writing to conventional 
zones
                ========================  
=========================================
 
-What:          /sys/fs/f2fs/<disk>/migration_window_granularity
-Date:          September 2024
-Contact:       "Daeho Jeong" <daehoje...@google.com>
-Description:   Controls migration window granularity of garbage collection on 
large
-               section. it can control the scanning window granularity for GC 
migration
-               in a unit of segment, while migration_granularity controls the 
number
-               of segments which can be migrated at the same turn.
-
 What:          /sys/fs/f2fs/<disk>/reserved_segments
 Date:          September 2024
 Contact:       "Daeho Jeong" <daehoje...@google.com>
diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c
index fcd2cf68612d..3496c1523be7 100644
--- a/fs/f2fs/gc.c
+++ b/fs/f2fs/gc.c
@@ -1750,7 +1750,7 @@ static int do_garbage_collect(struct f2fs_sb_info *sbi,
        unsigned int segno = start_segno;
        unsigned int end_segno = start_segno + SEGS_PER_SEC(sbi);
        unsigned int sec_end_segno;
-       unsigned int window_granularity = 1;
+       unsigned int migration_granularity = sbi->migration_granularity;
        int seg_freed = 0, migrated = 0;
        unsigned char type = IS_DATASEG(get_seg_entry(sbi, segno)->type) ?
                                                SUM_TYPE_DATA : SUM_TYPE_NODE;
@@ -1773,29 +1773,32 @@ static int do_garbage_collect(struct f2fs_sb_info *sbi,
                                        f2fs_usable_segs_in_sec(sbi);
 
                if (gc_type == BG_GC || one_time) {
-                       window_granularity =
-                               sbi->migration_window_granularity;
 
                        if (f2fs_sb_has_blkzoned(sbi) &&
                                        !has_enough_free_blocks(sbi,
                                        sbi->gc_thread->boost_zoned_gc_percent))
-                               window_granularity *=
+                               migration_granularity *=
                                        BOOST_GC_MULTIPLE;
                }
 
+               if (gc_type == FG_GC) {
+                       migration_granularity = sec_end_segno - start_segno;
+               }
+
                if (end_segno > sec_end_segno)
                        end_segno = sec_end_segno;
        }
 
        sanity_check_seg_type(sbi, get_seg_entry(sbi, segno)->type);
 
+
        for (segno = start_segno; segno < end_segno; segno++) {
                if (!get_valid_blocks(sbi, segno, false))
                        continue;
 
                /* readahead multi ssa blocks those have contiguous address */
                f2fs_ra_meta_pages(sbi, GET_SUM_BLOCK(sbi, segno),
-                               window_granularity, META_SSA, true);
+                               migration_granularity, META_SSA, true);
 
                /* reference all summary page */
                sum_page = f2fs_get_sum_page(sbi, segno);
@@ -1806,7 +1809,7 @@ static int do_garbage_collect(struct f2fs_sb_info *sbi,
                }
                add_gc_page_entry(&gc_page_list, sum_page, segno);
                unlock_page(sum_page);
-               if (++gc_list_count >= window_granularity)
+               if (++gc_list_count >= migration_granularity)
                        break;
        }
 
diff --git a/fs/f2fs/gc.h b/fs/f2fs/gc.h
index 9c8695efe394..5e933c2110d2 100644
--- a/fs/f2fs/gc.h
+++ b/fs/f2fs/gc.h
@@ -33,7 +33,7 @@
 
 #define LIMIT_NO_ZONED_GC      60 /* percentage over total user space of no gc 
for zoned devices */
 #define LIMIT_BOOST_ZONED_GC   25 /* percentage over total user space of 
boosted gc for zoned devices */
-#define DEF_MIGRATION_WINDOW_GRANULARITY_ZONED 3
+#define DEF_MIGRATION_GRANULARITY_ZONED        3
 #define BOOST_GC_MULTIPLE      5
 #define ZONED_PIN_SEC_REQUIRED_COUNT   1
 
diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index a3241730fe4f..775a6e3cc6a6 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -3890,9 +3890,8 @@ static void init_sb_info(struct f2fs_sb_info *sbi)
        sbi->next_victim_seg[BG_GC] = NULL_SEGNO;
        sbi->next_victim_seg[FG_GC] = NULL_SEGNO;
        sbi->max_victim_search = DEF_MAX_VICTIM_SEARCH;
-       sbi->migration_granularity = SEGS_PER_SEC(sbi);
-       sbi->migration_window_granularity = f2fs_sb_has_blkzoned(sbi) ?
-               DEF_MIGRATION_WINDOW_GRANULARITY_ZONED : SEGS_PER_SEC(sbi);
+       sbi->migration_granularity = f2fs_sb_has_blkzoned(sbi) ?
+               DEF_MIGRATION_GRANULARITY_ZONED : SEGS_PER_SEC(sbi);
        sbi->seq_file_ra_mul = MIN_RA_MUL;
        sbi->max_fragment_chunk = DEF_FRAGMENT_SIZE;
        sbi->max_fragment_hole = DEF_FRAGMENT_SIZE;
diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
index c69161366467..6a47b9c9c267 100644
--- a/fs/f2fs/sysfs.c
+++ b/fs/f2fs/sysfs.c
@@ -568,11 +568,6 @@ static ssize_t __sbi_store(struct f2fs_attr *a,
                        return -EINVAL;
        }
 
-       if (!strcmp(a->attr.name, "migration_window_granularity")) {
-               if (t == 0 || t > SEGS_PER_SEC(sbi))
-                       return -EINVAL;
-       }
-
        if (!strcmp(a->attr.name, "gc_urgent")) {
                if (t == 0) {
                        sbi->gc_mode = GC_NORMAL;
@@ -1085,7 +1080,6 @@ F2FS_SBI_RW_ATTR(gc_pin_file_thresh, 
gc_pin_file_threshold);
 F2FS_SBI_RW_ATTR(gc_reclaimed_segments, gc_reclaimed_segs);
 F2FS_SBI_GENERAL_RW_ATTR(max_victim_search);
 F2FS_SBI_GENERAL_RW_ATTR(migration_granularity);
-F2FS_SBI_GENERAL_RW_ATTR(migration_window_granularity);
 F2FS_SBI_GENERAL_RW_ATTR(dir_level);
 #ifdef CONFIG_F2FS_IOSTAT
 F2FS_SBI_GENERAL_RW_ATTR(iostat_enable);
@@ -1234,7 +1228,6 @@ static struct attribute *f2fs_attrs[] = {
        ATTR_LIST(reserved_segments),
        ATTR_LIST(max_victim_search),
        ATTR_LIST(migration_granularity),
-       ATTR_LIST(migration_window_granularity),
        ATTR_LIST(dir_level),
        ATTR_LIST(ram_thresh),
        ATTR_LIST(ra_nid_pages),
-- 
2.33.0



_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to