Re: [PATCH 1/6] [XFS] rename xfs_get_perag

[Dave Chinner <david@xxxxxxxxxxxxx>]

On Thu, Dec 10, 2009 at 06:15:11PM -0500, Christoph Hellwig wrote:
> On Wed, Dec 02, 2009 at 05:11:34PM +1100, Dave Chinner wrote:
> > xfs_get_perag is really getting the perag that an inode
> > belongs to based on it's inode number. Rename it appropriately
> > so we can use xfs_perag_get() to get the perag from a
> > provided ag number.
> 
> It does quite a bit more than the description suggests.  I also uses the
> new xfs_get_perag in some places and passes the perag around a bit more.

True - the patch grew beyond it's initial description...

> Might be worth splitting or at least needs a better description.

Better description. ;)

> Also I'm not sure the new xfs_perag_get_from_ino is actually all that useful,
> opencoding it is more descriptive and barely longer.

Ok, I'll redo it and open code all these as well.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx