<p dir="ltr"><br>
2016年4月14日 02:30，"Eric Sandeen" <<a href="mailto:sandeen@sandeen.net">sandeen@sandeen.net</a>>写道：<br>
><br>
> This looks fine, but I think Eryu already sent this patch on 3/31:<br>
><br>
> [PATCH] xfs: add missing break in xfs_parseargs()</p>
<p dir="ltr">Hi,</p>
<p dir="ltr">Sorry for this duplicate patch... I think I missed that patch email from mail list. Please ignore this DUP patch.</p>
<p dir="ltr">Thanks,<br>
Zorro</p>
<p dir="ltr">><br>
> -Eric<br>
><br>
> On 4/13/16 1:22 PM, Zorro Lang wrote:<br>
> > This's a regression bug caused by:<br>
> ><br>
> >   2e74af0 xfs: convert mount option parsing to tokens<br>
> ><br>
> > xfstests xfs/299 can reproduce this bug, it will fail when do<br>
> > gpnoenforce test, the error output as below(diff with 299.out):<br>
> ><br>
> >  *** push past the soft block limit<br>
> >   [ROOT] 0 0 0 00 [--------] 3 0 0 00 [--------] 0 0 0 00 [--------]<br>
> >   -[NAME] 140 100 500 00 [--------] 4 4 10 00 [--------] 0 0 0 00 [--------]<br>
> >   +[NAME] 140 100 500 00 [7 days] 4 4 10 00 [--------] 0 0 0 00 [--------]<br>
> ><br>
> >  *** push past the hard inode limit (expect EDQUOT)<br>
> >   [ROOT] 0 0 0 00 [--------] 3 0 0 00 [--------] 0 0 0 00 [--------]<br>
> >   -[NAME] 140 100 500 00 [--------] 16 4 10 00 [--none--] 0 0 0 00 [--------]<br>
> >   +[NAME] 140 100 500 00 [7 days] 10 4 10 00 [7 days] 0 0 0 00 [--------]<br>
> ><br>
> >  *** push past the hard block limit (expect EDQUOT)<br>
> >   [ROOT] 0 0 0 00 [--------] 3 0 0 00 [--------] 0 0 0 00 [--------]<br>
> >   -[NAME] 540 100 500 00 [--none--] 16 4 10 00 [--none--] 0 0 0 00 [--------]<br>
> >   +[NAME] 500 100 500 00 [7 days] 10 4 10 00 [7 days] 0 0 0 00 [--------]<br>
> ><br>
> > It shows that group quota is still enforced. But this bug only can<br>
> > be reproduced when mount v5 xfs with gqnoenforce and pqnoenforce<br>
> > options. The reason as I found is a missed "break" in xfs_parseargs()<br>
> > function.<br>
> ><br>
> > After deal with "pqnoenforce", it should break. But the code keep<br>
> > running into "gquota/grpquota" lines, and make gquota enforced.<br>
> ><br>
> > Signed-off-by: Zorro Lang <<a href="mailto:zlang@redhat.com">zlang@redhat.com</a>><br>
> > ---<br>
> ><br>
> > Hi,<br>
> ><br>
> > This's a tiny bug(only one line patch). But it's belong to<br>
> > regression bug and take me a few hours to find the problem.<br>
> > So I hope it can be fixed:)<br>
> ><br>
> > Thanks,<br>
> > Zorro<br>
> ><br>
> >  fs/xfs/xfs_super.c | 1 +<br>
> >  1 file changed, 1 insertion(+)<br>
> ><br>
> > diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c<br>
> > index 187e14b..f3c0ffc 100644<br>
> > --- a/fs/xfs/xfs_super.c<br>
> > +++ b/fs/xfs/xfs_super.c<br>
> > @@ -350,6 +350,7 @@ xfs_parseargs(<br>
> >               case Opt_pqnoenforce:<br>
> >                       mp->m_qflags |= (XFS_PQUOTA_ACCT | XFS_PQUOTA_ACTIVE);<br>
> >                       mp->m_qflags &= ~XFS_PQUOTA_ENFD;<br>
> > +                     break;<br>
> >               case Opt_gquota:<br>
> >               case Opt_grpquota:<br>
> >                       mp->m_qflags |= (XFS_GQUOTA_ACCT | XFS_GQUOTA_ACTIVE |<br>
> ><br>
><br>
> _______________________________________________<br>
> xfs mailing list<br>
> <a href="mailto:xfs@oss.sgi.com">xfs@oss.sgi.com</a><br>
> <a href="http://oss.sgi.com/mailman/listinfo/xfs">http://oss.sgi.com/mailman/listinfo/xfs</a><br>
</p>