From: Simon Marlow <simonmar@microsoft.com>
Date: Tue, 12 Dec 2006 15:33:53 +0000 (+0000)
Subject: Don't free sched_mutex until freeTaskManager has finished with it
X-Git-Tag: 2006-12-16~15
X-Git-Url: http://git.megacz.com/?p=ghc-hetmet.git;a=commitdiff_plain;h=6ffb1cec5bc6fcc834547fc456322d2eb2418ff6

Don't free sched_mutex until freeTaskManager has finished with it
Also move closeMutex() etc. into freeTaskManager, this is a free-ish thing
---

diff --git a/rts/Schedule.c b/rts/Schedule.c
index 77dcf41..c61e623 100644
--- a/rts/Schedule.c
+++ b/rts/Schedule.c
@@ -2578,7 +2578,6 @@ exitScheduler( void )
 	boundTaskExiting(task);
 	stopTaskManager();
     }
-    closeMutex(&sched_mutex);
 #endif
 }
 
@@ -2586,6 +2585,9 @@ void
 freeScheduler( void )
 {
     freeTaskManager();
+#if defined(THREADED_RTS)
+    closeMutex(&sched_mutex);
+#endif
 }
 
 /* ---------------------------------------------------------------------------
diff --git a/rts/Task.c b/rts/Task.c
index 0e51dbf..551f137 100644
--- a/rts/Task.c
+++ b/rts/Task.c
@@ -68,20 +68,10 @@ initTaskManager (void)
 void
 stopTaskManager (void)
 {
-    Task *task;
-
     debugTrace(DEBUG_sched, 
 	       "stopping task manager, %d tasks still running",
 	       tasksRunning);
-
-    ACQUIRE_LOCK(&sched_mutex);
-    for (task = task_free_list; task != NULL; task = task->next) {
-#if defined(THREADED_RTS)
-        closeCondition(&task->cond);
-        closeMutex(&task->lock);
-#endif
-    }
-    RELEASE_LOCK(&sched_mutex);
+    /* nothing to do */
 }
 
 
@@ -94,6 +84,10 @@ freeTaskManager (void)
 
     ACQUIRE_LOCK(&sched_mutex);
     for (task = task_free_list; task != NULL; task = next) {
+#if defined(THREADED_RTS)
+        closeCondition(&task->cond);
+        closeMutex(&task->lock);
+#endif
         next = task->next;
         stgFree(task);
     }