This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Separate pthread fixes #1


On Thu, Jun 04, 2009 at 12:11:24AM +0100, Dave Korn wrote:
>
>  The attached patch separates out the uncontroversial change to the
>__cygwin_lock* functions.
>
>winsup/cygwin/ChangeLog
>
>	* thread.cc (__cygwin_lock_lock):  Delete racy optimisation.
>	(__cygwin_lock_unlock):  Likewise.
>
>  OK?

Yes.  Thanks.

FWIW, I have made this same change many times over the years but I was
always afraid of the performance hit so I've ended up reverting it.
Since you've demonstrated a real problem, performance concerns obviously
don't matter.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]