Reversion of commit by Davidxu plus fixes since applied.

I'm not convinced there is anything major wrong with the patch but
them's the rules..

I am using my "David's mentor" hat to revert this as he's
offline for a while.

1 files changed, 14 insertions, 4 deletions

diff --git a/sys/kern/kern_exit.c b/sys/kern/kern_exit.c
index ce9a18c..0b2c2e8 100644
--- a/sys/kern/kern_exit.c
+++ b/sys/kern/kern_exit.c
@@ -147,7 +147,7 @@ exit1(td, rv)
 	}
 
 	/*
-	 * XXXKSE: MUST abort all other threads before proceeding past here.
+	 * XXXXKSE: MUST abort all other threads before proceeding past here.
 	 */
 	PROC_LOCK(p);
 	if (p->p_flag & P_KSES) {
@@ -156,6 +156,17 @@ exit1(td, rv)
 		 * if so, act apropriatly, (exit or suspend);
 		 */
 		thread_suspend_check(0);
+		/*
+		 * Here is a trick..
+		 * We need to free up our KSE to process other threads
+		 * so that we can safely set the UNBOUND flag
+		 * (whether or not we have a mailbox) as we are NEVER
+		 * going to return to the user.
+		 * The flag will not be set yet if we are exiting
+		 * because of a signal, pagefault, or similar
+		 * (or even an exit(2) from the UTS).
+		 */
+		td->td_flags |= TDF_UNBOUND;
 
 		/*
 		 * Kill off the other threads. This requires
@@ -181,6 +192,7 @@ exit1(td, rv)
 		 * Turn off threading support.
 		 */
 		p->p_flag &= ~P_KSES;
+		td->td_flags &= ~TDF_UNBOUND;
 		thread_single_end(); 	/* Don't need this any more. */
 	}
 	/*
@@ -225,10 +237,8 @@ exit1(td, rv)
 	 */
 	TAILQ_FOREACH(ep, &exit_list, next) 
 		(*ep->function)(p);
-	
-	PROC_LOCK(p);
+
 	stopprofclock(p);
-	PROC_UNLOCK(p);
 
 	MALLOC(p->p_ru, struct rusage *, sizeof(struct rusage),
 		M_ZOMBIE, 0);