f2fs: introduce trace_f2fs_replace_atomic_write_block
authorChao Yu <chao@kernel.org>
Mon, 9 Jan 2023 03:44:49 +0000 (11:44 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 11 Mar 2023 12:55:17 +0000 (13:55 +0100)
[ Upstream commit 2f3a9ae990a7881c9a57a073bb52ebe34fdc3160 ]

Commit 3db1de0e582c ("f2fs: change the current atomic write way")
removed old tracepoints, but it missed to add new one, this patch
fixes to introduce trace_f2fs_replace_atomic_write_block to trace
atomic_write commit flow.

Fixes: 3db1de0e582c ("f2fs: change the current atomic write way")
Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
fs/f2fs/segment.c
include/trace/events/f2fs.h

index c1d0713666ee5858d1f54f21f519f87f4f897d6d..af9a3b7996b4d61a4ec6ad5f6cd9ed7e3c73e7dd 100644 (file)
@@ -250,6 +250,9 @@ retry:
        }
 
        f2fs_put_dnode(&dn);
+
+       trace_f2fs_replace_atomic_write_block(inode, F2FS_I(inode)->cow_inode,
+                                       index, *old_addr, new_addr, recover);
        return 0;
 }
 
index ff57e7f9914ccb6abcd5261f0b5c76e5440a6c5d..e57f867191ef10deccf7b29c06704eef8ecdf7a8 100644 (file)
@@ -1286,6 +1286,43 @@ DEFINE_EVENT(f2fs__page, f2fs_vm_page_mkwrite,
        TP_ARGS(page, type)
 );
 
+TRACE_EVENT(f2fs_replace_atomic_write_block,
+
+       TP_PROTO(struct inode *inode, struct inode *cow_inode, pgoff_t index,
+                       block_t old_addr, block_t new_addr, bool recovery),
+
+       TP_ARGS(inode, cow_inode, index, old_addr, new_addr, recovery),
+
+       TP_STRUCT__entry(
+               __field(dev_t,  dev)
+               __field(ino_t,  ino)
+               __field(ino_t,  cow_ino)
+               __field(pgoff_t, index)
+               __field(block_t, old_addr)
+               __field(block_t, new_addr)
+               __field(bool, recovery)
+       ),
+
+       TP_fast_assign(
+               __entry->dev            = inode->i_sb->s_dev;
+               __entry->ino            = inode->i_ino;
+               __entry->cow_ino        = cow_inode->i_ino;
+               __entry->index          = index;
+               __entry->old_addr       = old_addr;
+               __entry->new_addr       = new_addr;
+               __entry->recovery       = recovery;
+       ),
+
+       TP_printk("dev = (%d,%d), ino = %lu, cow_ino = %lu, index = %lu, "
+                       "old_addr = 0x%llx, new_addr = 0x%llx, recovery = %d",
+               show_dev_ino(__entry),
+               __entry->cow_ino,
+               (unsigned long)__entry->index,
+               (unsigned long long)__entry->old_addr,
+               (unsigned long long)__entry->new_addr,
+               __entry->recovery)
+);
+
 TRACE_EVENT(f2fs_filemap_fault,
 
        TP_PROTO(struct inode *inode, pgoff_t index, unsigned long ret),