* [f2fs-dev] [PATCH] f2fs: export ipu policy in debugfs
@ 2023-02-09 3:40 Yangtao Li via Linux-f2fs-devel
2023-02-09 17:26 ` Jaegeuk Kim
0 siblings, 1 reply; 3+ messages in thread
From: Yangtao Li via Linux-f2fs-devel @ 2023-02-09 3:40 UTC (permalink / raw)
To: jaegeuk, chao; +Cc: Yangtao Li, linux-kernel, linux-f2fs-devel
Export ipu_policy as a string in debugfs for better readability and
it can help us better understand some strategies of the file system.
Since we use ipu_policy as a bitmap, and the bitmap API parameter is
unsigned long type data, let's change ipu_policy to unsigned long type.
Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
fs/f2fs/debug.c | 64 ++++++++++++++++++++++++++++++++---------------
fs/f2fs/f2fs.h | 4 +--
fs/f2fs/segment.h | 1 +
fs/f2fs/sysfs.c | 2 +-
4 files changed, 48 insertions(+), 23 deletions(-)
diff --git a/fs/f2fs/debug.c b/fs/f2fs/debug.c
index 32af4f0c5735..d50dc8e6abfd 100644
--- a/fs/f2fs/debug.c
+++ b/fs/f2fs/debug.c
@@ -354,6 +354,17 @@ static char *s_flag[] = {
[SBI_IS_FREEZING] = " freezefs",
};
+static const char *ipu_mode_names[F2FS_IPU_MAX] = {
+ [F2FS_IPU_FORCE] = "FORCE",
+ [F2FS_IPU_SSR] = "SSR",
+ [F2FS_IPU_UTIL] = "UTIL",
+ [F2FS_IPU_SSR_UTIL] = "SSR_UTIL",
+ [F2FS_IPU_FSYNC] = "FSYNC",
+ [F2FS_IPU_ASYNC] = "ASYNC",
+ [F2FS_IPU_NOCACHE] = "NOCACHE",
+ [F2FS_IPU_HONOR_OPU_WRITE] = "HONOR_OPU_WRITE",
+};
+
static int stat_show(struct seq_file *s, void *v)
{
struct f2fs_stat_info *si;
@@ -362,18 +373,20 @@ static int stat_show(struct seq_file *s, void *v)
raw_spin_lock_irqsave(&f2fs_stat_lock, flags);
list_for_each_entry(si, &f2fs_stat_list, stat_list) {
- update_general_status(si->sbi);
+ struct f2fs_sb_info *sbi = si->sbi;
+
+ update_general_status(sbi);
seq_printf(s, "\n=====[ partition info(%pg). #%d, %s, CP: %s]=====\n",
- si->sbi->sb->s_bdev, i++,
- f2fs_readonly(si->sbi->sb) ? "RO" : "RW",
- is_set_ckpt_flags(si->sbi, CP_DISABLED_FLAG) ?
- "Disabled" : (f2fs_cp_error(si->sbi) ? "Error" : "Good"));
- if (si->sbi->s_flag) {
+ sbi->sb->s_bdev, i++,
+ f2fs_readonly(sbi->sb) ? "RO" : "RW",
+ is_set_ckpt_flags(sbi, CP_DISABLED_FLAG) ?
+ "Disabled" : (f2fs_cp_error(sbi) ? "Error" : "Good"));
+ if (sbi->s_flag) {
seq_puts(s, "[SBI:");
- for_each_set_bit(j, &si->sbi->s_flag, 32)
+ for_each_set_bit(j, &sbi->s_flag, 32)
seq_puts(s, s_flag[j]);
- seq_puts(s, "]\n");
+ seq_puts(s, "]\n\n");
}
seq_printf(s, "[SB: 1] [CP: 2] [SIT: %d] [NAT: %d] ",
si->sit_area_segs, si->nat_area_segs);
@@ -383,8 +396,19 @@ static int stat_show(struct seq_file *s, void *v)
si->overp_segs, si->rsvd_segs);
seq_printf(s, "Current Time Sec: %llu / Mounted Time Sec: %llu\n\n",
ktime_get_boottime_seconds(),
- SIT_I(si->sbi)->mounted_time);
- if (test_opt(si->sbi, DISCARD))
+ SIT_I(sbi)->mounted_time);
+
+ seq_puts(s, "Policy:\n");
+ seq_puts(s, " - IPU: [");
+ if (IS_F2FS_IPU_DISABLE(sbi)) {
+ seq_puts(s, " DISABLE\n");
+ } else {
+ for_each_set_bit(j, &SM_I(sbi)->ipu_policy, F2FS_IPU_MAX)
+ seq_printf(s, " %s", ipu_mode_names[j]);
+ }
+ seq_puts(s, " ]\n");
+
+ if (test_opt(sbi, DISCARD))
seq_printf(s, "Utilization: %u%% (%u valid blocks, %u discard blocks)\n",
si->utilization, si->valid_count, si->discard_blks);
else
@@ -491,15 +515,15 @@ static int stat_show(struct seq_file *s, void *v)
seq_printf(s, " - node segments : %d (%d)\n",
si->node_segs, si->bg_node_segs);
seq_puts(s, " - Reclaimed segs :\n");
- seq_printf(s, " - Normal : %d\n", si->sbi->gc_reclaimed_segs[GC_NORMAL]);
- seq_printf(s, " - Idle CB : %d\n", si->sbi->gc_reclaimed_segs[GC_IDLE_CB]);
+ seq_printf(s, " - Normal : %d\n", sbi->gc_reclaimed_segs[GC_NORMAL]);
+ seq_printf(s, " - Idle CB : %d\n", sbi->gc_reclaimed_segs[GC_IDLE_CB]);
seq_printf(s, " - Idle Greedy : %d\n",
- si->sbi->gc_reclaimed_segs[GC_IDLE_GREEDY]);
- seq_printf(s, " - Idle AT : %d\n", si->sbi->gc_reclaimed_segs[GC_IDLE_AT]);
+ sbi->gc_reclaimed_segs[GC_IDLE_GREEDY]);
+ seq_printf(s, " - Idle AT : %d\n", sbi->gc_reclaimed_segs[GC_IDLE_AT]);
seq_printf(s, " - Urgent High : %d\n",
- si->sbi->gc_reclaimed_segs[GC_URGENT_HIGH]);
- seq_printf(s, " - Urgent Mid : %d\n", si->sbi->gc_reclaimed_segs[GC_URGENT_MID]);
- seq_printf(s, " - Urgent Low : %d\n", si->sbi->gc_reclaimed_segs[GC_URGENT_LOW]);
+ sbi->gc_reclaimed_segs[GC_URGENT_HIGH]);
+ seq_printf(s, " - Urgent Mid : %d\n", sbi->gc_reclaimed_segs[GC_URGENT_MID]);
+ seq_printf(s, " - Urgent Low : %d\n", sbi->gc_reclaimed_segs[GC_URGENT_LOW]);
seq_printf(s, "Try to move %d blocks (BG: %d)\n", si->tot_blks,
si->bg_data_blks + si->bg_node_blks);
seq_printf(s, " - data blocks : %d (%d)\n", si->data_blks,
@@ -565,7 +589,7 @@ static int stat_show(struct seq_file *s, void *v)
si->ndirty_imeta);
seq_printf(s, " - fsync mark: %4lld\n",
percpu_counter_sum_positive(
- &si->sbi->rf_node_block_count));
+ &sbi->rf_node_block_count));
seq_printf(s, " - NATs: %9d/%9d\n - SITs: %9d/%9d\n",
si->dirty_nats, si->nats, si->dirty_sits, si->sits);
seq_printf(s, " - free_nids: %9d/%9d\n - alloc_nids: %9d\n",
@@ -592,12 +616,12 @@ static int stat_show(struct seq_file *s, void *v)
si->block_count[LFS], si->segment_count[LFS]);
/* segment usage info */
- f2fs_update_sit_info(si->sbi);
+ f2fs_update_sit_info(sbi);
seq_printf(s, "\nBDF: %u, avg. vblocks: %u\n",
si->bimodal, si->avg_vblocks);
/* memory footprint */
- update_mem_info(si->sbi);
+ update_mem_info(sbi);
seq_printf(s, "\nMemory: %llu KB\n",
(si->base_mem + si->cache_mem + si->page_mem) >> 10);
seq_printf(s, " - static: %llu KB\n",
diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
index 44f2d76525bf..b699ed74f438 100644
--- a/fs/f2fs/f2fs.h
+++ b/fs/f2fs/f2fs.h
@@ -1070,7 +1070,7 @@ struct f2fs_sm_info {
struct list_head sit_entry_set; /* sit entry set list */
- unsigned int ipu_policy; /* in-place-update policy */
+ unsigned long ipu_policy; /* in-place-update policy */
unsigned int min_ipu_util; /* in-place-update threshold */
unsigned int min_fsync_blocks; /* threshold for fsync */
unsigned int min_seq_blocks; /* threshold for sequential blocks */
@@ -1322,7 +1322,7 @@ enum {
MAX_TIME,
};
-/* Note that you need to keep synchronization with this gc_mode_names array */
+/* Modification on enum should be synchronized with gc_mode_names array */
enum {
GC_NORMAL,
GC_IDLE_CB,
diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h
index 8ee5e5db9287..92c8be00d396 100644
--- a/fs/f2fs/segment.h
+++ b/fs/f2fs/segment.h
@@ -672,6 +672,7 @@ static inline int utilization(struct f2fs_sb_info *sbi)
#define F2FS_IPU_DISABLE 0
+/* Modification on enum should be synchronized with ipu_mode_names array */
enum {
F2FS_IPU_FORCE,
F2FS_IPU_SSR,
diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
index 6082e132257a..228df201f6d4 100644
--- a/fs/f2fs/sysfs.c
+++ b/fs/f2fs/sysfs.c
@@ -711,7 +711,7 @@ static ssize_t __sbi_store(struct f2fs_attr *a,
return -EINVAL;
if (t && f2fs_lfs_mode(sbi))
return -EINVAL;
- SM_I(sbi)->ipu_policy = (unsigned int)t;
+ SM_I(sbi)->ipu_policy = t;
return count;
}
--
2.25.1
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [f2fs-dev] [PATCH] f2fs: export ipu policy in debugfs
2023-02-09 3:40 [f2fs-dev] [PATCH] f2fs: export ipu policy in debugfs Yangtao Li via Linux-f2fs-devel
@ 2023-02-09 17:26 ` Jaegeuk Kim
2023-02-13 9:29 ` Chao Yu
0 siblings, 1 reply; 3+ messages in thread
From: Jaegeuk Kim @ 2023-02-09 17:26 UTC (permalink / raw)
To: Yangtao Li; +Cc: linux-kernel, linux-f2fs-devel
On 02/09, Yangtao Li wrote:
> Export ipu_policy as a string in debugfs for better readability and
> it can help us better understand some strategies of the file system.
>
> Since we use ipu_policy as a bitmap, and the bitmap API parameter is
> unsigned long type data, let's change ipu_policy to unsigned long type.
>
> Signed-off-by: Yangtao Li <frank.li@vivo.com>
> ---
> fs/f2fs/debug.c | 64 ++++++++++++++++++++++++++++++++---------------
> fs/f2fs/f2fs.h | 4 +--
> fs/f2fs/segment.h | 1 +
> fs/f2fs/sysfs.c | 2 +-
> 4 files changed, 48 insertions(+), 23 deletions(-)
>
> diff --git a/fs/f2fs/debug.c b/fs/f2fs/debug.c
> index 32af4f0c5735..d50dc8e6abfd 100644
> --- a/fs/f2fs/debug.c
> +++ b/fs/f2fs/debug.c
> @@ -354,6 +354,17 @@ static char *s_flag[] = {
> [SBI_IS_FREEZING] = " freezefs",
> };
>
> +static const char *ipu_mode_names[F2FS_IPU_MAX] = {
> + [F2FS_IPU_FORCE] = "FORCE",
> + [F2FS_IPU_SSR] = "SSR",
> + [F2FS_IPU_UTIL] = "UTIL",
> + [F2FS_IPU_SSR_UTIL] = "SSR_UTIL",
> + [F2FS_IPU_FSYNC] = "FSYNC",
> + [F2FS_IPU_ASYNC] = "ASYNC",
> + [F2FS_IPU_NOCACHE] = "NOCACHE",
> + [F2FS_IPU_HONOR_OPU_WRITE] = "HONOR_OPU_WRITE",
> +};
> +
> static int stat_show(struct seq_file *s, void *v)
> {
> struct f2fs_stat_info *si;
> @@ -362,18 +373,20 @@ static int stat_show(struct seq_file *s, void *v)
>
> raw_spin_lock_irqsave(&f2fs_stat_lock, flags);
> list_for_each_entry(si, &f2fs_stat_list, stat_list) {
> - update_general_status(si->sbi);
> + struct f2fs_sb_info *sbi = si->sbi;
> +
> + update_general_status(sbi);
>
> seq_printf(s, "\n=====[ partition info(%pg). #%d, %s, CP: %s]=====\n",
> - si->sbi->sb->s_bdev, i++,
> - f2fs_readonly(si->sbi->sb) ? "RO" : "RW",
> - is_set_ckpt_flags(si->sbi, CP_DISABLED_FLAG) ?
> - "Disabled" : (f2fs_cp_error(si->sbi) ? "Error" : "Good"));
> - if (si->sbi->s_flag) {
> + sbi->sb->s_bdev, i++,
> + f2fs_readonly(sbi->sb) ? "RO" : "RW",
> + is_set_ckpt_flags(sbi, CP_DISABLED_FLAG) ?
> + "Disabled" : (f2fs_cp_error(sbi) ? "Error" : "Good"));
> + if (sbi->s_flag) {
> seq_puts(s, "[SBI:");
> - for_each_set_bit(j, &si->sbi->s_flag, 32)
> + for_each_set_bit(j, &sbi->s_flag, 32)
> seq_puts(s, s_flag[j]);
> - seq_puts(s, "]\n");
> + seq_puts(s, "]\n\n");
> }
> seq_printf(s, "[SB: 1] [CP: 2] [SIT: %d] [NAT: %d] ",
> si->sit_area_segs, si->nat_area_segs);
> @@ -383,8 +396,19 @@ static int stat_show(struct seq_file *s, void *v)
> si->overp_segs, si->rsvd_segs);
> seq_printf(s, "Current Time Sec: %llu / Mounted Time Sec: %llu\n\n",
> ktime_get_boottime_seconds(),
> - SIT_I(si->sbi)->mounted_time);
> - if (test_opt(si->sbi, DISCARD))
> + SIT_I(sbi)->mounted_time);
> +
> + seq_puts(s, "Policy:\n");
> + seq_puts(s, " - IPU: [");
> + if (IS_F2FS_IPU_DISABLE(sbi)) {
> + seq_puts(s, " DISABLE\n");
> + } else {
> + for_each_set_bit(j, &SM_I(sbi)->ipu_policy, F2FS_IPU_MAX)
> + seq_printf(s, " %s", ipu_mode_names[j]);
> + }
> + seq_puts(s, " ]\n");
> +
> + if (test_opt(sbi, DISCARD))
> seq_printf(s, "Utilization: %u%% (%u valid blocks, %u discard blocks)\n",
> si->utilization, si->valid_count, si->discard_blks);
> else
> @@ -491,15 +515,15 @@ static int stat_show(struct seq_file *s, void *v)
> seq_printf(s, " - node segments : %d (%d)\n",
> si->node_segs, si->bg_node_segs);
> seq_puts(s, " - Reclaimed segs :\n");
> - seq_printf(s, " - Normal : %d\n", si->sbi->gc_reclaimed_segs[GC_NORMAL]);
> - seq_printf(s, " - Idle CB : %d\n", si->sbi->gc_reclaimed_segs[GC_IDLE_CB]);
> + seq_printf(s, " - Normal : %d\n", sbi->gc_reclaimed_segs[GC_NORMAL]);
> + seq_printf(s, " - Idle CB : %d\n", sbi->gc_reclaimed_segs[GC_IDLE_CB]);
> seq_printf(s, " - Idle Greedy : %d\n",
> - si->sbi->gc_reclaimed_segs[GC_IDLE_GREEDY]);
> - seq_printf(s, " - Idle AT : %d\n", si->sbi->gc_reclaimed_segs[GC_IDLE_AT]);
> + sbi->gc_reclaimed_segs[GC_IDLE_GREEDY]);
> + seq_printf(s, " - Idle AT : %d\n", sbi->gc_reclaimed_segs[GC_IDLE_AT]);
> seq_printf(s, " - Urgent High : %d\n",
> - si->sbi->gc_reclaimed_segs[GC_URGENT_HIGH]);
> - seq_printf(s, " - Urgent Mid : %d\n", si->sbi->gc_reclaimed_segs[GC_URGENT_MID]);
> - seq_printf(s, " - Urgent Low : %d\n", si->sbi->gc_reclaimed_segs[GC_URGENT_LOW]);
> + sbi->gc_reclaimed_segs[GC_URGENT_HIGH]);
> + seq_printf(s, " - Urgent Mid : %d\n", sbi->gc_reclaimed_segs[GC_URGENT_MID]);
> + seq_printf(s, " - Urgent Low : %d\n", sbi->gc_reclaimed_segs[GC_URGENT_LOW]);
> seq_printf(s, "Try to move %d blocks (BG: %d)\n", si->tot_blks,
> si->bg_data_blks + si->bg_node_blks);
> seq_printf(s, " - data blocks : %d (%d)\n", si->data_blks,
> @@ -565,7 +589,7 @@ static int stat_show(struct seq_file *s, void *v)
> si->ndirty_imeta);
> seq_printf(s, " - fsync mark: %4lld\n",
> percpu_counter_sum_positive(
> - &si->sbi->rf_node_block_count));
> + &sbi->rf_node_block_count));
> seq_printf(s, " - NATs: %9d/%9d\n - SITs: %9d/%9d\n",
> si->dirty_nats, si->nats, si->dirty_sits, si->sits);
> seq_printf(s, " - free_nids: %9d/%9d\n - alloc_nids: %9d\n",
> @@ -592,12 +616,12 @@ static int stat_show(struct seq_file *s, void *v)
> si->block_count[LFS], si->segment_count[LFS]);
>
> /* segment usage info */
> - f2fs_update_sit_info(si->sbi);
> + f2fs_update_sit_info(sbi);
> seq_printf(s, "\nBDF: %u, avg. vblocks: %u\n",
> si->bimodal, si->avg_vblocks);
>
> /* memory footprint */
> - update_mem_info(si->sbi);
> + update_mem_info(sbi);
> seq_printf(s, "\nMemory: %llu KB\n",
> (si->base_mem + si->cache_mem + si->page_mem) >> 10);
> seq_printf(s, " - static: %llu KB\n",
I don't think we need the changes below.
> diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
> index 44f2d76525bf..b699ed74f438 100644
> --- a/fs/f2fs/f2fs.h
> +++ b/fs/f2fs/f2fs.h
> @@ -1070,7 +1070,7 @@ struct f2fs_sm_info {
>
> struct list_head sit_entry_set; /* sit entry set list */
>
> - unsigned int ipu_policy; /* in-place-update policy */
> + unsigned long ipu_policy; /* in-place-update policy */
> unsigned int min_ipu_util; /* in-place-update threshold */
> unsigned int min_fsync_blocks; /* threshold for fsync */
> unsigned int min_seq_blocks; /* threshold for sequential blocks */
> @@ -1322,7 +1322,7 @@ enum {
> MAX_TIME,
> };
>
> -/* Note that you need to keep synchronization with this gc_mode_names array */
> +/* Modification on enum should be synchronized with gc_mode_names array */
> enum {
> GC_NORMAL,
> GC_IDLE_CB,
> diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h
> index 8ee5e5db9287..92c8be00d396 100644
> --- a/fs/f2fs/segment.h
> +++ b/fs/f2fs/segment.h
> @@ -672,6 +672,7 @@ static inline int utilization(struct f2fs_sb_info *sbi)
>
> #define F2FS_IPU_DISABLE 0
>
> +/* Modification on enum should be synchronized with ipu_mode_names array */
> enum {
> F2FS_IPU_FORCE,
> F2FS_IPU_SSR,
> diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
> index 6082e132257a..228df201f6d4 100644
> --- a/fs/f2fs/sysfs.c
> +++ b/fs/f2fs/sysfs.c
> @@ -711,7 +711,7 @@ static ssize_t __sbi_store(struct f2fs_attr *a,
> return -EINVAL;
> if (t && f2fs_lfs_mode(sbi))
> return -EINVAL;
> - SM_I(sbi)->ipu_policy = (unsigned int)t;
> + SM_I(sbi)->ipu_policy = t;
> return count;
> }
>
> --
> 2.25.1
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [f2fs-dev] [PATCH] f2fs: export ipu policy in debugfs
2023-02-09 17:26 ` Jaegeuk Kim
@ 2023-02-13 9:29 ` Chao Yu
0 siblings, 0 replies; 3+ messages in thread
From: Chao Yu @ 2023-02-13 9:29 UTC (permalink / raw)
To: Yangtao Li; +Cc: Jaegeuk Kim, linux-kernel, linux-f2fs-devel
On 2023/2/10 1:26, Jaegeuk Kim wrote:
> On 02/09, Yangtao Li wrote:
>> Export ipu_policy as a string in debugfs for better readability and
>> it can help us better understand some strategies of the file system.
>>
>> Since we use ipu_policy as a bitmap, and the bitmap API parameter is
>> unsigned long type data, let's change ipu_policy to unsigned long type.
>>
>> Signed-off-by: Yangtao Li <frank.li@vivo.com>
>> ---
>> fs/f2fs/debug.c | 64 ++++++++++++++++++++++++++++++++---------------
>> fs/f2fs/f2fs.h | 4 +--
>> fs/f2fs/segment.h | 1 +
>> fs/f2fs/sysfs.c | 2 +-
>> 4 files changed, 48 insertions(+), 23 deletions(-)
>>
>> diff --git a/fs/f2fs/debug.c b/fs/f2fs/debug.c
>> index 32af4f0c5735..d50dc8e6abfd 100644
>> --- a/fs/f2fs/debug.c
>> +++ b/fs/f2fs/debug.c
>> @@ -354,6 +354,17 @@ static char *s_flag[] = {
>> [SBI_IS_FREEZING] = " freezefs",
>> };
>>
>> +static const char *ipu_mode_names[F2FS_IPU_MAX] = {
>> + [F2FS_IPU_FORCE] = "FORCE",
>> + [F2FS_IPU_SSR] = "SSR",
>> + [F2FS_IPU_UTIL] = "UTIL",
>> + [F2FS_IPU_SSR_UTIL] = "SSR_UTIL",
>> + [F2FS_IPU_FSYNC] = "FSYNC",
>> + [F2FS_IPU_ASYNC] = "ASYNC",
>> + [F2FS_IPU_NOCACHE] = "NOCACHE",
>> + [F2FS_IPU_HONOR_OPU_WRITE] = "HONOR_OPU_WRITE",
>> +};
>> +
>> static int stat_show(struct seq_file *s, void *v)
>> {
>> struct f2fs_stat_info *si;
>> @@ -362,18 +373,20 @@ static int stat_show(struct seq_file *s, void *v)
>>
>> raw_spin_lock_irqsave(&f2fs_stat_lock, flags);
>> list_for_each_entry(si, &f2fs_stat_list, stat_list) {
>> - update_general_status(si->sbi);
>> + struct f2fs_sb_info *sbi = si->sbi;
>> +
>> + update_general_status(sbi);
>>
>> seq_printf(s, "\n=====[ partition info(%pg). #%d, %s, CP: %s]=====\n",
>> - si->sbi->sb->s_bdev, i++,
>> - f2fs_readonly(si->sbi->sb) ? "RO" : "RW",
>> - is_set_ckpt_flags(si->sbi, CP_DISABLED_FLAG) ?
>> - "Disabled" : (f2fs_cp_error(si->sbi) ? "Error" : "Good"));
>> - if (si->sbi->s_flag) {
>> + sbi->sb->s_bdev, i++,
>> + f2fs_readonly(sbi->sb) ? "RO" : "RW",
>> + is_set_ckpt_flags(sbi, CP_DISABLED_FLAG) ?
>> + "Disabled" : (f2fs_cp_error(sbi) ? "Error" : "Good"));
How about making another patch to replace si->sbi w/ sbi?
Thanks,
>> + if (sbi->s_flag) {
>> seq_puts(s, "[SBI:");
>> - for_each_set_bit(j, &si->sbi->s_flag, 32)
>> + for_each_set_bit(j, &sbi->s_flag, 32)
>> seq_puts(s, s_flag[j]);
>> - seq_puts(s, "]\n");
>> + seq_puts(s, "]\n\n");
>> }
>> seq_printf(s, "[SB: 1] [CP: 2] [SIT: %d] [NAT: %d] ",
>> si->sit_area_segs, si->nat_area_segs);
>> @@ -383,8 +396,19 @@ static int stat_show(struct seq_file *s, void *v)
>> si->overp_segs, si->rsvd_segs);
>> seq_printf(s, "Current Time Sec: %llu / Mounted Time Sec: %llu\n\n",
>> ktime_get_boottime_seconds(),
>> - SIT_I(si->sbi)->mounted_time);
>> - if (test_opt(si->sbi, DISCARD))
>> + SIT_I(sbi)->mounted_time);
>> +
>> + seq_puts(s, "Policy:\n");
>> + seq_puts(s, " - IPU: [");
>> + if (IS_F2FS_IPU_DISABLE(sbi)) {
>> + seq_puts(s, " DISABLE\n");
>> + } else {
>> + for_each_set_bit(j, &SM_I(sbi)->ipu_policy, F2FS_IPU_MAX)
>> + seq_printf(s, " %s", ipu_mode_names[j]);
>> + }
>> + seq_puts(s, " ]\n");
>> +
>> + if (test_opt(sbi, DISCARD))
>> seq_printf(s, "Utilization: %u%% (%u valid blocks, %u discard blocks)\n",
>> si->utilization, si->valid_count, si->discard_blks);
>> else
>> @@ -491,15 +515,15 @@ static int stat_show(struct seq_file *s, void *v)
>> seq_printf(s, " - node segments : %d (%d)\n",
>> si->node_segs, si->bg_node_segs);
>> seq_puts(s, " - Reclaimed segs :\n");
>> - seq_printf(s, " - Normal : %d\n", si->sbi->gc_reclaimed_segs[GC_NORMAL]);
>> - seq_printf(s, " - Idle CB : %d\n", si->sbi->gc_reclaimed_segs[GC_IDLE_CB]);
>> + seq_printf(s, " - Normal : %d\n", sbi->gc_reclaimed_segs[GC_NORMAL]);
>> + seq_printf(s, " - Idle CB : %d\n", sbi->gc_reclaimed_segs[GC_IDLE_CB]);
>> seq_printf(s, " - Idle Greedy : %d\n",
>> - si->sbi->gc_reclaimed_segs[GC_IDLE_GREEDY]);
>> - seq_printf(s, " - Idle AT : %d\n", si->sbi->gc_reclaimed_segs[GC_IDLE_AT]);
>> + sbi->gc_reclaimed_segs[GC_IDLE_GREEDY]);
>> + seq_printf(s, " - Idle AT : %d\n", sbi->gc_reclaimed_segs[GC_IDLE_AT]);
>> seq_printf(s, " - Urgent High : %d\n",
>> - si->sbi->gc_reclaimed_segs[GC_URGENT_HIGH]);
>> - seq_printf(s, " - Urgent Mid : %d\n", si->sbi->gc_reclaimed_segs[GC_URGENT_MID]);
>> - seq_printf(s, " - Urgent Low : %d\n", si->sbi->gc_reclaimed_segs[GC_URGENT_LOW]);
>> + sbi->gc_reclaimed_segs[GC_URGENT_HIGH]);
>> + seq_printf(s, " - Urgent Mid : %d\n", sbi->gc_reclaimed_segs[GC_URGENT_MID]);
>> + seq_printf(s, " - Urgent Low : %d\n", sbi->gc_reclaimed_segs[GC_URGENT_LOW]);
>> seq_printf(s, "Try to move %d blocks (BG: %d)\n", si->tot_blks,
>> si->bg_data_blks + si->bg_node_blks);
>> seq_printf(s, " - data blocks : %d (%d)\n", si->data_blks,
>> @@ -565,7 +589,7 @@ static int stat_show(struct seq_file *s, void *v)
>> si->ndirty_imeta);
>> seq_printf(s, " - fsync mark: %4lld\n",
>> percpu_counter_sum_positive(
>> - &si->sbi->rf_node_block_count));
>> + &sbi->rf_node_block_count));
>> seq_printf(s, " - NATs: %9d/%9d\n - SITs: %9d/%9d\n",
>> si->dirty_nats, si->nats, si->dirty_sits, si->sits);
>> seq_printf(s, " - free_nids: %9d/%9d\n - alloc_nids: %9d\n",
>> @@ -592,12 +616,12 @@ static int stat_show(struct seq_file *s, void *v)
>> si->block_count[LFS], si->segment_count[LFS]);
>>
>> /* segment usage info */
>> - f2fs_update_sit_info(si->sbi);
>> + f2fs_update_sit_info(sbi);
>> seq_printf(s, "\nBDF: %u, avg. vblocks: %u\n",
>> si->bimodal, si->avg_vblocks);
>>
>> /* memory footprint */
>> - update_mem_info(si->sbi);
>> + update_mem_info(sbi);
>> seq_printf(s, "\nMemory: %llu KB\n",
>> (si->base_mem + si->cache_mem + si->page_mem) >> 10);
>> seq_printf(s, " - static: %llu KB\n",
>
> I don't think we need the changes below.
>
>> diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
>> index 44f2d76525bf..b699ed74f438 100644
>> --- a/fs/f2fs/f2fs.h
>> +++ b/fs/f2fs/f2fs.h
>> @@ -1070,7 +1070,7 @@ struct f2fs_sm_info {
>>
>> struct list_head sit_entry_set; /* sit entry set list */
>>
>> - unsigned int ipu_policy; /* in-place-update policy */
>> + unsigned long ipu_policy; /* in-place-update policy */
>> unsigned int min_ipu_util; /* in-place-update threshold */
>> unsigned int min_fsync_blocks; /* threshold for fsync */
>> unsigned int min_seq_blocks; /* threshold for sequential blocks */
>> @@ -1322,7 +1322,7 @@ enum {
>> MAX_TIME,
>> };
>>
>> -/* Note that you need to keep synchronization with this gc_mode_names array */
>> +/* Modification on enum should be synchronized with gc_mode_names array */
>> enum {
>> GC_NORMAL,
>> GC_IDLE_CB,
>> diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h
>> index 8ee5e5db9287..92c8be00d396 100644
>> --- a/fs/f2fs/segment.h
>> +++ b/fs/f2fs/segment.h
>> @@ -672,6 +672,7 @@ static inline int utilization(struct f2fs_sb_info *sbi)
>>
>> #define F2FS_IPU_DISABLE 0
>>
>> +/* Modification on enum should be synchronized with ipu_mode_names array */
>> enum {
>> F2FS_IPU_FORCE,
>> F2FS_IPU_SSR,
>> diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
>> index 6082e132257a..228df201f6d4 100644
>> --- a/fs/f2fs/sysfs.c
>> +++ b/fs/f2fs/sysfs.c
>> @@ -711,7 +711,7 @@ static ssize_t __sbi_store(struct f2fs_attr *a,
>> return -EINVAL;
>> if (t && f2fs_lfs_mode(sbi))
>> return -EINVAL;
>> - SM_I(sbi)->ipu_policy = (unsigned int)t;
>> + SM_I(sbi)->ipu_policy = t;
>> return count;
>> }
>>
>> --
>> 2.25.1
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-02-13 9:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-09 3:40 [f2fs-dev] [PATCH] f2fs: export ipu policy in debugfs Yangtao Li via Linux-f2fs-devel
2023-02-09 17:26 ` Jaegeuk Kim
2023-02-13 9:29 ` Chao Yu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).