Re: [PATCH v2 3/8] xfs: get rid of XFS_IALLOC_BLOCKS macros

[Dave Chinner <david@xxxxxxxxxxxxx>]

On Thu, Dec 12, 2013 at 03:38:17PM +0800, Jeff Liu wrote:
> From: Jie Liu <jeff.liu@xxxxxxxxxx>
> 
> Get rid of XFS_IALLOC_BLOCKS() marcos, use mp->m_ialloc_blks directly.
> 
> Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
....
> @@ -1228,9 +1228,9 @@ xfs_difree(
>                       goto error0;
>               }
>  
> -             xfs_bmap_add_free(XFS_AGB_TO_FSB(mp,
> -                             agno, XFS_INO_TO_AGBNO(mp,rec.ir_startino)),
> -                             XFS_IALLOC_BLOCKS(mp), flist, mp);
> +             xfs_bmap_add_free(XFS_AGB_TO_FSB(mp, agno,
> +                               XFS_INO_TO_AGBNO(mp, rec.ir_startino)),
> +                               mp->m_ialloc_blks, flist, mp);
>       } else {
>               *delete = 0;
>  

git-am fails to apply this hunk, but I'm not sure why. I updated it
manually, the resulting patch looks identical.

Other than that, it looks good.

Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>

-- 
Dave Chinner
david@xxxxxxxxxxxxx