Re: [PATCH 3/6] cleanup maxrecs calculation

[Christoph Hellwig <hch@xxxxxx>]

On Tue, Sep 16, 2008 at 03:36:41PM +1000, Dave Chinner wrote:
> On Mon, Sep 15, 2008 at 02:46:48AM +0200, Christoph Hellwig wrote:
> > +   mp->m_alloc_mxr[0] = xfs_allocbt_maxrecs(mp, sbp->sb_blocksize, 1);
> > +   mp->m_alloc_mxr[1] = xfs_allocbt_maxrecs(mp, sbp->sb_blocksize, 0);
> 
> That's kind of strange - index 0 gets configured with a leaf (?)
> value of 1, and index 1 with a value of 0. Can we change the order
> of one of these so that the function and the variables match?
> 
> Would a define for what the function parameter means make sense?
> That way the code would document itself....

Yeah, probably should do this, currently the patch just keeps the
calling conventions of the previous macros.