Re: [PATCH v2 0/8] xfs: Introduce a common helper to calculate the number of blocks per cluster

[Dave Chinner <david@xxxxxxxxxxxxx>]

On Thu, Dec 12, 2013 at 03:37:42PM +0800, Jeff Liu wrote:
> Hi Folks,
> 
> Here is the v2 patch series to introduce a common helper which could be used
> to calculate the number of file system blocks in an inode cluster.
> 
> v2->v1:
> * Rename xfs_ialloc_blks_per_cluster() to xfs_icluster_size_fsb() as per 
> dave's
>   suggestion.
> * Remove three macros 
> XFS_IALLOC_INODES/XFS_INODE_CLUSTER_SIZE/XFS_IALLOC_BLOCKS
>   separately as preparation steps.
> * s/ninodes/inodes_per_cluster/g, as the later is more meaningful.
> * Remove wrong commit notes I previously written for using 
> xfs_ialloc_blks_per_cluster
>   in xfs_imap(). 

All looks good, Jeff. Nice work, and thanks for adding all the extra
cleanups to it :)

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx