Merge pull request #900 from Blewzman/FixAggregateExceptionGetBaseException
[mono.git] / mono / io-layer / wthreads.c
index e1c062874ff0d37d2f7345605ceda2ddcde74a05..0c34ba189ff8daf59b5132ae466e1913ed68ff84 100644 (file)
@@ -29,7 +29,6 @@
 #include <mono/io-layer/mutex-private.h>
 
 #include <mono/utils/mono-threads.h>
-#include <mono/utils/gc_wrapper.h>
 #include <mono/utils/atomic.h>
 #include <mono/utils/mono-mutex.h>
 
 #define WAIT_DEBUG(code) do { } while (0)
 #endif
 
-/* Hash threads with tids. I thought of using TLS for this, but that
- * would have to set the data in the new thread, which is more hassle
- */
-static mono_once_t thread_hash_once = MONO_ONCE_INIT;
-static pthread_key_t thread_hash_key;
-
-/* This key is used with attached threads and a destructor to signal
- * when attached threads exit, as they don't have the thread_exit()
- * infrastructure
- */
-static pthread_key_t thread_attached_key;
-
 struct _WapiHandleOps _wapi_thread_ops = {
        NULL,                           /* close */
        NULL,                           /* signal */
@@ -80,62 +67,27 @@ static void thread_ops_init (void)
 
 void _wapi_thread_cleanup (void)
 {
-       int ret;
-       
-       ret = pthread_key_delete (thread_hash_key);
-       g_assert (ret == 0);
-       
-       ret = pthread_key_delete (thread_attached_key);
-       g_assert (ret == 0);
 }
 
-/* Called by thread_exit(), but maybe indirectly by
- * mono_thread_manage() via mono_thread_signal_self() too
- */
-static void _wapi_thread_abandon_mutexes (gpointer handle)
+static gpointer
+get_current_thread_handle (void)
 {
-       struct _WapiHandle_thread *thread_handle;
-       gboolean ok;
-       int i;
-       pid_t pid = _wapi_getpid ();
-       pthread_t tid = pthread_self ();
-       
-       DEBUG ("%s: Thread %p abandoning held mutexes", __func__, handle);
+       MonoThreadInfo *info;
 
-       if (handle == NULL) {
-               handle = _wapi_thread_handle_from_id (pthread_self ());
-               if (handle == NULL) {
-                       /* Something gone badly wrong... */
-                       return;
-               }
-       }
-       
-       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;
-       }
-       
-       if (!pthread_equal (thread_handle->id, tid)) {
-               return;
-       }
-       
-       for (i = 0; i < thread_handle->owned_mutexes->len; i++) {
-               gpointer mutex = g_ptr_array_index (thread_handle->owned_mutexes, i);
-               
-               _wapi_mutex_abandon (mutex, pid, tid);
-               _wapi_thread_disown_mutex (mutex);
-       }
+       info = mono_thread_info_current ();
+       g_assert (info);
+       return info->handle;
 }
 
-void _wapi_thread_set_termination_details (gpointer handle,
+static void
+_wapi_thread_set_termination_details (gpointer handle,
                                           guint32 exitstatus)
 {
        struct _WapiHandle_thread *thread_handle;
        gboolean ok;
-       int thr_ret;
+       int i, thr_ret;
+       pid_t pid = _wapi_getpid ();
+       pthread_t tid = pthread_self ();
        
        if (_wapi_handle_issignalled (handle) ||
            _wapi_handle_type (handle) == WAPI_HANDLE_UNUSED) {
@@ -147,31 +99,27 @@ void _wapi_thread_set_termination_details (gpointer handle,
 
        DEBUG ("%s: Thread %p terminating", __func__, handle);
 
-       _wapi_thread_abandon_mutexes (handle);
-       
        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);
+       g_assert (ok);
 
-               return;
+       DEBUG ("%s: Thread %p abandoning held mutexes", __func__, handle);
+
+       for (i = 0; i < thread_handle->owned_mutexes->len; i++) {
+               gpointer mutex = g_ptr_array_index (thread_handle->owned_mutexes, i);
+
+               _wapi_mutex_abandon (mutex, pid, tid);
+               _wapi_thread_disown_mutex (mutex);
        }
+       g_ptr_array_free (thread_handle->owned_mutexes, TRUE);
        
-       pthread_cleanup_push ((void(*)(void *))_wapi_handle_unlock_handle,
-                             handle);
        thr_ret = _wapi_handle_lock_handle (handle);
        g_assert (thr_ret == 0);
-       
-       thread_handle->exitstatus = exitstatus;
-       thread_handle->state = THREAD_STATE_EXITED;
-       g_ptr_array_free (thread_handle->owned_mutexes, TRUE);
 
        _wapi_handle_set_signal_state (handle, TRUE, TRUE);
 
        thr_ret = _wapi_handle_unlock_handle (handle);
        g_assert (thr_ret == 0);
-       pthread_cleanup_pop (0);
        
        DEBUG("%s: Recording thread handle %p id %ld status as %d",
                  __func__, handle, thread_handle->id, exitstatus);
@@ -184,62 +132,19 @@ void _wapi_thread_signal_self (guint32 exitstatus)
 {
        gpointer handle;
        
-       handle = _wapi_thread_handle_from_id (pthread_self ());
-       if (handle == NULL) {
-               /* Something gone badly wrong... */
+       handle = get_current_thread_handle ();
+       if (handle == NULL)
                return;
-       }
        
        _wapi_thread_set_termination_details (handle, exitstatus);
 }
 
-/* Called by the thread creation code as a thread is finishing up, and
- * by ExitThread()
-*/
-static void thread_exit (guint32 exitstatus, gpointer handle) G_GNUC_NORETURN;
-#if defined(__native_client__)
-void nacl_shutdown_gc_thread(void);
-#endif
-static void thread_exit (guint32 exitstatus, gpointer handle)
-{
-#if defined(__native_client__)
-       nacl_shutdown_gc_thread();
-#endif
-       _wapi_thread_set_termination_details (handle, exitstatus);
-       
-       /* Call pthread_exit() to call destructors and really exit the
-        * thread
-        */
-       mono_gc_pthread_exit (NULL);
-}
-
 void
-wapi_thread_set_exit_code (guint32 exitstatus, gpointer handle)
+wapi_thread_handle_set_exited (gpointer handle, guint32 exitstatus)
 {
        _wapi_thread_set_termination_details (handle, exitstatus);
 }
 
-static void thread_attached_exit (gpointer handle)
-{
-       /* Drop the extra reference we take in thread_attach, now this
-        * thread is dead
-        */
-       
-       _wapi_thread_set_termination_details (handle, 0);
-}
-
-static void thread_hash_init(void)
-{
-       int thr_ret;
-       
-       thr_ret = pthread_key_create (&thread_hash_key, NULL);
-       g_assert (thr_ret == 0);
-
-       thr_ret = pthread_key_create (&thread_attached_key,
-                                     thread_attached_exit);
-       g_assert (thr_ret == 0);
-}
-
 /*
  * wapi_create_thread_handle:
  *
@@ -252,10 +157,8 @@ wapi_create_thread_handle (void)
        gpointer handle;
        int res;
 
-       mono_once (&thread_hash_once, thread_hash_init);
        mono_once (&thread_ops_once, thread_ops_init);
 
-       thread_handle.state = THREAD_STATE_START;
        thread_handle.owned_mutexes = g_ptr_array_new ();
 
        handle = _wapi_handle_new (WAPI_HANDLE_THREAD, &thread_handle);
@@ -270,12 +173,6 @@ wapi_create_thread_handle (void)
                                                           (gpointer *)&thread);
        g_assert (res);
 
-       thread->handle = handle;
-
-       res = pthread_setspecific (thread_hash_key, handle);
-       if (res)
-               mono_gc_pthread_exit (NULL);
-
        thread->id = pthread_self ();
 
        /*
@@ -289,6 +186,12 @@ wapi_create_thread_handle (void)
        return handle;
 }
 
+void
+wapi_ref_thread_handle (gpointer handle)
+{
+       _wapi_handle_ref (handle);
+}
+
 /* The only time this function is called when tid != pthread_self ()
  * is from OpenThread (), so we can fast-path most cases by just
  * looking up the handle in TLS.  OpenThread () must cope with a NULL
@@ -296,23 +199,10 @@ wapi_create_thread_handle (void)
  */
 gpointer _wapi_thread_handle_from_id (pthread_t tid)
 {
-       gpointer ret;
-
-       if (pthread_equal (tid, pthread_self ()) &&
-           (ret = pthread_getspecific (thread_hash_key)) != NULL) {
-               /* We know the handle */
-
-               DEBUG ("%s: Returning %p for self thread %ld from TLS",
-                          __func__, ret, tid);
-               
-               return(ret);
-       }
-       
-       DEBUG ("%s: Returning NULL for unknown or non-self thread %ld",
-                  __func__, tid);
-               
-
-       return(NULL);
+       if (pthread_equal (tid, pthread_self ()))
+               return get_current_thread_handle ();
+       else
+               return NULL;
 }
 
 static gboolean find_thread_by_id (gpointer handle, gpointer user_data)
@@ -355,12 +245,12 @@ gpointer OpenThread (guint32 access G_GNUC_UNUSED, gboolean inherit G_GNUC_UNUSE
 {
        gpointer ret=NULL;
        
-       mono_once (&thread_hash_once, thread_hash_init);
        mono_once (&thread_ops_once, thread_ops_init);
        
        DEBUG ("%s: looking up thread %"G_GSIZE_FORMAT, __func__, tid);
 
-       ret = _wapi_thread_handle_from_id ((pthread_t)tid);
+       if (pthread_equal ((pthread_t)tid, pthread_self ()))
+               ret = get_current_thread_handle ();
        if (ret == NULL) {
                /* We need to search for this thread */
                ret = _wapi_search_handle (WAPI_HANDLE_THREAD, find_thread_by_id, (gpointer)tid, NULL, FALSE/*TRUE*/);  /* FIXME: have a proper look at this, me might not need to set search_shared = TRUE */
@@ -376,71 +266,6 @@ gpointer OpenThread (guint32 access G_GNUC_UNUSED, gboolean inherit G_GNUC_UNUSE
        return(ret);
 }
 
-/**
- * ExitThread:
- * @exitcode: Sets the thread's exit code, which can be read from
- * another thread with GetExitCodeThread().
- *
- * Terminates the calling thread.  A thread can also exit by returning
- * from its start function. When the last thread in a process
- * terminates, the process itself terminates.
- */
-void ExitThread(guint32 exitcode)
-{
-       gpointer thread = _wapi_thread_handle_from_id (pthread_self ());
-       
-       if (thread != NULL) {
-               thread_exit(exitcode, thread);
-       } else {
-               /* Just blow this thread away */
-               mono_gc_pthread_exit (NULL);
-       }
-}
-
-/**
- * GetExitCodeThread:
- * @handle: The thread handle to query
- * @exitcode: The thread @handle exit code is stored here
- *
- * Finds the exit code of @handle, and stores it in @exitcode.  If the
- * thread @handle is still running, the value stored is %STILL_ACTIVE.
- *
- * Return value: %TRUE, or %FALSE on error.
- */
-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);
-       }
-       
-       DEBUG ("%s: Finding exit status for thread handle %p id %ld",
-                  __func__, handle, thread_handle->id);
-
-       if (exitcode == NULL) {
-               DEBUG ("%s: Nowhere to store exit code", __func__);
-               return(FALSE);
-       }
-       
-       if (thread_handle->state != THREAD_STATE_EXITED) {
-               DEBUG ("%s: Thread still active (state %d, exited is %d)",
-                          __func__, thread_handle->state,
-                          THREAD_STATE_EXITED);
-               *exitcode = STILL_ACTIVE;
-               return(TRUE);
-       }
-       
-       *exitcode = thread_handle->exitstatus;
-       
-       return(TRUE);
-}
-
 /**
  * GetCurrentThreadId:
  *
@@ -454,126 +279,16 @@ gboolean GetExitCodeThread(gpointer handle, guint32 *exitcode)
  */
 gsize GetCurrentThreadId(void)
 {
-       pthread_t tid = pthread_self();
-       
-#ifdef PTHREAD_POINTER_ID
-       /* 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(gsize *tid)
-{
-       struct _WapiHandle_thread thread_handle = {0}, *thread_handle_p;
-       gpointer handle;
-       gboolean ok;
-       int thr_ret;
-       
-       mono_once (&thread_hash_once, thread_hash_init);
-       mono_once (&thread_ops_once, thread_ops_init);
+       MonoNativeThreadId id;
 
-       thread_handle.state = THREAD_STATE_START;
-       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);
-
-       thread_handle_p->handle = handle;
-       thread_handle_p->id = pthread_self ();
-
-       thr_ret = pthread_setspecific (thread_hash_key, (void *)handle);
-       g_assert (thr_ret == 0);
-
-       thr_ret = pthread_setspecific (thread_attached_key, (void *)handle);
-       g_assert (thr_ret == 0);
-       
-       DEBUG("%s: Attached thread handle %p ID %ld", __func__, handle,
-                 thread_handle_p->id);
-
-       if (tid != NULL) {
-#ifdef PTHREAD_POINTER_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->id);
-#else
-               *tid = thread_handle_p->id;
-#endif
-       }
-
-cleanup:
-       thr_ret = _wapi_handle_unlock_handle (handle);
-       g_assert (thr_ret == 0);
-       pthread_cleanup_pop (0);
-       
-       return(handle);
+       id = mono_native_thread_id_get ();
+       return MONO_NATIVE_THREAD_ID_TO_UINT (id);
 }
 
 gpointer _wapi_thread_duplicate ()
 {
-       gpointer ret = NULL;
-       
-       mono_once (&thread_hash_once, thread_hash_init);
-       mono_once (&thread_ops_once, thread_ops_init);
-       
-       ret = _wapi_thread_handle_from_id (pthread_self ());
-       if (!ret) {
-               ret = thread_attach (NULL);
-       } else {
-               _wapi_handle_ref (ret);
-       }
-       
-       return(ret);
-}
-
-/**
- * GetCurrentThread:
- *
- * Looks up the handle associated with the current thread.  Under
- * Windows this is a pseudohandle, and must be duplicated with
- * DuplicateHandle() for some operations.
- *
- * Return value: The current thread handle, or %NULL on failure.
- * (Unknown whether Windows has a possible failure here.  It may be
- * necessary to implement the pseudohandle-constant behaviour).
- */
-gpointer GetCurrentThread(void)
-{
-       mono_once(&thread_hash_once, thread_hash_init);
-       mono_once (&thread_ops_once, thread_ops_init);
-       
-       return(_WAPI_THREAD_CURRENT);
+       g_assert_not_reached ();
+       return NULL;
 }
 
 /**
@@ -595,7 +310,7 @@ guint32 SleepEx(guint32 ms, gboolean alertable)
        DEBUG("%s: Sleeping for %d ms", __func__, ms);
 
        if (alertable) {
-               current_thread = _wapi_thread_handle_from_id (pthread_self ());
+               current_thread = get_current_thread_handle ();
                if (current_thread == NULL) {
                        SetLastError (ERROR_INVALID_HANDLE);
                        return(WAIT_FAILED);
@@ -649,7 +364,7 @@ void Sleep(guint32 ms)
 
 gboolean _wapi_thread_cur_apc_pending (void)
 {
-       gpointer thread = _wapi_thread_handle_from_id (pthread_self ());
+       gpointer thread = get_current_thread_handle ();
        
        if (thread == NULL) {
                SetLastError (ERROR_INVALID_HANDLE);
@@ -697,30 +412,32 @@ gboolean _wapi_thread_dispatch_apc_queue (gpointer handle)
 }
 
 /*
- * In this implementation, APC_CALLBACK is ignored.
- * if HANDLE refers to the current thread, the only effect this function has 
- * that if called from a signal handler, and the thread was waiting when receiving 
+ * wapi_interrupt_self:
+ *
+ * If this function called from a signal handler, and the thread was waiting when receiving
  * the signal, the wait will be broken after the signal handler returns.
- * In this case, this function is async-signal-safe.
+ * This function is async-signal-safe.
  */
-guint32 QueueUserAPC (WapiApcProc apc_callback, gpointer handle, 
-                     gpointer param)
+void
+wapi_thread_interrupt_self (void)
 {
+       HANDLE handle;
        struct _WapiHandle_thread *thread_handle;
        gboolean ok;
        
+       handle = get_current_thread_handle ();
+       g_assert (handle);
+
        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);
+               return;
        }
 
-       g_assert (thread_handle->id == (pthread_t)GetCurrentThreadId ());
        /* No locking/memory barriers are needed here */
        thread_handle->has_apc = TRUE;
-       return(1);
 }
 
 /*
@@ -739,61 +456,12 @@ guint32 QueueUserAPC (WapiApcProc apc_callback, gpointer handle,
  */
 void wapi_interrupt_thread (gpointer thread_handle)
 {
-       struct _WapiHandle_thread *thread;
-       gboolean ok;
-       gpointer prev_handle, wait_handle;
-       guint32 idx;
-       pthread_cond_t *cond;
-       mono_mutex_t *mutex;
-       
-       ok = _wapi_lookup_handle (thread_handle, WAPI_HANDLE_THREAD,
-                                 (gpointer *)&thread);
-       g_assert (ok);
+       gpointer wait_handle;
 
-       while (TRUE) {
-               wait_handle = thread->wait_handle;
-
-               /* 
-                * Atomically obtain the handle the thread is waiting on, and
-                * change it to a flag value.
-                */
-               prev_handle = InterlockedCompareExchangePointer (&thread->wait_handle,
-                                                                                                                INTERRUPTION_REQUESTED_HANDLE, wait_handle);
-               if (prev_handle == INTERRUPTION_REQUESTED_HANDLE)
-                       /* Already interrupted */
-                       return;
-               if (prev_handle == wait_handle)
-                       break;
-
-               /* Try again */
-       }
-
-       WAIT_DEBUG (printf ("%p: state -> INTERRUPTED.\n", thread->id););
-
-       if (!wait_handle)
-               /* Not waiting */
-               return;
-
-       /* If we reach here, then wait_handle is set to the flag value, 
-        * which means that the target thread is either
-        * - before the first CAS in timedwait, which means it won't enter the
-        * wait.
-        * - it is after the first CAS, so it is already waiting, or it will 
-        * enter the wait, and it will be interrupted by the broadcast.
-        */
-       idx = GPOINTER_TO_UINT(wait_handle);
-       cond = &_WAPI_PRIVATE_HANDLES(idx).signal_cond;
-       mutex = &_WAPI_PRIVATE_HANDLES(idx).signal_mutex;
-
-       mono_mutex_lock (mutex);
-       mono_cond_broadcast (cond);
-       mono_mutex_unlock (mutex);
-
-       /* ref added by set_wait_handle */
-       _wapi_handle_unref (wait_handle);
+       wait_handle = wapi_prepare_interrupt_thread (thread_handle);
+       wapi_finish_interrupt_thread (wait_handle);
 }
 
-
 gpointer wapi_prepare_interrupt_thread (gpointer thread_handle)
 {
        struct _WapiHandle_thread *thread;
@@ -856,7 +524,6 @@ void wapi_finish_interrupt_thread (gpointer wait_handle)
        _wapi_handle_unref (wait_handle);
 }
 
-
 /*
  * wapi_self_interrupt:
  *
@@ -865,42 +532,15 @@ void wapi_finish_interrupt_thread (gpointer wait_handle)
  */
 void wapi_self_interrupt (void)
 {
-       struct _WapiHandle_thread *thread;
-       gboolean ok;
-       gpointer prev_handle, wait_handle;
+       gpointer wait_handle;
        gpointer thread_handle;
 
-
        thread_handle = OpenThread (0, 0, GetCurrentThreadId ());
-       ok = _wapi_lookup_handle (thread_handle, WAPI_HANDLE_THREAD,
-                                                         (gpointer *)&thread);
-       g_assert (ok);
-
-       while (TRUE) {
-               wait_handle = thread->wait_handle;
-
-               /*
-                * Atomically obtain the handle the thread is waiting on, and
-                * change it to a flag value.
-                */
-               prev_handle = InterlockedCompareExchangePointer (&thread->wait_handle,
-                                                                                                                INTERRUPTION_REQUESTED_HANDLE, wait_handle);
-               if (prev_handle == INTERRUPTION_REQUESTED_HANDLE)
-                       /* Already interrupted */
-                       goto cleanup;
-               /*We did not get interrupted*/
-               if (prev_handle == wait_handle)
-                       break;
-
-               /* Try again */
-       }
-
-       if (wait_handle) {
+       wait_handle = wapi_prepare_interrupt_thread (thread_handle);
+       if (wait_handle)
                /* ref added by set_wait_handle */
                _wapi_handle_unref (wait_handle);
-       }
 
-cleanup:
        _wapi_handle_unref (thread_handle);
 }
 
@@ -1045,7 +685,7 @@ void _wapi_thread_own_mutex (gpointer mutex)
        gboolean ok;
        gpointer thread;
        
-       thread = _wapi_thread_handle_from_id (pthread_self ());
+       thread = get_current_thread_handle ();
        if (thread == NULL) {
                g_warning ("%s: error looking up thread by ID", __func__);
                return;
@@ -1070,7 +710,7 @@ void _wapi_thread_disown_mutex (gpointer mutex)
        gboolean ok;
        gpointer thread;
 
-       thread = _wapi_thread_handle_from_id (pthread_self ());
+       thread = get_current_thread_handle ();
        if (thread == NULL) {
                g_warning ("%s: error looking up thread by ID", __func__);
                return;