[RFC PATCH] xfs: fix calculation of freed inode cluster blocks

Dave Chinner david at fromorbit.com
Tue Dec 3 15:28:14 CST 2013


On Tue, Dec 03, 2013 at 03:00:54PM -0600, Ben Myers wrote:
> rec.ir_startino is an agino rather than an ino.  Use the correct macro
> when dealing with it in xfs_difree.
> 
> ---
> 
> This is very lightly tested so it's RFC.  Looks like using "ino to
> agbno" is actually harmless when you pass in an agino, so there
> shouldn't have been any trouble caused by this one.

Right, the XFS_INO_TO_AGBNO() macro will behave correctly when
passed an agino - it's the same as an inode from AG 0. Changing the
macro to the correct one is effectively a no-op, so just send a
patch and get it reviewed and integrated...

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com



More information about the xfs mailing list