X-Git-Url: http://git.megacz.com/?a=blobdiff_plain;f=rts%2Fwin32%2FTicker.c;h=929e6f4086bef8f08fb49d4c32023c903cd3b7a9;hb=723365de1b9ab1b8a8cf59a936624891d075e554;hp=6be1b3a7c6163cc881db79b77f6f9755f3055da9;hpb=e4173b1b96807a80d3432d1e35f1409f8a3c96f7;p=ghc-hetmet.git

diff --git a/rts/win32/Ticker.c b/rts/win32/Ticker.c
index 6be1b3a..929e6f4 100644
--- a/rts/win32/Ticker.c
+++ b/rts/win32/Ticker.c
@@ -127,7 +127,7 @@ stopTicker(void)
 }
 
 void
-exitTicker(void)
+exitTicker (rtsBool wait)
 {
     // We must wait for the ticker thread to terminate, since if we
     // are in a DLL that is about to be unloaded, the ticker thread
@@ -138,10 +138,20 @@ exitTicker(void)
 	DWORD exitCode;
         ticker_state = TickerExit;
 	SetEvent(hStopEvent);
-	while (1) {
-	    WaitForSingleObject(tickThread, 20);
+	while (wait) {
+            // See #3748:
+            //
+            // when the RTS is compiled into a DLL (wait==rtsTrue),
+            // the ticker thread must stop before we exit, or chaos
+            // will ensue.  We can't kill it, because it may be
+            // holding a lock.
+            //
+            // When not compiled into a DLL, we wait for
+            // the thread out of courtesy, but give up after 200ms if
+            // it still hasn't stopped.
+	    WaitForSingleObject(tickThread, 200);
 	    if (!GetExitCodeThread(tickThread, &exitCode)) {
-		return 1;
+		return;
 	    }
 	    if (exitCode != STILL_ACTIVE) {
 		tickThread = INVALID_HANDLE_VALUE;
@@ -149,9 +159,8 @@ exitTicker(void)
 		    CloseHandle(hStopEvent);
 		    hStopEvent = INVALID_HANDLE_VALUE;
 		}
-		return 0;
+		return;
 	    }
-	    TerminateThread(tickThread, 0);
 	}
     }
 }