f2fs: trace checkpoint reason in fsync()
This patch slightly changes need_do_checkpoint to return the detail info that indicates why we need do checkpoint, then caller could print it with trace message. Signed-off-by: Chao Yu <yuchao0@huawei.com> Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
This commit is contained in:
@@ -136,6 +136,18 @@ TRACE_DEFINE_ENUM(CP_TRIMMED);
|
||||
{ CP_UMOUNT, "Umount" }, \
|
||||
{ CP_TRIMMED, "Trimmed" })
|
||||
|
||||
#define show_fsync_cpreason(type) \
|
||||
__print_symbolic(type, \
|
||||
{ CP_NO_NEEDED, "no needed" }, \
|
||||
{ CP_NON_REGULAR, "non regular" }, \
|
||||
{ CP_HARDLINK, "hardlink" }, \
|
||||
{ CP_SB_NEED_CP, "sb needs cp" }, \
|
||||
{ CP_WRONG_PINO, "wrong pino" }, \
|
||||
{ CP_NO_SPC_ROLL, "no space roll forward" }, \
|
||||
{ CP_NODE_NEED_CP, "node needs cp" }, \
|
||||
{ CP_FASTBOOT_MODE, "fastboot mode" }, \
|
||||
{ CP_SPEC_LOG_NUM, "log type is 2" })
|
||||
|
||||
struct victim_sel_policy;
|
||||
struct f2fs_map_blocks;
|
||||
|
||||
@@ -210,14 +222,14 @@ DEFINE_EVENT(f2fs__inode, f2fs_sync_file_enter,
|
||||
|
||||
TRACE_EVENT(f2fs_sync_file_exit,
|
||||
|
||||
TP_PROTO(struct inode *inode, int need_cp, int datasync, int ret),
|
||||
TP_PROTO(struct inode *inode, int cp_reason, int datasync, int ret),
|
||||
|
||||
TP_ARGS(inode, need_cp, datasync, ret),
|
||||
TP_ARGS(inode, cp_reason, datasync, ret),
|
||||
|
||||
TP_STRUCT__entry(
|
||||
__field(dev_t, dev)
|
||||
__field(ino_t, ino)
|
||||
__field(int, need_cp)
|
||||
__field(int, cp_reason)
|
||||
__field(int, datasync)
|
||||
__field(int, ret)
|
||||
),
|
||||
@@ -225,15 +237,15 @@ TRACE_EVENT(f2fs_sync_file_exit,
|
||||
TP_fast_assign(
|
||||
__entry->dev = inode->i_sb->s_dev;
|
||||
__entry->ino = inode->i_ino;
|
||||
__entry->need_cp = need_cp;
|
||||
__entry->cp_reason = cp_reason;
|
||||
__entry->datasync = datasync;
|
||||
__entry->ret = ret;
|
||||
),
|
||||
|
||||
TP_printk("dev = (%d,%d), ino = %lu, checkpoint is %s, "
|
||||
TP_printk("dev = (%d,%d), ino = %lu, cp_reason: %s, "
|
||||
"datasync = %d, ret = %d",
|
||||
show_dev_ino(__entry),
|
||||
__entry->need_cp ? "needed" : "not needed",
|
||||
show_fsync_cpreason(__entry->cp_reason),
|
||||
__entry->datasync,
|
||||
__entry->ret)
|
||||
);
|
||||
|
Reference in New Issue
Block a user