[PATCH 11/15] xfs: move v1 inode conversion to xfs_inode_from_disk

Dave Chinner david at fromorbit.com
Wed Feb 17 01:48:21 CST 2016


On Tue, Feb 16, 2016 at 11:32:04PM -0800, Christoph Hellwig wrote:
> On Wed, Feb 17, 2016 at 06:20:48PM +1100, Dave Chinner wrote:
> > +
> > +	sbversion |= XFS_SB_VERSION_NLINKBIT;
> > +	setlink_inode(id, be32_to_cpu(dip->di_nlink), type == DBM_DIR, security);
> > +
> 
> So we always convert to v2 inodes here.  Given that's it's xfs_check
> it's only in-core so harmless, but a comment here might be useful.

Yeah, probably. I'll have to look at this a bit closer - I think I
assumed that we're always going to have v2 inode format here (like
we do in the kernel), but that may not be true. I'll have a closer
look.

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com



More information about the xfs mailing list