> + if (max_blen > *blen) { > + if (max_blen_agno != startag) { > + ap->blkno = XFS_AGB_TO_FSB(mp, max_blen_agno, 0); > + ap->aeof = false; > + } > + *blen = max_blen; > + } A comment based on the commit message on why we do this would be nice here. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>