On 6/16/25 13:04, yohan.joung wrote:
> to allow users to dynamically tune
> the boost_zoned_gc_percent parameter
>
> Signed-off-by: yohan.joung <yohan.jo...@sk.com>
> ---
> fs/f2fs/gc.h | 3 ++-
> fs/f2fs/sysfs.c | 5 +++++
> 2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/fs/f2fs/gc.h b/fs/f2fs/gc.h
> index 5c1eaf55e127..11fba7636af7 100644
> --- a/fs/f2fs/gc.h
> +++ b/fs/f2fs/gc.h
> @@ -194,6 +194,7 @@ static inline bool has_enough_invalid_blocks(struct
> f2fs_sb_info *sbi)
> static inline bool need_to_boost_gc(struct f2fs_sb_info *sbi)
> {
> if (f2fs_sb_has_blkzoned(sbi))
> - return !has_enough_free_blocks(sbi, LIMIT_BOOST_ZONED_GC);
> + return !has_enough_free_blocks(sbi,
> + sbi->gc_thread->boost_zoned_gc_percent);
> return has_enough_invalid_blocks(sbi);
> }
> diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
> index 75134d69a0bd..6c26b5e10c8f 100644
> --- a/fs/f2fs/sysfs.c
> +++ b/fs/f2fs/sysfs.c
> @@ -628,6 +628,11 @@ static ssize_t __sbi_store(struct f2fs_attr *a,
> return count;
> }
>
> + if (!strcmp(a->attr.name, "gc_boost_zoned_gc_percent")) {
> + if (t > 100)
> + return -EINVAL;
We'd better assign boost_zoned_gc_percent here and return directly to
avoid all below unnecessary comparison between a->attr.name and entry
names.
Thanks,
> + }
> +
> #ifdef CONFIG_F2FS_IOSTAT
> if (!strcmp(a->attr.name, "iostat_enable")) {
> sbi->iostat_enable = !!t;
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
