Fix thread destruction

Releasing a thread reference uses a mutex, which means the kernel
resources used for synchronization must be available until late during
the destruction process. For simplicity, merge __pthread_thread_dealloc
into __pthread_thread_terminate.

* Makefile (libpthread-routines): Remove pt-thread-dealloc.
* pthread/pt-create.c (__pthread_create_internal): Remove call to
__pthread_thread_dealloc.
* pthread/pt-exit.c (__pthread_exit): Likewise.
* pthread/pt-internal.h (__pthread_thread_dealloc): Remove declaration.
(__pthread_thread_terminate): Update description.
* sysdeps/mach/pt-thread-dealloc.c: Remove file.
* sysdeps/mach/pt-thread-terminate.c (__pthread_thread_terminate):
Destroy the wake up port.

2 files changed, 6 insertions, 41 deletions

diff --git a/sysdeps/mach/pt-thread-dealloc.c b/sysdeps/mach/pt-thread-dealloc.c
deleted file mode 100644
index e977c3f..0000000
--- a/sysdeps/mach/pt-thread-dealloc.c
+++ /dev/null
@@ -1,40 +0,0 @@
-/* Deallocate the kernel thread resources.  Mach version.
-   Copyright (C) 2005 Free Software Foundation, Inc.
-   This file is part of the GNU C Library.
-
-   The GNU C Library is free software; you can redistribute it and/or
-   modify it under the terms of the GNU Library General Public License as
-   published by the Free Software Foundation; either version 2 of the
-   License, or (at your option) any later version.
-
-   The GNU C Library is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-   Library General Public License for more details.
-
-   You should have received a copy of the GNU Library General Public
-   License along with the GNU C Library; see the file COPYING.LIB.  If not,
-   write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-   Boston, MA 02111-1307, USA.  */
-
-#include <assert.h>
-#include <errno.h>
-#include <mach.h>
-
-#include <pt-internal.h>
-
-/* Deallocate any kernel resources associated with THREAD except don't
-   terminate the thread itself.  */
-void
-__pthread_thread_dealloc (struct __pthread *thread)
-{
-  /* Why no assert?  Easy.  When Mach kills a task, it starts by
-     invalidating the task port and then terminating the threads one
-     by one.  But while it is terminating them, they are still
-     eligible to be scheduled.  Imagine we have two threads, one calls
-     exit, one calls pthread_exit.  The second one may run this after
-     the mask port can been destroyed thus gratuitously triggering the
-     assert.  */
-  __mach_port_destroy (__mach_task_self (),
-		       thread->wakeupmsg.msgh_remote_port);
-}
diff --git a/sysdeps/mach/pt-thread-terminate.c b/sysdeps/mach/pt-thread-terminate.c
index b9d08fc..6672065 100644
--- a/sysdeps/mach/pt-thread-terminate.c
+++ b/sysdeps/mach/pt-thread-terminate.c
@@ -32,7 +32,7 @@ void
 __pthread_thread_terminate (struct __pthread *thread)
 {
   thread_t kernel_thread, self_ktid;
-  mach_port_t reply_port;
+  mach_port_t wakeup_port, reply_port;
   void *stackaddr;
   size_t stacksize;
   error_t err;
@@ -50,6 +50,8 @@ __pthread_thread_terminate (struct __pthread *thread)
       stacksize = 0;
     }
 
+  wakeup_port = thread->wakeupmsg.msgh_remote_port;
+
   /* Each thread has its own reply port, allocated from MiG stub code calling
      __mig_get_reply_port.  Destroying it is a bit tricky because the calls
      involved are also RPCs, causing the creation of a new reply port if
@@ -64,6 +66,9 @@ __pthread_thread_terminate (struct __pthread *thread)
   /* Finally done with the thread structure.  */
   __pthread_dealloc (thread);
 
+  /* The wake up port is now no longer needed.  */
+  __mach_port_destroy (__mach_task_self (), wakeup_port);
+
   /* Terminate and release all that's left.  */
   err = __thread_terminate_release (kernel_thread, mach_task_self (),
 				    kernel_thread, reply_port,