[PATCH 1/4] xfsprogs: Remove incore use of XFS_OQUOTA_ENFD and XFS_OQUOTA_CHKD

[Chandra Seetharaman]

On Thu, 2012-05-03 at 15:04 -0500, Ben Myers wrote:
> On Mon, Jan 23, 2012 at 11:32:49AM -0600, Chandra Seetharaman wrote:
> > >From ba28fc49821079a734f24af05671155f6b32b20c Mon Sep 17 00:00:00 2001
> > From: Chandra Seetharaman <sekharan at us.ibm.com>
> > Date: Tue, 13 Dec 2011 15:55:34 -0600
> > Subject: [PATCH 1/4] Remove incore use of XFS_OQUOTA_ENFD and XFS_OQUOTA_CHKD. Instead,
> >  start using XFS_GQUOTA_.* XFS_PQUOTA_.* counterparts.
> > 
> > No changes is made to the on-disk version of the superblock yet. On-disk
>              are						   The 
> 
will fix
 
> > copy still uses XFS_OQUOTA_ENFD and XFS_OQUOTA_CHKD.
> <

<snip>
> >  
> > +static inline int xfs_sb_version_hasnooquota(xfs_sb_t *sbp)
> 
> IIRC, in the kernel you used xfs_sb_version_has_no_oquota.  I think you should
> use the same name in xfsprogs.

yes, will fix.
> 
> > +{
> > +	return xfs_sb_version_hasmorebits(sbp) &&
> > +		(sbp->sb_features2 & XFS_SB_VERSION2_NO_OQUOTA);
> > +}
> > +
> > +static inline void xfs_sb_version_addnooquota(xfs_sb_t *sbp)
> > +{
> > +	sbp->sb_versionnum |= XFS_SB_VERSION_MOREBITSBIT;
> > +	sbp->sb_features2 |= XFS_SB_VERSION2_PROJID32BIT;
> 
> I'm not sure about this.  I don't think addnooquota necessarily implies a 32
> bit project id.  What say you?

It is cut-n-paste issue, it should be XFS_SB_VERSION2_NO_OQUOTA
will fix it.
> 
> Else, the patch looks pretty good.
> 
> -Ben
>