On Thu, Aug 12, 2021 at 09:10:56AM +0100, Christoph Hellwig wrote:

On Tue, Aug 10, 2021 at 03:28:49PM +1000, Dave Chinner wrote:

quoted

From: Dave Chinner <redacted>

Introduce a helper function xfs_buf_daddr() to extract the disk
address of the buffer from the struct xfs_buf. This will replace
direct accesses to bp->b_bn and bp->b_maps[0].bm_bn, as well as
the XFS_BUF_ADDR() macro.

This patch introduces the helper function and replaces all uses of
XFS_BUF_ADDR() as this is just a simple sed replacement.

The end result looks sane, but I would have preferred to do one patch
that just does the script rename of XFS_BUF_ADDR, and one ore more to
clean up the rest.

Which is exactly what this patch does - I should have looked at it
more carefully and not just the applied series.

So maybe tweak the above commit message bit, otherwise:

Reviewed-by: Christoph Hellwig <hch@lst.de>