On 1/30/20 7:33 AM, Christoph Hellwig wrote: > Just dereference bp->b_addr directly and make the code a little > simpler and more clear. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> ... > @@ -2747,7 +2744,7 @@ xfs_alloc_log_agf( > sizeof(xfs_agf_t) > }; > > - trace_xfs_agf(tp->t_mountp, XFS_BUF_TO_AGF(bp), fields, _RET_IP_); > + trace_xfs_agf(tp->t_mountp, bp->b_addr, fields, _RET_IP_); Why is this not passing in agf?