Re: [PATCH v2] Use atomic_t and wait_event to track dquot pincount

[Lachlan McIlroy <lachlan@xxxxxxx>]

Peter Leckie wrote:
> Dave Chinner wrote:
> > but it doesn't fix the underlying problem that was causing the
> > spurious wakeups, which is the fact that xfs_qm_dqflush() is not
> > obeying non-blocking flush directions. The patch below should fix
> > that. Can you please test it before you add your patch?
> >   
> Yeah I already had this idea I  just have not posted a patch because 
> Lachlan though
> it might introduce a deadlock.
I suggested some changes a while back to make tail pushing non-blocking
and Dave thought it might cause a deadlock.

http://oss.sgi.com/archives/xfs/2008-07/msg00472.html

I actually did hit a deadlock with this change but could not figure out
why.  It may have been the same issue Pete is trying to fix here.

> If you think this is a good Idea I will 
> update my patch
> to  be non blocking.