xfs
[Top] [All Lists]

Re: [PATCH] xfs: don't fill statvfs with project quota for a directory i

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: [PATCH] xfs: don't fill statvfs with project quota for a directory if it was not enabled.
From: Chandra Seetharaman <sekharan@xxxxxxxxxx>
Date: Mon, 02 Apr 2012 13:36:13 -0500
Cc: Jeff Liu <jeff.liu@xxxxxxxxxx>, xfs@xxxxxxxxxxx, Ben Myers <bpm@xxxxxxx>
In-reply-to: <20120402163903.GA6425@xxxxxxxxxxxxx>
Organization: IBM
References: <4F77DFDB.1040802@xxxxxxxxxx> <20120402163903.GA6425@xxxxxxxxxxxxx>
Reply-to: sekharan@xxxxxxxxxx
Will look into it, and report back.

On Mon, 2012-04-02 at 12:39 -0400, Christoph Hellwig wrote:
> On Sun, Apr 01, 2012 at 12:55:55PM +0800, Jeff Liu wrote:
> > Hello,
> > 
> > I can trigger a BUG() at fs/xfs/xfs_dquot.c on vanilla kernel 3.3.0 by the 
> > following steps:
> > 
> > 1. mount a XFS partition without 'pquota' option.
> >    /dev/sda7 on /xfs type xfs (rw)
> > 
> > 2. setup project1 on it.
> >    $ cat /etc/projects 
> >    1:/xfs
> >    $ cat /etc/projid
> >    project1:1
> >    $ sudo xfs_quota -x -c 'project -s project1' /xfs
> > 
> > 3. du -sh /xfs
> 
> Can you wire this up as a test case for xfstests?
> 
> > +               ((mp->m_qflags & (XFS_PQUOTA_ACCT|XFS_OQUOTA_ENFD))) ==
> > +                (XFS_PQUOTA_ACCT|XFS_OQUOTA_ENFD)) {
> 
> This check is supposed to return false.  I guess Chandras separate
> project quota inode preparations somehow broke it.
> 
> Chandra, can you look into this issue?
> 


<Prev in Thread] Current Thread [Next in Thread>