> + /* > + * 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> _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs