* [PATCH] f2fs: show the list of donation files
@ 2025-08-12 23:58 Jaegeuk Kim
2025-08-13 2:20 ` [f2fs-dev] " Chao Yu
2025-08-13 16:24 ` [PATCH v2] " Jaegeuk Kim
0 siblings, 2 replies; 7+ messages in thread
From: Jaegeuk Kim @ 2025-08-12 23:58 UTC (permalink / raw)
To: linux-kernel, linux-f2fs-devel; +Cc: Jaegeuk Kim
This patch introduces a proc entry to show the currently enrolled donation
files.
- "File path" indicates a file.
- "Status"
a. "Donated" means the file is registed in the donation list by
fadvise(offset, length, POSIX_FADV_NOREUSE)
b. "Evicted" means the donated pages were reclaimed.
- "Offset (kb)" and "Length (kb) show the registered donation range.
- "Cached pages (kb)" shows the amount of cached pages in the inode page cache.
For example,
Donation List
# of files : 2
File path Status Offset (kb) Length (kb) Cached pages (kb)
---
/local/tmp/test2 Donated 0 1048576 2097152
/local/tmp/test Evicted 0 1048576 1048576
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
---
fs/f2fs/sysfs.c | 64 +++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 64 insertions(+)
diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
index 1ffaf9e74ce9..2de7557bfebf 100644
--- a/fs/f2fs/sysfs.c
+++ b/fs/f2fs/sysfs.c
@@ -1769,6 +1769,68 @@ static int __maybe_unused disk_map_seq_show(struct seq_file *seq,
return 0;
}
+static int __maybe_unused donation_list_seq_show(struct seq_file *seq,
+ void *offset)
+{
+ struct super_block *sb = seq->private;
+ struct f2fs_sb_info *sbi = F2FS_SB(sb);
+ struct inode *inode;
+ struct f2fs_inode_info *fi;
+ struct dentry *dentry;
+ char *buf, *path;
+ int i;
+
+ buf = f2fs_getname(sbi);
+ if (!buf)
+ return 0;
+
+ seq_printf(seq, "Donation List\n");
+ seq_printf(seq, " # of files : %u\n", sbi->donate_files);
+ seq_printf(seq, " %-50s %10s %15s %15s %20s\n",
+ "File path", "Status", "Offset (kb)",
+ "Length (kb)", "Cached pages (kb)");
+ seq_printf(seq, "---\n");
+
+ for (i = 0; i < sbi->donate_files; i++) {
+ spin_lock(&sbi->inode_lock[DONATE_INODE]);
+ if (list_empty(&sbi->inode_list[DONATE_INODE])) {
+ spin_unlock(&sbi->inode_lock[DONATE_INODE]);
+ break;
+ }
+ fi = list_first_entry(&sbi->inode_list[DONATE_INODE],
+ struct f2fs_inode_info, gdonate_list);
+ list_move_tail(&fi->gdonate_list, &sbi->inode_list[DONATE_INODE]);
+ inode = igrab(&fi->vfs_inode);
+ spin_unlock(&sbi->inode_lock[DONATE_INODE]);
+
+ if (!inode)
+ continue;
+
+ inode_lock(inode);
+
+ dentry = d_find_alias(inode);
+ if (!dentry) {
+ path = NULL;
+ } else {
+ path = dentry_path_raw(dentry, buf, PATH_MAX);
+ if (IS_ERR(path))
+ goto next;
+ }
+ seq_printf(seq, " %-50s %10s %15lu %15lu %20lu\n",
+ path ? path : "<unlinked>",
+ is_inode_flag_set(inode, FI_DONATE_FINISHED) ?
+ "Evicted" : "Donated",
+ fi->donate_start << (PAGE_SHIFT - 10),
+ (fi->donate_end + 1) << (PAGE_SHIFT - 10),
+ inode->i_mapping->nrpages << (PAGE_SHIFT - 10));
+next:
+ inode_unlock(inode);
+ iput(inode);
+ }
+ f2fs_putname(buf);
+ return 0;
+}
+
#ifdef CONFIG_F2FS_FAULT_INJECTION
static int __maybe_unused inject_stats_seq_show(struct seq_file *seq,
void *offset)
@@ -1878,6 +1940,8 @@ int f2fs_register_sysfs(struct f2fs_sb_info *sbi)
discard_plist_seq_show, sb);
proc_create_single_data("disk_map", 0444, sbi->s_proc,
disk_map_seq_show, sb);
+ proc_create_single_data("donation_list", 0444, sbi->s_proc,
+ donation_list_seq_show, sb);
#ifdef CONFIG_F2FS_FAULT_INJECTION
proc_create_single_data("inject_stats", 0444, sbi->s_proc,
inject_stats_seq_show, sb);
--
2.51.0.rc1.163.g2494970778-goog
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [f2fs-dev] [PATCH] f2fs: show the list of donation files
2025-08-12 23:58 [PATCH] f2fs: show the list of donation files Jaegeuk Kim
@ 2025-08-13 2:20 ` Chao Yu
2025-08-13 16:24 ` [PATCH v2] " Jaegeuk Kim
1 sibling, 0 replies; 7+ messages in thread
From: Chao Yu @ 2025-08-13 2:20 UTC (permalink / raw)
To: Jaegeuk Kim, linux-kernel, linux-f2fs-devel; +Cc: chao
On 8/13/25 07:58, Jaegeuk Kim via Linux-f2fs-devel wrote:
> This patch introduces a proc entry to show the currently enrolled donation
> files.
>
> - "File path" indicates a file.
> - "Status"
> a. "Donated" means the file is registed in the donation list by
> fadvise(offset, length, POSIX_FADV_NOREUSE)
> b. "Evicted" means the donated pages were reclaimed.
> - "Offset (kb)" and "Length (kb) show the registered donation range.
> - "Cached pages (kb)" shows the amount of cached pages in the inode page cache.
>
> For example,
>
> Donation List
> # of files : 2
> File path Status Offset (kb) Length (kb) Cached pages (kb)
> ---
> /local/tmp/test2 Donated 0 1048576 2097152
> /local/tmp/test Evicted 0 1048576 1048576
>
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
> ---
> fs/f2fs/sysfs.c | 64 +++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 64 insertions(+)
>
> diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
> index 1ffaf9e74ce9..2de7557bfebf 100644
> --- a/fs/f2fs/sysfs.c
> +++ b/fs/f2fs/sysfs.c
> @@ -1769,6 +1769,68 @@ static int __maybe_unused disk_map_seq_show(struct seq_file *seq,
> return 0;
> }
>
> +static int __maybe_unused donation_list_seq_show(struct seq_file *seq,
> + void *offset)
> +{
> + struct super_block *sb = seq->private;
> + struct f2fs_sb_info *sbi = F2FS_SB(sb);
> + struct inode *inode;
> + struct f2fs_inode_info *fi;
> + struct dentry *dentry;
> + char *buf, *path;
> + int i;
> +
> + buf = f2fs_getname(sbi);
> + if (!buf)
> + return 0;
> +
> + seq_printf(seq, "Donation List\n");
> + seq_printf(seq, " # of files : %u\n", sbi->donate_files);
> + seq_printf(seq, " %-50s %10s %15s %15s %20s\n",
> + "File path", "Status", "Offset (kb)",
> + "Length (kb)", "Cached pages (kb)");
> + seq_printf(seq, "---\n");
> +
> + for (i = 0; i < sbi->donate_files; i++) {
> + spin_lock(&sbi->inode_lock[DONATE_INODE]);
> + if (list_empty(&sbi->inode_list[DONATE_INODE])) {
> + spin_unlock(&sbi->inode_lock[DONATE_INODE]);
> + break;
> + }
> + fi = list_first_entry(&sbi->inode_list[DONATE_INODE],
> + struct f2fs_inode_info, gdonate_list);
> + list_move_tail(&fi->gdonate_list, &sbi->inode_list[DONATE_INODE]);
> + inode = igrab(&fi->vfs_inode);
> + spin_unlock(&sbi->inode_lock[DONATE_INODE]);
> +
> + if (!inode)
> + continue;
> +
> + inode_lock(inode);
The following flow doesn't update any fields in inode structure, so what about
using inode_lock_shared() instead?
Thanks,
> +
> + dentry = d_find_alias(inode);
> + if (!dentry) {
> + path = NULL;
> + } else {
> + path = dentry_path_raw(dentry, buf, PATH_MAX);
> + if (IS_ERR(path))
> + goto next;
> + }
> + seq_printf(seq, " %-50s %10s %15lu %15lu %20lu\n",
> + path ? path : "<unlinked>",
> + is_inode_flag_set(inode, FI_DONATE_FINISHED) ?
> + "Evicted" : "Donated",
> + fi->donate_start << (PAGE_SHIFT - 10),
> + (fi->donate_end + 1) << (PAGE_SHIFT - 10),
> + inode->i_mapping->nrpages << (PAGE_SHIFT - 10));
> +next:
> + inode_unlock(inode);
> + iput(inode);
> + }
> + f2fs_putname(buf);
> + return 0;
> +}
> +
> #ifdef CONFIG_F2FS_FAULT_INJECTION
> static int __maybe_unused inject_stats_seq_show(struct seq_file *seq,
> void *offset)
> @@ -1878,6 +1940,8 @@ int f2fs_register_sysfs(struct f2fs_sb_info *sbi)
> discard_plist_seq_show, sb);
> proc_create_single_data("disk_map", 0444, sbi->s_proc,
> disk_map_seq_show, sb);
> + proc_create_single_data("donation_list", 0444, sbi->s_proc,
> + donation_list_seq_show, sb);
> #ifdef CONFIG_F2FS_FAULT_INJECTION
> proc_create_single_data("inject_stats", 0444, sbi->s_proc,
> inject_stats_seq_show, sb);
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] f2fs: show the list of donation files
2025-08-12 23:58 [PATCH] f2fs: show the list of donation files Jaegeuk Kim
2025-08-13 2:20 ` [f2fs-dev] " Chao Yu
@ 2025-08-13 16:24 ` Jaegeuk Kim
2025-08-14 2:22 ` [f2fs-dev] " Chao Yu
2025-08-15 16:20 ` [f2fs-dev] [PATCH v3] " Jaegeuk Kim
1 sibling, 2 replies; 7+ messages in thread
From: Jaegeuk Kim @ 2025-08-13 16:24 UTC (permalink / raw)
To: linux-kernel, linux-f2fs-devel
This patch introduces a proc entry to show the currently enrolled donation
files.
- "File path" indicates a file.
- "Status"
a. "Donated" means the file is registed in the donation list by
fadvise(offset, length, POSIX_FADV_NOREUSE)
b. "Evicted" means the donated pages were reclaimed.
- "Offset (kb)" and "Length (kb) show the registered donation range.
- "Cached pages (kb)" shows the amount of cached pages in the inode page cache.
For example,
Donation List
# of files : 2
File path Status Offset (kb) Length (kb) Cached pages (kb)
---
/local/tmp/test2 Donated 0 1048576 2097152
/local/tmp/test Evicted 0 1048576 1048576
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
---
v2: use inode_lock|unlock_shared
fs/f2fs/sysfs.c | 64 +++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 64 insertions(+)
diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
index 1ffaf9e74ce9..9f90d1878ec6 100644
--- a/fs/f2fs/sysfs.c
+++ b/fs/f2fs/sysfs.c
@@ -1769,6 +1769,68 @@ static int __maybe_unused disk_map_seq_show(struct seq_file *seq,
return 0;
}
+static int __maybe_unused donation_list_seq_show(struct seq_file *seq,
+ void *offset)
+{
+ struct super_block *sb = seq->private;
+ struct f2fs_sb_info *sbi = F2FS_SB(sb);
+ struct inode *inode;
+ struct f2fs_inode_info *fi;
+ struct dentry *dentry;
+ char *buf, *path;
+ int i;
+
+ buf = f2fs_getname(sbi);
+ if (!buf)
+ return 0;
+
+ seq_printf(seq, "Donation List\n");
+ seq_printf(seq, " # of files : %u\n", sbi->donate_files);
+ seq_printf(seq, " %-50s %10s %15s %15s %20s\n",
+ "File path", "Status", "Offset (kb)",
+ "Length (kb)", "Cached pages (kb)");
+ seq_printf(seq, "---\n");
+
+ for (i = 0; i < sbi->donate_files; i++) {
+ spin_lock(&sbi->inode_lock[DONATE_INODE]);
+ if (list_empty(&sbi->inode_list[DONATE_INODE])) {
+ spin_unlock(&sbi->inode_lock[DONATE_INODE]);
+ break;
+ }
+ fi = list_first_entry(&sbi->inode_list[DONATE_INODE],
+ struct f2fs_inode_info, gdonate_list);
+ list_move_tail(&fi->gdonate_list, &sbi->inode_list[DONATE_INODE]);
+ inode = igrab(&fi->vfs_inode);
+ spin_unlock(&sbi->inode_lock[DONATE_INODE]);
+
+ if (!inode)
+ continue;
+
+ inode_lock_shared(inode);
+
+ dentry = d_find_alias(inode);
+ if (!dentry) {
+ path = NULL;
+ } else {
+ path = dentry_path_raw(dentry, buf, PATH_MAX);
+ if (IS_ERR(path))
+ goto next;
+ }
+ seq_printf(seq, " %-50s %10s %15lu %15lu %20lu\n",
+ path ? path : "<unlinked>",
+ is_inode_flag_set(inode, FI_DONATE_FINISHED) ?
+ "Evicted" : "Donated",
+ fi->donate_start << (PAGE_SHIFT - 10),
+ (fi->donate_end + 1) << (PAGE_SHIFT - 10),
+ inode->i_mapping->nrpages << (PAGE_SHIFT - 10));
+next:
+ inode_unlock_shared(inode);
+ iput(inode);
+ }
+ f2fs_putname(buf);
+ return 0;
+}
+
#ifdef CONFIG_F2FS_FAULT_INJECTION
static int __maybe_unused inject_stats_seq_show(struct seq_file *seq,
void *offset)
@@ -1878,6 +1940,8 @@ int f2fs_register_sysfs(struct f2fs_sb_info *sbi)
discard_plist_seq_show, sb);
proc_create_single_data("disk_map", 0444, sbi->s_proc,
disk_map_seq_show, sb);
+ proc_create_single_data("donation_list", 0444, sbi->s_proc,
+ donation_list_seq_show, sb);
#ifdef CONFIG_F2FS_FAULT_INJECTION
proc_create_single_data("inject_stats", 0444, sbi->s_proc,
inject_stats_seq_show, sb);
--
2.51.0.rc1.163.g2494970778-goog
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [f2fs-dev] [PATCH v2] f2fs: show the list of donation files
2025-08-13 16:24 ` [PATCH v2] " Jaegeuk Kim
@ 2025-08-14 2:22 ` Chao Yu
2025-08-15 16:20 ` [f2fs-dev] [PATCH v3] " Jaegeuk Kim
1 sibling, 0 replies; 7+ messages in thread
From: Chao Yu @ 2025-08-14 2:22 UTC (permalink / raw)
To: Jaegeuk Kim, linux-kernel, linux-f2fs-devel; +Cc: chao
On 8/14/25 00:24, Jaegeuk Kim via Linux-f2fs-devel wrote:
> This patch introduces a proc entry to show the currently enrolled donation
> files.
>
> - "File path" indicates a file.
> - "Status"
> a. "Donated" means the file is registed in the donation list by
> fadvise(offset, length, POSIX_FADV_NOREUSE)
> b. "Evicted" means the donated pages were reclaimed.
> - "Offset (kb)" and "Length (kb) show the registered donation range.
> - "Cached pages (kb)" shows the amount of cached pages in the inode page cache.
>
> For example,
>
> Donation List
> # of files : 2
> File path Status Offset (kb) Length (kb) Cached pages (kb)
> ---
> /local/tmp/test2 Donated 0 1048576 2097152
> /local/tmp/test Evicted 0 1048576 1048576
>
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
> ---
> v2: use inode_lock|unlock_shared
>
> fs/f2fs/sysfs.c | 64 +++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 64 insertions(+)
>
> diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
> index 1ffaf9e74ce9..9f90d1878ec6 100644
> --- a/fs/f2fs/sysfs.c
> +++ b/fs/f2fs/sysfs.c
> @@ -1769,6 +1769,68 @@ static int __maybe_unused disk_map_seq_show(struct seq_file *seq,
> return 0;
> }
>
> +static int __maybe_unused donation_list_seq_show(struct seq_file *seq,
> + void *offset)
> +{
> + struct super_block *sb = seq->private;
> + struct f2fs_sb_info *sbi = F2FS_SB(sb);
> + struct inode *inode;
> + struct f2fs_inode_info *fi;
> + struct dentry *dentry;
> + char *buf, *path;
> + int i;
> +
> + buf = f2fs_getname(sbi);
> + if (!buf)
> + return 0;
> +
> + seq_printf(seq, "Donation List\n");
> + seq_printf(seq, " # of files : %u\n", sbi->donate_files);
> + seq_printf(seq, " %-50s %10s %15s %15s %20s\n",
> + "File path", "Status", "Offset (kb)",
> + "Length (kb)", "Cached pages (kb)");
> + seq_printf(seq, "---\n");
> +
> + for (i = 0; i < sbi->donate_files; i++) {
> + spin_lock(&sbi->inode_lock[DONATE_INODE]);
> + if (list_empty(&sbi->inode_list[DONATE_INODE])) {
> + spin_unlock(&sbi->inode_lock[DONATE_INODE]);
> + break;
> + }
> + fi = list_first_entry(&sbi->inode_list[DONATE_INODE],
> + struct f2fs_inode_info, gdonate_list);
> + list_move_tail(&fi->gdonate_list, &sbi->inode_list[DONATE_INODE]);
> + inode = igrab(&fi->vfs_inode);
> + spin_unlock(&sbi->inode_lock[DONATE_INODE]);
> +
> + if (!inode)
> + continue;
> +
> + inode_lock_shared(inode);
> +
> + dentry = d_find_alias(inode);
> + if (!dentry) {
> + path = NULL;
> + } else {
> + path = dentry_path_raw(dentry, buf, PATH_MAX);
> + if (IS_ERR(path))
> + goto next;
> + }
> + seq_printf(seq, " %-50s %10s %15lu %15lu %20lu\n",
> + path ? path : "<unlinked>",
> + is_inode_flag_set(inode, FI_DONATE_FINISHED) ?
> + "Evicted" : "Donated",
> + fi->donate_start << (PAGE_SHIFT - 10),
It needs to avoid overflow while left shift?
(loff_t)fi->donate_start << (PAGE_SHIFT - 10)
(loff_t)(fi->donate_end + 1) << (PAGE_SHIFT - 10)
(loff_t)inode->i_mapping->nrpages << (PAGE_SHIFT - 10))
Thanks,
> + (fi->donate_end + 1) << (PAGE_SHIFT - 10),
> + inode->i_mapping->nrpages << (PAGE_SHIFT - 10));
> +next:
> + inode_unlock_shared(inode);
> + iput(inode);
> + }
> + f2fs_putname(buf);
> + return 0;
> +}
> +
> #ifdef CONFIG_F2FS_FAULT_INJECTION
> static int __maybe_unused inject_stats_seq_show(struct seq_file *seq,
> void *offset)
> @@ -1878,6 +1940,8 @@ int f2fs_register_sysfs(struct f2fs_sb_info *sbi)
> discard_plist_seq_show, sb);
> proc_create_single_data("disk_map", 0444, sbi->s_proc,
> disk_map_seq_show, sb);
> + proc_create_single_data("donation_list", 0444, sbi->s_proc,
> + donation_list_seq_show, sb);
> #ifdef CONFIG_F2FS_FAULT_INJECTION
> proc_create_single_data("inject_stats", 0444, sbi->s_proc,
> inject_stats_seq_show, sb);
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [f2fs-dev] [PATCH v3] f2fs: show the list of donation files
2025-08-13 16:24 ` [PATCH v2] " Jaegeuk Kim
2025-08-14 2:22 ` [f2fs-dev] " Chao Yu
@ 2025-08-15 16:20 ` Jaegeuk Kim
2025-08-16 7:05 ` Chao Yu
2025-08-19 20:30 ` [f2fs-dev] [PATCH v4] " Jaegeuk Kim
1 sibling, 2 replies; 7+ messages in thread
From: Jaegeuk Kim @ 2025-08-15 16:20 UTC (permalink / raw)
To: linux-kernel, linux-f2fs-devel
This patch introduces a proc entry to show the currently enrolled donation
files.
- "File path" indicates a file.
- "Status"
a. "Donated" means the file is registed in the donation list by
fadvise(offset, length, POSIX_FADV_NOREUSE)
b. "Evicted" means the donated pages were reclaimed.
- "Offset (kb)" and "Length (kb) show the registered donation range.
- "Cached pages (kb)" shows the amount of cached pages in the inode page cache.
For example,
Donation List
# of files : 2
File path Status Offset (kb) Length (kb) Cached pages (kb)
---
/local/tmp/test2 Donated 0 1048576 2097152
/local/tmp/test Evicted 0 1048576 1048576
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
---
v3 - use loff_t
fs/f2fs/sysfs.c | 64 +++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 64 insertions(+)
diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
index 1ffaf9e74ce9..5d0f7364dbb8 100644
--- a/fs/f2fs/sysfs.c
+++ b/fs/f2fs/sysfs.c
@@ -1769,6 +1769,68 @@ static int __maybe_unused disk_map_seq_show(struct seq_file *seq,
return 0;
}
+static int __maybe_unused donation_list_seq_show(struct seq_file *seq,
+ void *offset)
+{
+ struct super_block *sb = seq->private;
+ struct f2fs_sb_info *sbi = F2FS_SB(sb);
+ struct inode *inode;
+ struct f2fs_inode_info *fi;
+ struct dentry *dentry;
+ char *buf, *path;
+ int i;
+
+ buf = f2fs_getname(sbi);
+ if (!buf)
+ return 0;
+
+ seq_printf(seq, "Donation List\n");
+ seq_printf(seq, " # of files : %u\n", sbi->donate_files);
+ seq_printf(seq, " %-50s %10s %15s %15s %20s\n",
+ "File path", "Status", "Offset (kb)",
+ "Length (kb)", "Cached pages (kb)");
+ seq_printf(seq, "---\n");
+
+ for (i = 0; i < sbi->donate_files; i++) {
+ spin_lock(&sbi->inode_lock[DONATE_INODE]);
+ if (list_empty(&sbi->inode_list[DONATE_INODE])) {
+ spin_unlock(&sbi->inode_lock[DONATE_INODE]);
+ break;
+ }
+ fi = list_first_entry(&sbi->inode_list[DONATE_INODE],
+ struct f2fs_inode_info, gdonate_list);
+ list_move_tail(&fi->gdonate_list, &sbi->inode_list[DONATE_INODE]);
+ inode = igrab(&fi->vfs_inode);
+ spin_unlock(&sbi->inode_lock[DONATE_INODE]);
+
+ if (!inode)
+ continue;
+
+ inode_lock_shared(inode);
+
+ dentry = d_find_alias(inode);
+ if (!dentry) {
+ path = NULL;
+ } else {
+ path = dentry_path_raw(dentry, buf, PATH_MAX);
+ if (IS_ERR(path))
+ goto next;
+ }
+ seq_printf(seq, " %-50s %10s %15llu %15llu %20llu\n",
+ path ? path : "<unlinked>",
+ is_inode_flag_set(inode, FI_DONATE_FINISHED) ?
+ "Evicted" : "Donated",
+ (loff_t)fi->donate_start << (PAGE_SHIFT - 10),
+ (loff_t)(fi->donate_end + 1) << (PAGE_SHIFT - 10),
+ (loff_t)inode->i_mapping->nrpages << (PAGE_SHIFT - 10));
+next:
+ inode_unlock_shared(inode);
+ iput(inode);
+ }
+ f2fs_putname(buf);
+ return 0;
+}
+
#ifdef CONFIG_F2FS_FAULT_INJECTION
static int __maybe_unused inject_stats_seq_show(struct seq_file *seq,
void *offset)
@@ -1878,6 +1940,8 @@ int f2fs_register_sysfs(struct f2fs_sb_info *sbi)
discard_plist_seq_show, sb);
proc_create_single_data("disk_map", 0444, sbi->s_proc,
disk_map_seq_show, sb);
+ proc_create_single_data("donation_list", 0444, sbi->s_proc,
+ donation_list_seq_show, sb);
#ifdef CONFIG_F2FS_FAULT_INJECTION
proc_create_single_data("inject_stats", 0444, sbi->s_proc,
inject_stats_seq_show, sb);
--
2.51.0.rc1.163.g2494970778-goog
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [f2fs-dev] [PATCH v3] f2fs: show the list of donation files
2025-08-15 16:20 ` [f2fs-dev] [PATCH v3] " Jaegeuk Kim
@ 2025-08-16 7:05 ` Chao Yu
2025-08-19 20:30 ` [f2fs-dev] [PATCH v4] " Jaegeuk Kim
1 sibling, 0 replies; 7+ messages in thread
From: Chao Yu @ 2025-08-16 7:05 UTC (permalink / raw)
To: Jaegeuk Kim, linux-kernel, linux-f2fs-devel; +Cc: chao
On 2025/8/16 00:20, Jaegeuk Kim via Linux-f2fs-devel wrote:
> This patch introduces a proc entry to show the currently enrolled donation
> files.
>
> - "File path" indicates a file.
> - "Status"
> a. "Donated" means the file is registed in the donation list by
> fadvise(offset, length, POSIX_FADV_NOREUSE)
> b. "Evicted" means the donated pages were reclaimed.
> - "Offset (kb)" and "Length (kb) show the registered donation range.
> - "Cached pages (kb)" shows the amount of cached pages in the inode page cache.
>
> For example,
>
> Donation List
> # of files : 2
> File path Status Offset (kb) Length (kb) Cached pages (kb)
> ---
> /local/tmp/test2 Donated 0 1048576 2097152
> /local/tmp/test Evicted 0 1048576 1048576
>
> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Reviewed-by: Chao Yu <chao@kernel.org>
Thanks,
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [f2fs-dev] [PATCH v4] f2fs: show the list of donation files
2025-08-15 16:20 ` [f2fs-dev] [PATCH v3] " Jaegeuk Kim
2025-08-16 7:05 ` Chao Yu
@ 2025-08-19 20:30 ` Jaegeuk Kim
1 sibling, 0 replies; 7+ messages in thread
From: Jaegeuk Kim @ 2025-08-19 20:30 UTC (permalink / raw)
To: linux-kernel, linux-f2fs-devel
This patch introduces a proc entry to show the currently enrolled donation
files.
- "File path" indicates a file.
- "Status"
a. "Donated" means the file is registed in the donation list by
fadvise(offset, length, POSIX_FADV_NOREUSE)
b. "Evicted" means the donated pages were reclaimed.
- "Offset (kb)" and "Length (kb) show the registered donation range.
- "Cached pages (kb)" shows the amount of cached pages in the inode page cache.
For example,
# of files : 2
File path Status Donation offset (kb) Donation size (kb) File cached size (kb)
---
/local/test2 Donated 0 1048576 2097152
/local/test Evicted 0 1048576 1048576
Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
---
v4:
- change the entry names clearly
fs/f2fs/sysfs.c | 64 +++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 64 insertions(+)
diff --git a/fs/f2fs/sysfs.c b/fs/f2fs/sysfs.c
index 1ffaf9e74ce9..86ebe97c256d 100644
--- a/fs/f2fs/sysfs.c
+++ b/fs/f2fs/sysfs.c
@@ -1769,6 +1769,68 @@ static int __maybe_unused disk_map_seq_show(struct seq_file *seq,
return 0;
}
+static int __maybe_unused donation_list_seq_show(struct seq_file *seq,
+ void *offset)
+{
+ struct super_block *sb = seq->private;
+ struct f2fs_sb_info *sbi = F2FS_SB(sb);
+ struct inode *inode;
+ struct f2fs_inode_info *fi;
+ struct dentry *dentry;
+ char *buf, *path;
+ int i;
+
+ buf = f2fs_getname(sbi);
+ if (!buf)
+ return 0;
+
+ seq_printf(seq, "Donation List\n");
+ seq_printf(seq, " # of files : %u\n", sbi->donate_files);
+ seq_printf(seq, " %-50s %10s %20s %20s %22s\n",
+ "File path", "Status", "Donation offset (kb)",
+ "Donation size (kb)", "File cached size (kb)");
+ seq_printf(seq, "---\n");
+
+ for (i = 0; i < sbi->donate_files; i++) {
+ spin_lock(&sbi->inode_lock[DONATE_INODE]);
+ if (list_empty(&sbi->inode_list[DONATE_INODE])) {
+ spin_unlock(&sbi->inode_lock[DONATE_INODE]);
+ break;
+ }
+ fi = list_first_entry(&sbi->inode_list[DONATE_INODE],
+ struct f2fs_inode_info, gdonate_list);
+ list_move_tail(&fi->gdonate_list, &sbi->inode_list[DONATE_INODE]);
+ inode = igrab(&fi->vfs_inode);
+ spin_unlock(&sbi->inode_lock[DONATE_INODE]);
+
+ if (!inode)
+ continue;
+
+ inode_lock_shared(inode);
+
+ dentry = d_find_alias(inode);
+ if (!dentry) {
+ path = NULL;
+ } else {
+ path = dentry_path_raw(dentry, buf, PATH_MAX);
+ if (IS_ERR(path))
+ goto next;
+ }
+ seq_printf(seq, " %-50s %10s %20llu %20llu %22llu\n",
+ path ? path : "<unlinked>",
+ is_inode_flag_set(inode, FI_DONATE_FINISHED) ?
+ "Evicted" : "Donated",
+ (loff_t)fi->donate_start << (PAGE_SHIFT - 10),
+ (loff_t)(fi->donate_end + 1) << (PAGE_SHIFT - 10),
+ (loff_t)inode->i_mapping->nrpages << (PAGE_SHIFT - 10));
+next:
+ inode_unlock_shared(inode);
+ iput(inode);
+ }
+ f2fs_putname(buf);
+ return 0;
+}
+
#ifdef CONFIG_F2FS_FAULT_INJECTION
static int __maybe_unused inject_stats_seq_show(struct seq_file *seq,
void *offset)
@@ -1878,6 +1940,8 @@ int f2fs_register_sysfs(struct f2fs_sb_info *sbi)
discard_plist_seq_show, sb);
proc_create_single_data("disk_map", 0444, sbi->s_proc,
disk_map_seq_show, sb);
+ proc_create_single_data("donation_list", 0444, sbi->s_proc,
+ donation_list_seq_show, sb);
#ifdef CONFIG_F2FS_FAULT_INJECTION
proc_create_single_data("inject_stats", 0444, sbi->s_proc,
inject_stats_seq_show, sb);
--
2.51.0.rc2.233.g662b1ed5c5-goog
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2025-08-19 20:30 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-12 23:58 [PATCH] f2fs: show the list of donation files Jaegeuk Kim
2025-08-13 2:20 ` [f2fs-dev] " Chao Yu
2025-08-13 16:24 ` [PATCH v2] " Jaegeuk Kim
2025-08-14 2:22 ` [f2fs-dev] " Chao Yu
2025-08-15 16:20 ` [f2fs-dev] [PATCH v3] " Jaegeuk Kim
2025-08-16 7:05 ` Chao Yu
2025-08-19 20:30 ` [f2fs-dev] [PATCH v4] " Jaegeuk Kim
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).