f2fs: introduce tracepoint for f2fs_rename()

This patch adds tracepoints for f2fs_rename().

Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>

diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c
index a765db9e26c2cf59718a337ec5848088480a6b9f..ede6afb8176264db60bbb8a78de53f2496b83864 100644 (file)
--- a/fs/f2fs/namei.c
+++ b/fs/f2fs/namei.c
@@ -1315,21 +1315,27 @@ static int f2fs_rename2(struct mnt_idmap *idmap,
        if (flags & ~(RENAME_NOREPLACE | RENAME_EXCHANGE | RENAME_WHITEOUT))
                return -EINVAL;
 
+       trace_f2fs_rename_start(old_dir, old_dentry, new_dir, new_dentry,
+                                                               flags);
+
        err = fscrypt_prepare_rename(old_dir, old_dentry, new_dir, new_dentry,
                                     flags);
        if (err)
                return err;
 
-       if (flags & RENAME_EXCHANGE) {
-               return f2fs_cross_rename(old_dir, old_dentry,
-                                        new_dir, new_dentry);
-       }
+       if (flags & RENAME_EXCHANGE)
+               err = f2fs_cross_rename(old_dir, old_dentry,
+                                       new_dir, new_dentry);
+       else
        /*
         * VFS has already handled the new dentry existence case,
         * here, we just deal with "RENAME_NOREPLACE" as regular rename.
         */
-       return f2fs_rename(idmap, old_dir, old_dentry,
+               err = f2fs_rename(idmap, old_dir, old_dentry,
                                        new_dir, new_dentry, flags);
+
+       trace_f2fs_rename_end(old_dentry, new_dentry, flags, err);
+       return err;
 }
 
 static const char *f2fs_encrypted_get_link(struct dentry *dentry,

diff --git a/include/trace/events/f2fs.h b/include/trace/events/f2fs.h
index 793f82cc1515a81653b16f300620d254f7458b36..36d9e29ca3c5fdd010744ab8630205aafc638b55 100644 (file)
--- a/include/trace/events/f2fs.h
+++ b/include/trace/events/f2fs.h
@@ -866,6 +866,75 @@ TRACE_EVENT(f2fs_lookup_end,
                __entry->err)
 );
 
+TRACE_EVENT(f2fs_rename_start,
+
+       TP_PROTO(struct inode *old_dir, struct dentry *old_dentry,
+                       struct inode *new_dir, struct dentry *new_dentry,
+                       unsigned int flags),
+
+       TP_ARGS(old_dir, old_dentry, new_dir, new_dentry, flags),
+
+       TP_STRUCT__entry(
+               __field(dev_t,          dev)
+               __field(ino_t,          ino)
+               __string(old_name,      old_dentry->d_name.name)
+               __field(ino_t,          new_pino)
+               __string(new_name,      new_dentry->d_name.name)
+               __field(unsigned int,   flags)
+       ),
+
+       TP_fast_assign(
+               __entry->dev            = old_dir->i_sb->s_dev;
+               __entry->ino            = old_dir->i_ino;
+               __assign_str(old_name, old_dentry->d_name.name);
+               __entry->new_pino       = new_dir->i_ino;
+               __assign_str(new_name, new_dentry->d_name.name);
+               __entry->flags          = flags;
+       ),
+
+       TP_printk("dev = (%d,%d), old_dir = %lu, old_name: %s, "
+               "new_dir = %lu, new_name: %s, flags = %u",
+               show_dev_ino(__entry),
+               __get_str(old_name),
+               __entry->new_pino,
+               __get_str(new_name),
+               __entry->flags)
+);
+
+TRACE_EVENT(f2fs_rename_end,
+
+       TP_PROTO(struct dentry *old_dentry, struct dentry *new_dentry,
+                       unsigned int flags, int ret),
+
+       TP_ARGS(old_dentry, new_dentry, flags, ret),
+
+       TP_STRUCT__entry(
+               __field(dev_t,          dev)
+               __field(ino_t,          ino)
+               __string(old_name,      old_dentry->d_name.name)
+               __string(new_name,      new_dentry->d_name.name)
+               __field(unsigned int,   flags)
+               __field(int,            ret)
+       ),
+
+       TP_fast_assign(
+               __entry->dev            = old_dentry->d_sb->s_dev;
+               __entry->ino            = old_dentry->d_inode->i_ino;
+               __assign_str(old_name, old_dentry->d_name.name);
+               __assign_str(new_name, new_dentry->d_name.name);
+               __entry->flags          = flags;
+               __entry->ret            = ret;
+       ),
+
+       TP_printk("dev = (%d,%d), ino = %lu, old_name: %s, "
+               "new_name: %s, flags = %u, ret = %d",
+               show_dev_ino(__entry),
+               __get_str(old_name),
+               __get_str(new_name),
+               __entry->flags,
+               __entry->ret)
+);
+
 TRACE_EVENT(f2fs_readdir,
 
        TP_PROTO(struct inode *dir, loff_t start_pos, loff_t end_pos, int err),