From: Chao Yu <chao@kernel.org>
To: jaegeuk@kernel.org
Cc: linux-f2fs-devel@lists.sourceforge.net,
linux-kernel@vger.kernel.org, Chao Yu <chao@kernel.org>
Subject: [PATCH v1 11/12] f2fs: cache: introduce tracepoints
Date: Thu, 20 Aug 2026 11:17:20 +0800 [thread overview]
Message-ID: <20260820031721.12218-12-chao@kernel.org> (raw)
In-Reply-To: <20260820031721.12218-1-chao@kernel.org>
This patch introduces ftrace tracepoints to observe and profile metadata
cache operations:
- trace_f2fs_cache_set_dirty to trace marking a cached block dirty
- trace_f2fs_write_cache to trace single block writeback submission
- trace_f2fs_write_caches to tracesbatch writeback and sync sessions
Signed-off-by: Chao Yu <chao@kernel.org>
---
fs/f2fs/cache.c | 2 ++
fs/f2fs/checkpoint.c | 6 ++++
fs/f2fs/segment.c | 3 ++
include/trace/events/f2fs.h | 71 +++++++++++++++++++++++++++++++++++++
4 files changed, 82 insertions(+)
diff --git a/fs/f2fs/cache.c b/fs/f2fs/cache.c
index c1e5c945391f..d57adf52c3a5 100644
--- a/fs/f2fs/cache.c
+++ b/fs/f2fs/cache.c
@@ -16,6 +16,7 @@
#include "f2fs.h"
#include "cache.h"
#include "node.h"
+#include <trace/events/f2fs.h>
#include "segment.h"
void f2fs_cache_wait_writeback_cond(struct f2fs_cached_block *entry,
@@ -78,6 +79,7 @@ bool f2fs_mark_cache_dirty(struct f2fs_cached_block *entry)
enum count_type type = IS_META_CACHE(cache) ?
F2FS_DIRTY_META : F2FS_DIRTY_NODES;
+ trace_f2fs_cache_set_dirty(entry, IS_META_CACHE(cache) ? META : NODE);
f2fs_cache_update_tag(entry, 0, F2FS_CACHE_TAG_DIRTY);
inc_page_count(cache->sbi, type);
return true;
diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
index 508132652693..8bb3b701ce89 100644
--- a/fs/f2fs/checkpoint.c
+++ b/fs/f2fs/checkpoint.c
@@ -550,6 +550,8 @@ static bool __f2fs_write_meta_cache(struct f2fs_cached_block *entry,
{
struct f2fs_sb_info *sbi = entry->cache->sbi;
+ trace_f2fs_write_cache(entry, META);
+
if (unlikely(f2fs_cp_error(sbi))) {
if (is_sbi_flag_set(sbi, SBI_IS_CLOSE)) {
f2fs_force_clear_cache_dirty(entry);
@@ -608,6 +610,8 @@ long f2fs_sync_meta_caches(struct f2fs_sb_info *sbi, long nr_to_write,
struct blk_plug plug;
bool background = nr_to_write != LONG_MAX;
+ trace_f2fs_write_caches(sbi, nr_to_write, 0, META);
+
blk_start_plug(&plug);
while ((nr = f2fs_cache_gang_lookup_tag(META_CACHE(sbi), entries,
@@ -663,6 +667,8 @@ long f2fs_sync_meta_caches(struct f2fs_sb_info *sbi, long nr_to_write,
blk_finish_plug(&plug);
+ trace_f2fs_write_caches(sbi, nr_to_write, nwritten, META);
+
return nwritten;
}
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index 3a585fc7c197..e37d57052b49 100644
--- a/fs/f2fs/segment.c
+++ b/fs/f2fs/segment.c
@@ -4170,6 +4170,9 @@ void f2fs_do_write_node_page(unsigned int nid, struct f2fs_io_info *fio)
{
struct f2fs_summary sum;
+ if (fio->is_cache)
+ trace_f2fs_write_cache(fio->cache_entry, NODE);
+
set_summary(&sum, nid, 0, 0);
do_write_page(&sum, fio);
diff --git a/include/trace/events/f2fs.h b/include/trace/events/f2fs.h
index 1dd9fc5afc46..7eb3528d35be 100644
--- a/include/trace/events/f2fs.h
+++ b/include/trace/events/f2fs.h
@@ -1429,6 +1429,50 @@ DEFINE_EVENT(f2fs__folio, f2fs_set_page_dirty,
TP_ARGS(folio, type)
);
+DECLARE_EVENT_CLASS(f2fs__cached_block,
+
+ TP_PROTO(struct f2fs_cached_block *block, int type),
+
+ TP_ARGS(block, type),
+
+ TP_STRUCT__entry(
+ __field(dev_t, dev)
+ __field(pgoff_t, index)
+ __field(int, type)
+ __field(int, dirty)
+ __field(int, uptodate)
+ ),
+
+ TP_fast_assign(
+ __entry->dev = block->cache->sbi->sb->s_dev;
+ __entry->index = block->index;
+ __entry->type = type;
+ __entry->dirty = f2fs_cache_test_dirty(block);
+ __entry->uptodate = f2fs_cache_test_uptodate(block);
+ ),
+
+ TP_printk("dev = (%d,%d), %s, index = %lu, dirty = %d, uptodate = %d",
+ show_dev(__entry->dev),
+ show_block_type(__entry->type),
+ (unsigned long)__entry->index,
+ __entry->dirty,
+ __entry->uptodate)
+);
+
+DEFINE_EVENT(f2fs__cached_block, f2fs_write_cache,
+
+ TP_PROTO(struct f2fs_cached_block *block, int type),
+
+ TP_ARGS(block, type)
+);
+
+DEFINE_EVENT(f2fs__cached_block, f2fs_cache_set_dirty,
+
+ TP_PROTO(struct f2fs_cached_block *block, int type),
+
+ TP_ARGS(block, type)
+);
+
TRACE_EVENT(f2fs_replace_atomic_write_block,
TP_PROTO(struct inode *inode, struct inode *cow_inode, pgoff_t index,
@@ -1573,6 +1617,33 @@ TRACE_EVENT(f2fs_writepages,
__entry->for_sync)
);
+TRACE_EVENT(f2fs_write_caches,
+
+ TP_PROTO(struct f2fs_sb_info *sbi, long nr_to_write, long nwritten, int type),
+
+ TP_ARGS(sbi, nr_to_write, nwritten, type),
+
+ TP_STRUCT__entry(
+ __field(dev_t, dev)
+ __field(long, nr_to_write)
+ __field(long, nwritten)
+ __field(int, type)
+ ),
+
+ TP_fast_assign(
+ __entry->dev = sbi->sb->s_dev;
+ __entry->nr_to_write = nr_to_write;
+ __entry->nwritten = nwritten;
+ __entry->type = type;
+ ),
+
+ TP_printk("dev = (%d,%d), %s, nr_to_write = %ld, nwritten = %ld",
+ show_dev(__entry->dev),
+ show_block_type(__entry->type),
+ __entry->nr_to_write,
+ __entry->nwritten)
+);
+
TRACE_EVENT(f2fs_readpages,
TP_PROTO(struct inode *inode, pgoff_t start, unsigned int nrpage),
--
2.49.0
next prev parent reply other threads:[~2026-08-20 3:17 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-20 3:17 [PATCH v1 00/12] f2fs: introduce metadata cache Chao Yu
2026-08-20 3:17 ` [PATCH v1 01/12] f2fs: cache: implement " Chao Yu
2026-08-20 5:08 ` [f2fs-dev] " Jaegeuk Kim
2026-08-20 5:13 ` Chao Yu
2026-08-20 3:17 ` [PATCH v1 02/12] f2fs: cache: initialize meta cache Chao Yu
2026-08-20 3:17 ` [PATCH v1 03/12] f2fs: cache: introduce shrinker Chao Yu
2026-08-20 3:17 ` [PATCH v1 04/12] f2fs: cache: introduce writeback thread Chao Yu
2026-08-20 5:09 ` [f2fs-dev] " Jaegeuk Kim
2026-08-20 3:17 ` [PATCH v1 05/12] f2fs: cache: use meta cache Chao Yu
2026-08-20 5:11 ` [f2fs-dev] " Jaegeuk Kim
2026-08-20 3:17 ` [PATCH v1 06/12] f2fs: cache: initialize node cache Chao Yu
2026-08-20 3:17 ` [PATCH v1 07/12] f2fs: cache: use " Chao Yu
2026-08-20 3:17 ` [PATCH v1 08/12] f2fs: cache: initialize compress cache Chao Yu
2026-08-20 3:17 ` [PATCH v1 09/12] f2fs: cache: use " Chao Yu
2026-08-20 3:17 ` [PATCH v1 10/12] f2fs: cache: support fault injection Chao Yu
2026-08-20 3:17 ` Chao Yu [this message]
2026-08-20 3:17 ` [PATCH v1 12/12] f2fs: cache: show per-cache usage in debugfs Chao Yu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260820031721.12218-12-chao@kernel.org \
--to=chao@kernel.org \
--cc=jaegeuk@kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox