From 3e78b5d75cc6e88e8730d94d7e73ef269891fd52 Mon Sep 17 00:00:00 2001 From: Christopher Faylor Date: Tue, 13 Apr 2004 02:59:19 +0000 Subject: * thread.cc (pthread::create): Use thread mutex to control synchronization rather than creating a suspended thread. Wait for "cancellation event" to indicate that started thread has been properly initialized. (pthread::thread_init_wrapper): Use set_tls_self_pointer() to set tid and cygtls. Signal with cancel_event when done. --- winsup/cygwin/thread.cc | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) (limited to 'winsup/cygwin/thread.cc') diff --git a/winsup/cygwin/thread.cc b/winsup/cygwin/thread.cc index 56074425a..24c3f56ce 100644 --- a/winsup/cygwin/thread.cc +++ b/winsup/cygwin/thread.cc @@ -290,9 +290,9 @@ pthread::create (void *(*func) (void *), pthread_attr *newattr, function = func; arg = threadarg; + mutex.lock (); win32_obj_id = ::CreateThread (&sec_none_nih, attr.stacksize, - thread_init_wrapper, this, CREATE_SUSPENDED, - &thread_id); + thread_init_wrapper, this, 0, &thread_id); if (!win32_obj_id) { @@ -301,9 +301,12 @@ pthread::create (void *(*func) (void *), pthread_attr *newattr, } else { + if (WaitForSingleObject (cancel_event, 5000) != WAIT_OBJECT_0) + thread_printf ("event never arrived after CreateThread"); + ResetEvent (cancel_event); postcreate (); - ResumeThread (win32_obj_id); } + mutex.unlock (); } void @@ -1861,7 +1864,8 @@ DWORD WINAPI pthread::thread_init_wrapper (void *arg) { pthread *thread = (pthread *) arg; - _my_tls.tid = thread; + set_tls_self_pointer (thread); + SetEvent (thread->cancel_event); thread->mutex.lock (); -- cgit v1.2.3