On Tue, Aug 17, 2021 at 04:42:56PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@xxxxxxxxxx>
>
> Some of our tracepoints describe fields as "offset". That name doesn't
> describe any units, which makes the fields not very useful. Rename the
> fields to capture units and ensure the format is hexadecimal.
>
> "fileoff" means file offset, in units of fs blocks
> "pos" means file offset, in bytes
> "forkoff" means inode fork offset, in bytes
>
> The one remaining "offset" value is for iclogs, since that's the byte
> offset of the end of where we've written into the current iclog.
>
> Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx>
> ---
...
> @@ -2145,7 +2145,7 @@ DECLARE_EVENT_CLASS(xfs_swap_extent_class,
> __entry->fork_off = XFS_IFORK_BOFF(ip);
> ),
> TP_printk("dev %d:%d ino 0x%llx (%s), %s format, num_extents %d, "
> - "broot size %d, fork offset %d",
> + "broot size %d, forkoff %d",
forkoff 0x%x?
Other than that:
Reviewed-by: Carlos Maiolino <cmaiolino@xxxxxxxxxx>
--
Carlos
