On Wed, Apr 23, 2014 at 04:12:33PM +1000, Dave Chinner wrote: > On Thu, Apr 10, 2014 at 12:11:03PM -0400, Brian Foster wrote: > > Account for the finobt in calc_mkfs(). > > > > Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx> > > --- > > repair/xfs_repair.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/repair/xfs_repair.c b/repair/xfs_repair.c > > index 08b25f0..356125b 100644 > > --- a/repair/xfs_repair.c > > +++ b/repair/xfs_repair.c > > @@ -407,6 +407,8 @@ calc_mkfs(xfs_mount_t *mp) > > bcntbt_root = bnobt_root + 1; > > inobt_root = bnobt_root + 2; > > fino_bno = inobt_root + XFS_MIN_FREELIST_RAW(1, 1, mp) + 1; > > + if (xfs_sb_version_hasfinobt(&mp->m_sb)) > > + fino_bno++; > > The comment about this code needs updating, too. > Oops, indeed. How about the following (also fixed the line length)? /* * Pre-calculate the geometry of ag 0. We know what it looks like * because we know what mkfs does: 2 allocation btree roots (by block * and by size), the inode allocation btree root, the free inode * allocation btree root (if enabled) and some number of blocks to * prefill the agfl. */ Brian > Cheers, > > Dave. > -- > Dave Chinner > david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
