Skip to contentSkip to conversation

gfs2: gfs2_iomap_end tracepoint: log block address

PATCH New Andreas Gruenbacher 2 participants 2 messages Mar 12, 2018 – Mar 15, 2018 GFS2 file system

lore
Share & tools
lore

Andreas Gruenbacher

author maintainer
#0 lore raw

PATCH 1/1 gfs2: gfs2_iomap_end tracepoint: log block address

In the gfs2_iomap_end tracepoint, log the physical block address, just as in the gfs2_bmap tracepoint.

---
 fs/gfs2/trace_gfs2.h | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
fs/gfs2/trace_gfs2.h +7 -2
index b9318b49ff8f..cb10b95efe0f 100644
@@ -515,6 +515,7 @@ TRACE_EVENT(gfs2_iomap_end,
 		__field(	u64,	inum			)
 		__field(	loff_t, offset			)
 		__field(	ssize_t, length			)
+		__field(	sector_t, pblock		)
 		__field(	u16,	flags			)
 		__field(	u16,	type			)
 		__field(	int,	ret			)
@@ -525,16 +526,20 @@ TRACE_EVENT(gfs2_iomap_end,
 		__entry->inum		= ip->i_no_addr;
 		__entry->offset		= iomap->offset;
 		__entry->length		= iomap->length;
+		__entry->pblock		= iomap->addr == IOMAP_NULL_ADDR ? 0 :
+					 (iomap->addr >> ip->i_inode.i_blkbits);
 		__entry->flags		= iomap->flags;
 		__entry->type		= iomap->type;
 		__entry->ret		= ret;
 	),
 
-	TP_printk("%u,%u bmap %llu iomap end %llu/%lu ty:%d flags:%08x rc:%d",
+	TP_printk("%u,%u bmap %llu iomap end %llu/%lu to %llu ty:%d flags:%08x rc:%d",
 		  MAJOR(__entry->dev), MINOR(__entry->dev),
 		  (unsigned long long)__entry->inum,
 		  (unsigned long long)__entry->offset,
-		  (unsigned long)__entry->length, (u16)__entry->type,
+		  (unsigned long)__entry->length,
+		  (long long)__entry->pblock,
+		  (u16)__entry->type,
 		  (u16)__entry->flags, __entry->ret)
 );
 
Signature · 1 line
2.14.3
#1 lore raw

PATCH 1/1 gfs2: gfs2_iomap_end tracepoint: log block address

----- Original Message -----

| In the gfs2_iomap_end tracepoint, log the physical block address, just
| as in the gfs2_bmap tracepoint.
| 
| Signed-off-by: Andreas Gruenbacher <[email protected]>
| ---
|  fs/gfs2/trace_gfs2.h | 9 +++++++--
|  1 file changed, 7 insertions(+), 2 deletions(-)

Hi,

Thanks. This is now pushed to the for-next branch of the linux-gfs2 tree:
https://git.kernel.org/pub/scm/linux/kernel/git/gfs2/linux-gfs2.git/commit/fs/gfs2?h=for-next&id=ee6ed857c8058177614b21a0efbf27e02c4e2a77

Regards,

Bob Peterson
Red Hat File Systems