Re: [PATCH 2/5] xfs: limit extent length for allocation to AG size

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

> +     /*
> +      * The requested extent can be larger than an AG, so trim the block
> +      * count back to the maximum sized extent in an AG. A typical empty AG
> +      * consumes 1 block for headers, 1 block for each btree root (3) and 4
> +      * blocks for the free list.
> +      */
> +     args.maxlen = MIN(ap->alen, mp->m_sb.sb_agblocks - 8);

Maybe add a define for the over head blocks to document it more clearly?

Otherwise looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>