Re: [PATCH] xfs: xfs_swap_extents needs to handle dynamic fork offsets

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

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.