[PATCH 09/16] xfs: flatten the dquot lock ordering
Dave Chinner
david at fromorbit.com
Mon Dec 5 03:34:37 CST 2011
On Mon, Dec 05, 2011 at 04:11:21AM -0500, Christoph Hellwig wrote:
> On Mon, Dec 05, 2011 at 04:04:28PM +1100, Dave Chinner wrote:
> > > @@ -509,46 +518,27 @@ xfs_qm_dqpurge_int(
> > > */
> > > xfs_qm_detach_gdquots(mp);
> > >
> > > - again:
> > > - nmisses = 0;
> >
> > I don't think that nmisses is initialised to zero correctly anymore.
>
> We do a
>
> int nmisses = 0;
>
> at the top of the function. Now that there are no retry loops that's
> sufficient.
Hmmmm. My unmodified tree just has the declaration without
initialisation, and I didn't find any place where the initialisation
was added in the preceding patches. I'll go back and have another
look.
Cheers,
Dave.
--
Dave Chinner
david at fromorbit.com
More information about the xfs
mailing list