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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxx

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs




[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux