[PATCH] xfs: xfs_swap_extents needs to handle dynamic fork offsets
Christoph Hellwig
hch at infradead.org
Thu Jan 14 00:35:10 CST 2010
On Wed, Jan 13, 2010 at 04:32:12PM -0600, Alex Elder wrote:
> Christoph Hellwig wrote:
> >> + TP_printk("dev %d:%d %s inode 0x%llx, %s format, num_extents %d, "
> >> + "Max in-fork extents %d, broot size %d, fork offset %d",
> >
> > It would be nice to keep the
> >
> > "dev %d:%d ino 0x%llx"
> >
> > prefix as a convention so that all trace records are similar at their
> > beginning.
>
> Perhaps:
> + TP_printk("dev %d:%d inode 0x%llx (%s), %s format, num_extents %d, "
> ^
> +--- symbolic entry->which
Sounds fine, except s/inode/ino/ above to stay consistant.
More information about the xfs
mailing list