Re: condition variables - Mailing list pgsql-hackers

From Amit Kapila
Subject Re: condition variables
Date
Msg-id CAA4eK1Lvgi=-Nx0xB=nUN6DB8EMi0zpMweHWkJiAFVfcGmYGaA@mail.gmail.com
Whole thread Raw
In response to Re: condition variables  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: condition variables  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
On Mon, Aug 15, 2016 at 10:35 PM, Robert Haas <robertmhaas@gmail.com> wrote:
>>> Don't you need to set proc->cvSleeping = false in ConditionVariableSignal?
>>
>> I poked at this a bit... OK, a lot... and have some feedback:
>>
>> 1.  As above, we need to clear cvSleeping before setting the latch.
>
> Right, OK.
>

I have independently faced this problem while using your patch and for
now I have updated my local copy.  If possible, please send an updated
patch as this patch could be used for development of various
parallelism projects.

-- 
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com



pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: Assert(LWLockHeldByMeInMode(lock, LW_EXCLUSIVE))
Next
From: Julien Rouhaud
Date:
Subject: Re: Assert(LWLockHeldByMeInMode(lock, LW_EXCLUSIVE))