On Sun, Jul 7, 2019 at 3:09 PM Thomas Munro <thomas.mu...@gmail.com> wrote: > + /* Timed out */ > + if (rc == 0) > + return true;
Here's a version without that bit, because I don't think we need it. > That still leaves the danger that the CV can be signalled some time > after ConditionVariableTimedSleep() returns. So now I'm wondering if > ConditionVariableCancelSleep() should signal the CV if it discovers > that this process is not in the proclist, on the basis that that must > indicate that we've been signalled even though we're not interested in > the message anymore, and yet some other process else might be > interested, and that might have been the only signal that is ever > going to be delivered by ConditionVariableSignal(). And a separate patch to do that. Thoughts? -- Thomas Munro https://enterprisedb.com
0001-Introduce-timed-waits-for-condition-variables-v6.patch
Description: Binary data
0002-Forward-received-condition-variable-signals-on-ca-v6.patch
Description: Binary data
