[PATCH 1/6] [XFS] rename xfs_get_perag
Dave Chinner
david at fromorbit.com
Thu Dec 10 17:25:29 CST 2009
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 at fromorbit.com
More information about the xfs
mailing list