1.5.24 (and later): race condition in sigproc.cc

[Christopher Faylor]

On Sat, Jul 14, 2007 at 11:29:44AM -0400, Christopher Faylor wrote:
>On Sat, Jul 14, 2007 at 11:18:13AM -0400, Christopher Faylor wrote:
>>On Sat, Jul 14, 2007 at 07:03:44AM -0700, Scott Stanton wrote:
>>>Unfortunately the NO_COPY change doesn't appear to solve the problem.  
>>>
>>>The semantics of DllMain are that it acts like a monitor so only one
>>>thread can be inside the DllMain routine at a time.
>>
>>You are stating this with great assurance.  Do you have a reference?  This
>>does not jive with my understanding of the way this is supposed to work
>>but I'm willing to be educated by a definitive source.
>
>Nevermind.  I've reeducated myself on the loader lock.  Have you noticed
>any negative effects from just removing the in_dllentry test entirely?
>This flag may be an artifact and may no longer be required.

I've removed the flag entirely and am rebuilding a snapshot now.

cgf