* configure.in (MONO_VERSION): Emit it into config.make. Don't build Consts.cs.
[mono.git] / mono / io-layer / threads.c
index a4d09dca219f98c0c6cd423e55ecb0bf86166ff4..989c6e80eeb1cd81a9ec54916c4eb3ec240fc17c 100644 (file)
@@ -8,10 +8,6 @@
  */
 
 #include <config.h>
-#if HAVE_BOEHM_GC
-#include <mono/os/gc_wrapper.h>
-#include "mono/utils/mono-hash.h"
-#endif
 #include <stdio.h>
 #include <glib.h>
 #include <string.h>
@@ -48,14 +44,11 @@ static mono_once_t thread_hash_once = MONO_ONCE_INIT;
 static mono_mutex_t thread_hash_mutex = MONO_MUTEX_INITIALIZER;
 static GHashTable *thread_hash=NULL;
 
-#if HAVE_BOEHM_GC
-static MonoGHashTable *tls_gc_hash = NULL;
-#endif
-
+static void thread_close (gpointer handle, gpointer data);
 static gboolean thread_own (gpointer handle);
 
 struct _WapiHandleOps _wapi_thread_ops = {
-       NULL,                           /* close_shared */
+       thread_close,                   /* close */
        NULL,                           /* signal */
        thread_own,                     /* own */
        NULL,                           /* is_owned */
@@ -77,9 +70,19 @@ static void thread_ops_init (void)
 #endif
 }
 
+static void thread_close (gpointer handle, gpointer data)
+{
+       struct _WapiHandle_thread *thread_handle = (struct _WapiHandle_thread *)data;
+
+#ifdef DEBUG
+       g_message ("%s: closing thread handle %p", __func__, handle);
+#endif
+
+       g_ptr_array_free (thread_handle->owned_mutexes, TRUE);
+}
+
 static gboolean thread_own (gpointer handle)
 {
-       struct _WapiHandleShared shared_handle;
        struct _WapiHandle_thread *thread_handle;
        gboolean ok;
        
@@ -87,32 +90,17 @@ static gboolean thread_own (gpointer handle)
        g_message ("%s: owning thread handle %p", __func__, handle);
 #endif
 
-       ok = _wapi_copy_handle (handle, WAPI_HANDLE_THREAD,
-                               &shared_handle);
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
        if (ok == FALSE) {
-               g_warning ("%s: error copying thread handle %p", __func__,
+               g_warning ("%s: error looking up thread handle %p", __func__,
                           handle);
                return(FALSE);
        }
-       thread_handle = &shared_handle.u.thread;
-
-       if (thread_handle->owner_pid != getpid()) {
-#ifdef DEBUG
-               g_message ("%s: can't join thread, %d not owner process %d",
-                          __func__, getpid(), thread_handle->owner_pid);
-#endif
-               /* FIXME: might need to return TRUE here so that other
-                * processes can WaitFor thread handles
-                */
-               return(FALSE);
-       }
        
        if (thread_handle->joined == FALSE) {
                _wapi_timed_thread_join (thread_handle->thread, NULL, NULL);
                thread_handle->joined = TRUE;
-
-               _wapi_replace_handle (handle, WAPI_HANDLE_THREAD,
-                                     &shared_handle);
        }
 
        return(TRUE);
@@ -120,26 +108,27 @@ static gboolean thread_own (gpointer handle)
 
 static void thread_exit(guint32 exitstatus, gpointer handle)
 {
-       struct _WapiHandleShared shared_handle;
        struct _WapiHandle_thread *thread_handle;
        gboolean ok;
        int thr_ret;
-
+       int i;
+       
        pthread_cleanup_push ((void(*)(void *))_wapi_handle_unlock_handle,
                              handle);
        thr_ret = _wapi_handle_lock_handle (handle);
        g_assert (thr_ret == 0);
        
-       ok = _wapi_copy_handle (handle, WAPI_HANDLE_THREAD,
-                               &shared_handle);
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
        if (ok == FALSE) {
-               g_warning ("%s: error copying thread handle %p", __func__,
+               g_warning ("%s: error looking up thread handle %p", __func__,
                           handle);
                return;
        }
-       thread_handle = &shared_handle.u.thread;
        
-       _wapi_mutex_check_abandoned (getpid (), thread_handle->thread->id);
+       for (i = 0; i < thread_handle->owned_mutexes->len; i++) {
+               _wapi_mutex_abandon (g_ptr_array_index (thread_handle->owned_mutexes, i), getpid (), thread_handle->thread->id);
+       }
 
 #ifdef DEBUG
        g_message ("%s: Recording thread handle %p exit status", __func__,
@@ -149,9 +138,7 @@ static void thread_exit(guint32 exitstatus, gpointer handle)
        thread_handle->exitstatus = exitstatus;
        thread_handle->state = THREAD_STATE_EXITED;
 
-       _wapi_replace_handle (handle, WAPI_HANDLE_THREAD, &shared_handle);
-       
-       _wapi_shared_handle_set_signal_state (handle, TRUE);
+       _wapi_handle_set_signal_state (handle, TRUE, TRUE);
 
        thr_ret = _wapi_handle_unlock_handle (handle);
        g_assert (thr_ret == 0);
@@ -168,7 +155,7 @@ static void thread_exit(guint32 exitstatus, gpointer handle)
        thr_ret = mono_mutex_lock(&thread_hash_mutex);
        g_assert (thr_ret == 0);
        
-       g_hash_table_remove(thread_hash, &thread_handle->thread->id);
+       g_hash_table_remove (thread_hash, (gpointer)(thread_handle->thread->id));
 
        thr_ret = mono_mutex_unlock(&thread_hash_mutex);
        g_assert (thr_ret == 0);
@@ -180,7 +167,7 @@ static void thread_exit(guint32 exitstatus, gpointer handle)
 
 static void thread_hash_init(void)
 {
-       thread_hash = g_hash_table_new(g_int_hash, g_int_equal);
+       thread_hash = g_hash_table_new (NULL, NULL);
 }
 
 /**
@@ -193,7 +180,9 @@ static void thread_hash_init(void)
  * @create: If 0, the new thread is ready to run immediately.  If
  * %CREATE_SUSPENDED, the new thread will be in the suspended state,
  * requiring a ResumeThread() call to continue running.
- * @tid: If non-NULL, the ID of the new thread is stored here.
+ * @tid: If non-NULL, the ID of the new thread is stored here.  NB
+ * this is defined as a DWORD (ie 32bit) in the MS API, but we need to
+ * cope with 64 bit IDs for s390x and amd64.
  *
  * Creates a new threading handle.
  *
@@ -201,15 +190,15 @@ static void thread_hash_init(void)
  */
 gpointer CreateThread(WapiSecurityAttributes *security G_GNUC_UNUSED, guint32 stacksize,
                      WapiThreadStart start, gpointer param, guint32 create,
-                     guint32 *tid) 
+                     gsize *tid) 
 {
-       struct _WapiHandleShared shared_handle;
-       struct _WapiHandle_thread thread_handle = {0};
+       struct _WapiHandle_thread thread_handle = {0}, *thread_handle_p;
        pthread_attr_t attr;
        gpointer handle;
        gboolean ok;
        int ret;
        int thr_ret;
+       int i, unrefs = 0;
        gpointer ct_ret = NULL;
        
        mono_once (&thread_hash_once, thread_hash_init);
@@ -220,7 +209,44 @@ gpointer CreateThread(WapiSecurityAttributes *security G_GNUC_UNUSED, guint32 st
        }
 
        thread_handle.state = THREAD_STATE_START;
-       thread_handle.owner_pid = getpid();
+       thread_handle.owner_pid = getpid ();
+       thread_handle.owned_mutexes = g_ptr_array_new ();
+       
+       handle = _wapi_handle_new (WAPI_HANDLE_THREAD, &thread_handle);
+       if (handle == _WAPI_HANDLE_INVALID) {
+               g_warning ("%s: error creating thread handle", __func__);
+               SetLastError (ERROR_GEN_FAILURE);
+               
+               return (NULL);
+       }
+
+       pthread_cleanup_push ((void(*)(void *))_wapi_handle_unlock_handle,
+                             handle);
+       thr_ret = _wapi_handle_lock_handle (handle);
+       g_assert (thr_ret == 0);
+       
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle_p);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               SetLastError (ERROR_GEN_FAILURE);
+               
+               goto cleanup;
+       }
+
+       /* Hold a reference while the thread is active, because we use
+        * the handle to store thread exit information
+        */
+       _wapi_handle_ref (handle);
+       
+       /* Lock around the thread create, so that the new thread cant
+        * race us to look up the thread handle in GetCurrentThread()
+        */
+       pthread_cleanup_push ((void(*)(void *))mono_mutex_unlock_in_cleanup,
+                             (void *)&thread_hash_mutex);
+       thr_ret = mono_mutex_lock(&thread_hash_mutex);
+       g_assert (thr_ret == 0);
        
        /* Set a 2M stack size.  This is the default on Linux, but BSD
         * needs it.  (The original bug report from Martin Dvorak <md@9ll.cz>
@@ -231,40 +257,28 @@ gpointer CreateThread(WapiSecurityAttributes *security G_GNUC_UNUSED, guint32 st
        g_assert (thr_ret == 0);
        
        /* defaults of 2Mb for 32bits and 4Mb for 64bits */
+       /* temporarily changed to use 1 MB: this allows more threads
+        * to be used, as well as using less virtual memory and so
+        * more is available for the GC heap.
+        */
        if (stacksize == 0){
 #if HAVE_VALGRIND_MEMCHECK_H
                if (RUNNING_ON_VALGRIND) {
                        stacksize = 1 << 20;
                } else {
-                       stacksize = (SIZEOF_VOID_P / 2) * 1024 * 1024;
+                       stacksize = (SIZEOF_VOID_P / 4) * 1024 * 1024;
                }
 #else
-               stacksize = (SIZEOF_VOID_P / 2) * 1024 * 1024;
+               stacksize = (SIZEOF_VOID_P / 4) * 1024 * 1024;
 #endif
-               
        }
 
 #ifdef HAVE_PTHREAD_ATTR_SETSTACKSIZE
        thr_ret = pthread_attr_setstacksize(&attr, stacksize);
        g_assert (thr_ret == 0);
 #endif
-       
-       /* Lock around the thread create, so that the new thread cant
-        * race us to look up the thread handle in GetCurrentThread()
-        */
-       pthread_cleanup_push ((void(*)(void *))mono_mutex_unlock_in_cleanup,
-                             (void *)&thread_hash_mutex);
-       thr_ret = mono_mutex_lock(&thread_hash_mutex);
-       g_assert (thr_ret == 0);
-       
-       handle = _wapi_handle_new (WAPI_HANDLE_THREAD, &thread_handle);
-       if (handle == _WAPI_HANDLE_INVALID) {
-               g_warning ("%s: error creating thread handle", __func__);
-               goto thread_hash_cleanup;
-       }
-       ct_ret = handle;
 
-       ret = _wapi_timed_thread_create (&thread_handle.thread, &attr,
+       ret = _wapi_timed_thread_create (&thread_handle_p->thread, &attr,
                                         create, start, thread_exit, param,
                                         handle);
        if (ret != 0) {
@@ -272,41 +286,33 @@ gpointer CreateThread(WapiSecurityAttributes *security G_GNUC_UNUSED, guint32 st
                g_message ("%s: Thread create error: %s", __func__,
                           strerror(ret));
 #endif
-               _wapi_handle_unref (handle);
-               goto thread_hash_cleanup;
-       }
 
-       /* Need to update the handle with the state info that
-        * _wapi_timed_thread_create created, after the initial handle
-        * info was copied into shared memory
-        */
-       ok = _wapi_copy_handle (handle, WAPI_HANDLE_THREAD, &shared_handle);
-       if (ok == FALSE) {
-               g_warning ("%s: error copying thread handle %p", __func__,
-                          handle);
-               _wapi_handle_unref (handle);
+               /* Two, because of the reference we took above */
+               unrefs = 2;
+               
                goto thread_hash_cleanup;
        }
-       shared_handle.u.thread.thread = thread_handle.thread;
-       _wapi_replace_handle (handle, WAPI_HANDLE_THREAD, &shared_handle);
-       
-       /* Hold a reference while the thread is active, because we use
-        * the handle to store thread exit information
-        */
-       _wapi_handle_ref (handle);
+       ct_ret = handle;
        
-       g_hash_table_insert (thread_hash, &thread_handle.thread->id, handle);
+       g_hash_table_insert (thread_hash,
+                            (gpointer)(thread_handle_p->thread->id),
+                            handle);
        
 #ifdef DEBUG
        g_message("%s: Started thread handle %p thread %p ID %ld", __func__,
-                 handle, thread_handle.thread, thread_handle.thread->id);
+                 handle, thread_handle_p->thread,
+                 thread_handle_p->thread->id);
 #endif
        
        if (tid != NULL) {
 #ifdef PTHREAD_POINTER_ID
-               *tid = GPOINTER_TO_UINT (thread_handle.thread->id);
+               /* Don't use GPOINTER_TO_UINT here, it can't cope with
+                * sizeof(void *) > sizeof(uint) when a cast to uint
+                * would overflow
+                */
+               *tid = (gsize)(thread_handle_p->thread->id);
 #else
-               *tid = thread_handle.thread->id;
+               *tid = thread_handle_p->thread->id;
 #endif
        }
 
@@ -314,33 +320,56 @@ thread_hash_cleanup:
        thr_ret = mono_mutex_unlock (&thread_hash_mutex);
        g_assert (thr_ret == 0);
        pthread_cleanup_pop (0);
+
+cleanup:
+       thr_ret = _wapi_handle_unlock_handle (handle);
+       g_assert (thr_ret == 0);
+       pthread_cleanup_pop (0);
+       
+       /* Must not call _wapi_handle_unref() with the handle already
+        * locked
+        */
+       for (i = 0; i < unrefs; i++) {
+               _wapi_handle_unref (handle);
+       }
        
        return(ct_ret);
 }
 
-gpointer OpenThread (guint32 access G_GNUC_UNUSED, gboolean inherit G_GNUC_UNUSED, guint32 tid)
+gpointer _wapi_thread_handle_from_id (pthread_t tid)
 {
        gpointer ret=NULL;
        int thr_ret;
-       
-       mono_once(&thread_hash_once, thread_hash_init);
-       mono_once (&thread_ops_once, thread_ops_init);
-       
-#ifdef DEBUG
-       g_message ("%s: looking up thread %d", __func__, tid);
-#endif
 
        pthread_cleanup_push ((void(*)(void *))mono_mutex_unlock_in_cleanup,
                              (void *)&thread_hash_mutex);
        thr_ret = mono_mutex_lock(&thread_hash_mutex);
        g_assert (thr_ret == 0);
        
-       ret=g_hash_table_lookup(thread_hash, &tid);
+       ret = g_hash_table_lookup (thread_hash, (gpointer)(tid));
 
        thr_ret = mono_mutex_unlock(&thread_hash_mutex);
        g_assert (thr_ret == 0);
        pthread_cleanup_pop (0);
+
+       return(ret);
+}
+
+/* NB tid is 32bit in MS API, but we need 64bit on amd64 and s390x
+ * (and probably others)
+ */
+gpointer OpenThread (guint32 access G_GNUC_UNUSED, gboolean inherit G_GNUC_UNUSED, gsize tid)
+{
+       gpointer ret=NULL;
+       
+       mono_once (&thread_hash_once, thread_hash_init);
+       mono_once (&thread_ops_once, thread_ops_init);
        
+#ifdef DEBUG
+       g_message ("%s: looking up thread %"G_GSIZE_FORMAT, __func__, tid);
+#endif
+
+       ret = _wapi_thread_handle_from_id ((pthread_t)tid);
        if(ret!=NULL) {
                _wapi_handle_ref (ret);
        }
@@ -378,10 +407,20 @@ void ExitThread(guint32 exitcode)
  */
 gboolean GetExitCodeThread(gpointer handle, guint32 *exitcode)
 {
+       struct _WapiHandle_thread *thread_handle;
+       gboolean ok;
+       
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               return (FALSE);
+       }
+       
 #ifdef DEBUG
        g_message ("%s: Finding exit status for thread handle %p id %ld",
-                  __func__, handle,
-                  WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).thread->id);
+                  __func__, handle, thread_handle->thread->id);
 #endif
 
        if (exitcode == NULL) {
@@ -391,18 +430,17 @@ gboolean GetExitCodeThread(gpointer handle, guint32 *exitcode)
                return(FALSE);
        }
        
-       if (WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).state != THREAD_STATE_EXITED) {
+       if (thread_handle->state != THREAD_STATE_EXITED) {
 #ifdef DEBUG
                g_message ("%s: Thread still active (state %d, exited is %d)",
-                          __func__,
-                          WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).state,
+                          __func__, thread_handle->state,
                           THREAD_STATE_EXITED);
 #endif
                *exitcode = STILL_ACTIVE;
                return(TRUE);
        }
        
-       *exitcode = WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).exitstatus;
+       *exitcode = thread_handle->exitstatus;
        
        return(TRUE);
 }
@@ -413,35 +451,71 @@ gboolean GetExitCodeThread(gpointer handle, guint32 *exitcode)
  * Looks up the thread ID of the current thread.  This ID can be
  * passed to OpenThread() to create a new handle on this thread.
  *
- * Return value: the thread ID.
+ * Return value: the thread ID.  NB this is defined as DWORD (ie 32
+ * bit) in the MS API, but we need to cope with 64 bit IDs for s390x
+ * and amd64.  This doesn't really break the API, it just embraces and
+ * extends it on 64bit platforms :)
  */
-guint32 GetCurrentThreadId(void)
+gsize GetCurrentThreadId(void)
 {
        pthread_t tid = pthread_self();
        
 #ifdef PTHREAD_POINTER_ID
-       return(GPOINTER_TO_UINT(tid));
+       /* Don't use GPOINTER_TO_UINT here, it can't cope with
+        * sizeof(void *) > sizeof(uint) when a cast to uint would
+        * overflow
+        */
+       return((gsize)tid);
 #else
        return(tid);
 #endif
 }
 
-static gpointer thread_attach(guint32 *tid)
+static gpointer thread_attach(gsize *tid)
 {
-       struct _WapiHandleShared shared_handle;
-       struct _WapiHandle_thread thread_handle = {0};
+       struct _WapiHandle_thread thread_handle = {0}, *thread_handle_p;
        gpointer handle;
        gboolean ok;
        int ret;
        int thr_ret;
+       int i, unrefs = 0;
        gpointer ta_ret = NULL;
        
        mono_once (&thread_hash_once, thread_hash_init);
        mono_once (&thread_ops_once, thread_ops_init);
 
        thread_handle.state = THREAD_STATE_START;
-       thread_handle.owner_pid = getpid();
+       thread_handle.owner_pid = getpid ();
+       thread_handle.owned_mutexes = g_ptr_array_new ();
 
+       handle = _wapi_handle_new (WAPI_HANDLE_THREAD, &thread_handle);
+       if (handle == _WAPI_HANDLE_INVALID) {
+               g_warning ("%s: error creating thread handle", __func__);
+               
+               SetLastError (ERROR_GEN_FAILURE);
+               return (NULL);
+       }
+
+       pthread_cleanup_push ((void(*)(void *))_wapi_handle_unlock_handle,
+                             handle);
+       thr_ret = _wapi_handle_lock_handle (handle);
+       g_assert (thr_ret == 0);
+       
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle_p);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               
+               SetLastError (ERROR_GEN_FAILURE);
+               goto cleanup;
+       }
+
+       /* Hold a reference while the thread is active, because we use
+        * the handle to store thread exit information
+        */
+       _wapi_handle_ref (handle);
+       
        /* Lock around the thread create, so that the new thread cant
         * race us to look up the thread handle in GetCurrentThread()
         */
@@ -450,14 +524,7 @@ static gpointer thread_attach(guint32 *tid)
        thr_ret = mono_mutex_lock(&thread_hash_mutex);
        g_assert (thr_ret == 0);
 
-       handle = _wapi_handle_new (WAPI_HANDLE_THREAD, &thread_handle);
-       if (handle == _WAPI_HANDLE_INVALID) {
-               g_warning ("%s: error creating thread handle", __func__);
-               goto thread_hash_cleanup;
-       }
-       ta_ret = handle;
-
-       ret = _wapi_timed_thread_attach (&thread_handle.thread, thread_exit,
+       ret = _wapi_timed_thread_attach (&thread_handle_p->thread, thread_exit,
                                         handle);
        if (ret != 0) {
 #ifdef DEBUG
@@ -465,41 +532,32 @@ static gpointer thread_attach(guint32 *tid)
                           strerror(ret));
 #endif
 
-               _wapi_handle_unref (handle);
-               goto thread_hash_cleanup;
-       }
-
-       /* Need to update the handle with the state info that
-        * _wapi_timed_thread_create created, after the initial handle
-        * info was copied into shared memory
-        */
-       ok = _wapi_copy_handle (handle, WAPI_HANDLE_THREAD, &shared_handle);
-       if (ok == FALSE) {
-               g_warning ("%s: error copying thread handle %p", __func__,
-                          handle);
-               _wapi_handle_unref (handle);
+               /* Two, because of the reference we took above */
+               unrefs = 2;
+               
                goto thread_hash_cleanup;
        }
-       shared_handle.u.thread.thread = thread_handle.thread;
-       _wapi_replace_handle (handle, WAPI_HANDLE_THREAD, &shared_handle);
-
-       /* Hold a reference while the thread is active, because we use
-        * the handle to store thread exit information
-        */
-       _wapi_handle_ref (handle);
+       ta_ret = handle;
        
-       g_hash_table_insert (thread_hash, &thread_handle.thread->id, handle);
+       g_hash_table_insert (thread_hash,
+                            (gpointer)(thread_handle_p->thread->id),
+                            handle);
 
 #ifdef DEBUG
        g_message("%s: Attached thread handle %p thread %p ID %ld", __func__,
-                 handle, thread_handle.thread, thread_handle.thread->id);
+                 handle, thread_handle_p->thread,
+                 thread_handle_p->thread->id);
 #endif
 
        if (tid != NULL) {
 #ifdef PTHREAD_POINTER_ID
-               *tid = GPOINTER_TO_UINT(thread_handle.thread->id);
+               /* Don't use GPOINTER_TO_UINT here, it can't cope with
+                * sizeof(void *) > sizeof(uint) when a cast to uint
+                * would overflow
+                */
+               *tid = (gsize)(thread_handle_p->thread->id);
 #else
-               *tid = thread_handle.thread->id;
+               *tid = thread_handle_p->thread->id;
 #endif
        }
 
@@ -507,6 +565,18 @@ thread_hash_cleanup:
        thr_ret = mono_mutex_unlock (&thread_hash_mutex);
        g_assert (thr_ret == 0);
        pthread_cleanup_pop (0);
+
+cleanup:
+       thr_ret = _wapi_handle_unlock_handle (handle);
+       g_assert (thr_ret == 0);
+       pthread_cleanup_pop (0);
+       
+       /* Must not call _wapi_handle_unref() with the handle already
+        * locked
+        */
+       for (i = 0; i < unrefs; i++) {
+               _wapi_handle_unref (handle);
+       }
        
        return(ta_ret);
 }
@@ -525,25 +595,14 @@ thread_hash_cleanup:
 gpointer GetCurrentThread(void)
 {
        gpointer ret=NULL;
-       guint32 tid;
-       int thr_ret;
+       gsize tid;
        
        mono_once(&thread_hash_once, thread_hash_init);
        mono_once (&thread_ops_once, thread_ops_init);
        
-       tid=GetCurrentThreadId();
-       
-       pthread_cleanup_push ((void(*)(void *))mono_mutex_unlock_in_cleanup,
-                             (void *)&thread_hash_mutex);
-       thr_ret = mono_mutex_lock(&thread_hash_mutex);
-       g_assert (thr_ret == 0);
-
-       ret=g_hash_table_lookup(thread_hash, &tid);
-
-       thr_ret = mono_mutex_unlock(&thread_hash_mutex);
-       g_assert (thr_ret == 0);
-       pthread_cleanup_pop (0);
+       tid = GetCurrentThreadId();
        
+       ret = _wapi_thread_handle_from_id ((pthread_t)tid);
        if (!ret) {
                ret = thread_attach (NULL);
        }
@@ -562,21 +621,33 @@ gpointer GetCurrentThread(void)
  */
 guint32 ResumeThread(gpointer handle)
 {
-       if (WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).thread == NULL) {
+       struct _WapiHandle_thread *thread_handle;
+       gboolean ok;
+       
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               
+               return (0xFFFFFFFF);
+       }
+       
+       if (thread_handle->thread == NULL) {
                return(0xFFFFFFFF);
        }
 
 #ifdef WITH_INCLUDED_LIBGC
-       if (WAPI_SHARED_HANDLE_DATA(handle, thread).thread->suspend_count <= 1)
-               _wapi_timed_thread_resume (WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).thread);
+       if (thread_handle->thread->suspend_count <= 1)
+               _wapi_timed_thread_resume (thread_handle->thread);
        
-       return (--(WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).thread->suspend_count));
+       return (--thread_handle->thread->suspend_count));
 #else
        /* This is still a kludge that only copes with starting a
         * thread that was suspended on create, so don't bother with
         * the suspend count crap yet
         */
-       _wapi_timed_thread_resume (WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).thread);
+       _wapi_timed_thread_resume (thread_handle->thread);
        return(0xFFFFFFFF);
 #endif
 }
@@ -593,20 +664,29 @@ guint32 ResumeThread(gpointer handle)
 guint32 SuspendThread(gpointer handle)
 {
 #ifdef WITH_INCLUDED_LIBGC
+       struct _WapiHandle_thread *thread_handle;
        gpointer current;
+       gboolean ok;
 
        current = GetCurrentThread ();
-
-       if (WAPI_SHARED_HANDLE_DATA(handle, thread).thread == NULL) {
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               return (0xFFFFFFFF);
+       }
+       
+       if (thread_handle->thread == NULL) {
                return(0xFFFFFFFF);
        }
 
-       if (!WAPI_SHARED_HANDLE_DATA(handle, thread).thread->suspend_count) {
+       if (!thread_handle->thread->suspend_count) {
                if (handle == current)
-                       _wapi_timed_thread_suspend (WAPI_SHARED_HANDLE_DATA(handle, thread).thread);
+                       _wapi_timed_thread_suspend (thread_handle->thread);
                else {
-                       pthread_kill (WAPI_SHARED_HANDLE_DATA(handle, thread).thread->id, SIGPWR);
-                       while (MONO_SEM_WAIT (&WAPI_SHARED_HANDLE_DATA(handle, thread).thread->suspended_sem) != 0) {
+                       pthread_kill (thread_handle->thread->id, SIGPWR);
+                       while (MONO_SEM_WAIT (&thread_handle->thread->suspended_sem) != 0) {
                                if (errno != EINTR) {
                                        return(0xFFFFFFFF);
                                }
@@ -614,7 +694,7 @@ guint32 SuspendThread(gpointer handle)
                }
        }
 
-       return (WAPI_SHARED_HANDLE_DATA(handle, thread).thread->suspend_count++);
+       return (thread_handle->thread->suspend_count++);
 #else
        return(0xFFFFFFFF);
 #endif
@@ -714,10 +794,6 @@ gboolean TlsFree(guint32 idx)
        thr_ret = pthread_key_delete(TLS_keys[idx]);
        g_assert (thr_ret == 0);
        
-#if HAVE_BOEHM_GC
-       mono_g_hash_table_remove (tls_gc_hash, MAKE_GC_ID (idx));
-#endif
-
        MONO_SPIN_UNLOCK (TLS_spinlock);
        
        return(TRUE);
@@ -791,14 +867,6 @@ gboolean TlsSetValue(guint32 idx, gpointer value)
                return(FALSE);
        }
        
-#if HAVE_BOEHM_GC
-       if (!tls_gc_hash) {
-               MONO_GC_REGISTER_ROOT (tls_gc_hash);
-               tls_gc_hash = mono_g_hash_table_new(g_direct_hash, g_direct_equal);
-       }
-       mono_g_hash_table_insert (tls_gc_hash, MAKE_GC_ID (idx), value);
-#endif
-
        MONO_SPIN_UNLOCK (TLS_spinlock);
        
        return(TRUE);
@@ -871,25 +939,22 @@ void Sleep(guint32 ms)
        SleepEx(ms, FALSE);
 }
 
-gboolean
-BindIoCompletionCallback (gpointer handle,
-                         WapiOverlappedCB callback,
-                         guint64 flags)
-{
-       WapiHandleType type;
-       
-       type = _wapi_handle_type (handle);
-       if (type == WAPI_HANDLE_FILE || type == WAPI_HANDLE_PIPE)
-               return _wapi_io_add_callback (handle, callback, flags);
-
-       SetLastError (ERROR_NOT_SUPPORTED);
-       return FALSE;
-}
-
 guint32 QueueUserAPC (WapiApcProc apc_callback, gpointer handle, 
                                        gpointer param)
 {
-       _wapi_timed_thread_queue_apc (WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).thread, apc_callback, param);
+       struct _WapiHandle_thread *thread_handle;
+       gboolean ok;
+       
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               return (0);
+       }
+       
+       _wapi_timed_thread_queue_apc (thread_handle->thread, apc_callback,
+                                     param);
        return(1);
 }
 
@@ -900,31 +965,108 @@ gboolean _wapi_thread_cur_apc_pending (void)
 
 gboolean _wapi_thread_apc_pending (gpointer handle)
 {
-       return(_wapi_timed_thread_apc_pending (WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).thread));
+       struct _WapiHandle_thread *thread_handle;
+       gboolean ok;
+       
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               return (FALSE);
+       }
+       
+       return(_wapi_timed_thread_apc_pending (thread_handle->thread));
 }
 
 gboolean _wapi_thread_dispatch_apc_queue (gpointer handle)
 {
-       _wapi_timed_thread_dispatch_apc_queue (WAPI_SHARED_HANDLE_TYPED_DATA(handle, thread).thread);
+       struct _WapiHandle_thread *thread_handle;
+       gboolean ok;
+       
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               return (0);
+       }
+       
+       _wapi_timed_thread_dispatch_apc_queue (thread_handle->thread);
        return(1);
 }
 
+void _wapi_thread_own_mutex (pthread_t tid, gpointer mutex)
+{
+       struct _WapiHandle_thread *thread_handle;
+       gboolean ok;
+       gpointer thread;
+
+       thread = _wapi_thread_handle_from_id (tid);
+       if (thread == NULL) {
+               g_warning ("%s: error looking up thread by ID", __func__);
+               return;
+       }
+
+       ok = _wapi_lookup_handle (thread, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          thread);
+               return;
+       }
+
+       g_ptr_array_add (thread_handle->owned_mutexes, mutex);
+}
+
+void _wapi_thread_disown_mutex (pthread_t tid, gpointer mutex)
+{
+       struct _WapiHandle_thread *thread_handle;
+       gboolean ok;
+       gpointer thread;
+
+       thread = _wapi_thread_handle_from_id (tid);
+       if (thread == NULL) {
+               g_warning ("%s: error looking up thread by ID", __func__);
+               return;
+       }
+
+       ok = _wapi_lookup_handle (thread, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          thread);
+               return;
+       }
+
+       g_ptr_array_remove (thread_handle->owned_mutexes, mutex);
+}
+
 
 
 #ifdef WITH_INCLUDED_LIBGC
 
 static void GC_suspend_handler (int sig)
 {
+       struct _WapiHandle_thread *thread_handle;
        gpointer handle;
+       gboolean ok;
 
        handle = GetCurrentThread ();
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               return;
+       }
+       
+       thread_handle->thread->stack_ptr = &ok;
+       MONO_SEM_POST (&thread_handle->thread->suspended_sem);
 
-       WAPI_SHARED_HANDLE_DATA(handle, thread).thread->stack_ptr = &ok;
-       MONO_SEM_POST (&WAPI_SHARED_HANDLE_DATA(handle, thread).thread->suspended_sem);
-
-       _wapi_timed_thread_suspend (WAPI_SHARED_HANDLE_DATA(handle, thread).thread);
+       _wapi_timed_thread_suspend (thread_handle->thread);
 
-       WAPI_SHARED_HANDLE_DATA(handle, thread).thread->stack_ptr = NULL;
+       thread_handle->thread->stack_ptr = NULL;
 }
 
 static void gc_init (void)
@@ -937,7 +1079,18 @@ static void gc_init (void)
 
 void mono_wapi_push_thread_stack (gpointer handle, gpointer stack_ptr)
 {
-       GC_push_all_stack (WAPI_SHARED_HANDLE_DATA(handle, thread).thread->stack_ptr, stack_ptr);
+       struct _WapiHandle_thread *thread_handle;
+       gboolean ok;
+       
+       ok = _wapi_lookup_handle (handle, WAPI_HANDLE_THREAD,
+                                 (gpointer *)&thread_handle);
+       if (ok == FALSE) {
+               g_warning ("%s: error looking up thread handle %p", __func__,
+                          handle);
+               return;
+       }
+       
+       GC_push_all_stack (thread_handle->thread->stack_ptr, stack_ptr);
 }
 
 #endif /* WITH_INCLUDED_LIBGC */