> - xfs_buf_t *bp = (xfs_buf_t *)node; > + struct xfs_buf *bp = (struct xfs_buf *)node; I'd much prefer to use container_of here (although it seems that currently only exists under the name list_entry in xfsprogs). But I guess that's better left for a later patch.. Otherwise looks fine, Reviewed-by: Christoph Hellwig <hch at lst.de>