On 2022/9/15 21:44, Zhang Qilong wrote:
For f2fs_update_extent_tree_range_compressed, keeping
the trace and function name consistent to distinguish
from trace of f2fs_update_extent_tree_range. And we add
"c_len" into the trace, result like:
[003] ..... 228.568157: f2fs_update_extent_tree_range_compressed:
dev = (8,0), ino = 5, pgofs = 96, blkaddr = 2548760,
len = 4, c_len = 1
Signed-off-by: Zhang Qilong <zhangqilo...@huawei.com>
---
fs/f2fs/extent_cache.c | 2 +-
include/trace/events/f2fs.h | 36 ++++++++++++++++++++++++++++++++++++
2 files changed, 37 insertions(+), 1 deletion(-)
diff --git a/fs/f2fs/extent_cache.c b/fs/f2fs/extent_cache.c
index 761fd42c93f2..df41eca6386a 100644
--- a/fs/f2fs/extent_cache.c
+++ b/fs/f2fs/extent_cache.c
@@ -675,7 +675,7 @@ void f2fs_update_extent_tree_range_compressed(struct inode
*inode,
struct rb_node **insert_p = NULL, *insert_parent = NULL;
bool leftmost = false;
- trace_f2fs_update_extent_tree_range(inode, fofs, blkaddr, llen);
+ trace_f2fs_update_extent_tree_range_compressed(inode, fofs, blkaddr,
llen, c_len);
/* it is safe here to check FI_NO_EXTENT w/o et->lock in ro image */
if (is_inode_flag_set(inode, FI_NO_EXTENT))
diff --git a/include/trace/events/f2fs.h b/include/trace/events/f2fs.h
index b262985f0c3a..54a9f6feca1b 100644
--- a/include/trace/events/f2fs.h
+++ b/include/trace/events/f2fs.h
@@ -1606,6 +1606,42 @@ TRACE_EVENT(f2fs_update_extent_tree_range,
__entry->len)
);
+TRACE_EVENT(f2fs_update_extent_tree_range_compressed,
Why not adding parameter c_len in trace_f2fs_update_extent_tree_range()
to avoid duplicated codes...
Thanks,
+
+ TP_PROTO(struct inode *inode, unsigned int pgofs, block_t blkaddr,
+ unsigned int len,
+ unsigned int c_len),
+
+ TP_ARGS(inode, pgofs, blkaddr, len, c_len),
+
+ TP_STRUCT__entry(
+ __field(dev_t, dev)
+ __field(ino_t, ino)
+ __field(unsigned int, pgofs)
+ __field(u32, blk)
+ __field(unsigned int, len)
+ __field(unsigned int, c_len)
+ ),
+
+ TP_fast_assign(
+ __entry->dev = inode->i_sb->s_dev;
+ __entry->ino = inode->i_ino;
+ __entry->pgofs = pgofs;
+ __entry->blk = blkaddr;
+ __entry->len = len;
+ __entry->c_len = c_len;
+ ),
+
+ TP_printk("dev = (%d,%d), ino = %lu, pgofs = %u, "
+ "blkaddr = %u, len = %u, "
+ "c_len = %u",
+ show_dev_ino(__entry),
+ __entry->pgofs,
+ __entry->blk,
+ __entry->len,
+ __entry->c_len)
+);
+
TRACE_EVENT(f2fs_shrink_extent_tree,
TP_PROTO(struct f2fs_sb_info *sbi, unsigned int node_cnt,
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
