Re: [PATCH 09/14] xfs: fix maxlevels comparisons in the btree staging code

Christoph Hellwig
2021-09-21T08:56:51Z

On Fri, Sep 17, 2021 at 06:29:59PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@kernel.org>
>
> The btree geometry computation function has an off-by-one error in that
> it does not allow maximally tall btrees (nlevels == XFS_BTREE_MAXLEVELS).
> This can result in repairs failing unnecessarily on very fragmented
> filesystems. Subsequent patches to remove MAXLEVELS usage in favor of
> the per-btree type computations will make this a much more likely
> occurrence.

Shouldn't this go in first as a fix?

Looks good:

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