Re: [PATCH] conditions variables module

2008-08-11 Thread James Youngman
On Fri, Aug 8, 2008 at 8:10 PM, Yoann Vandoorselaere [EMAIL PROTECTED] wrote: The supposed way to use pthread_atfork is that the prepare fork handler will be called prior to the process forking so that the application can lock the program to solidify it in a known state. Yes, my point is that

Re: [PATCH] conditions variables module

2008-08-08 Thread James Youngman
On Fri, Aug 8, 2008 at 9:43 AM, Yoann Vandoorselaere [EMAIL PROTECTED] wrote: Le vendredi 08 août 2008 à 00:37 +0200, Bruno Haible a écrit : gl_thread_atfork ?! Sounds advanced and weird to implement as well. This one is especially important for library using threads: if an application fork,

Re: [PATCH] conditions variables module

2008-08-08 Thread Yoann Vandoorselaere
Le vendredi 08 août 2008 à 17:10 +0100, James Youngman a écrit : On Fri, Aug 8, 2008 at 9:43 AM, Yoann Vandoorselaere [EMAIL PROTECTED] wrote: Le vendredi 08 août 2008 à 00:37 +0200, Bruno Haible a écrit : gl_thread_atfork ?! Sounds advanced and weird to implement as well. This one is

Re: [PATCH] conditions variables module

2008-08-08 Thread Yoann Vandoorselaere
Hi Bruno, Le vendredi 08 août 2008 à 00:01 +0200, Bruno Haible a écrit : I'm not competent to write the Win32 implementation unfortunately. Will you be able to do this part of the work? Yes. Win32 has the necessary primitives, it's only a matter of looking them up... I can do that.

Re: [PATCH] conditions variables module

2008-08-08 Thread Yoann Vandoorselaere
Le vendredi 08 août 2008 à 00:37 +0200, Bruno Haible a écrit : - Include a new glthread module (which doesn't handle sched_yield, since that might pull another dependency in. I guess we need yet another separate module for this one). It's a pity, because one cannot write a reasonable test

Re: [PATCH] conditions variables module

2008-08-07 Thread Bruno Haible
Hi Yoann, Attached is an initial implementation of the glcond module, based on your lock module. I like about it that its API is in the same spirit as 'lock' and 'tls'. A unit test and a Woe32 implementation are still missing. The prefix 'gl' for the module and header file is probably

Re: [PATCH] conditions variables module

2008-08-07 Thread Yoann Vandoorselaere
Hello Bruno, Le jeudi 07 août 2008 à 11:15 +0200, Bruno Haible a écrit : Attached is an initial implementation of the glcond module, based on your lock module. I like about it that its API is in the same spirit as 'lock' and 'tls'. A unit test and a Woe32 implementation are still

Re: [PATCH] conditions variables module

2008-08-07 Thread Bruno Haible
Hi Yoann, Attached is an updated patch: Thanks. - Include a new glthread module (which doesn't handle sched_yield, since that might pull another dependency in. I guess we need yet another separate module for this one). It's a pity, because one cannot write a reasonable test program that