From: Taylor R Campbell <campbell@mumble.net>
Date: Wed, 29 May 2019 04:10:57 +0000 (+0000)
Subject: Process pending thread events when unblocking them.
X-Git-Tag: mit-scheme-pucked-10.1.11~6^2~20
X-Git-Url: https://birchwood-abbey.net/git?a=commitdiff_plain;h=b0b11d54bc0d0cf639ca7205fc8e3396e4fd6807;p=mit-scheme.git

Process pending thread events when unblocking them.

Partly fixes bug where profiler never got a chance to run when the
program was doing most of its work in short routines that block and
unblock thread events, like opening and closing files.

Only partly, because it seems something else unblocks thread events
without processing them -- until we do another block/unblock cycle as
in channel-close.  Verified that very little time is actually spent
in channel-close; haven't yet tracked down who the culprit is.
---

diff --git a/src/runtime/thread.scm b/src/runtime/thread.scm
index d030a2925..954d00d42 100644
--- a/src/runtime/thread.scm
+++ b/src/runtime/thread.scm
@@ -897,9 +897,7 @@ USA.
 			value))
 		    'with-thread-events-blocked
 		    block-events?)))
-	      (let ((thread first-running-thread))
-		(if thread
-		    (set-thread/block-events?! thread block-events?)))
+	      (%set-thread-event-block! block-events?)
 	      (set-interrupt-enables! interrupt-mask)
 	      value))
 	  (begin
@@ -917,10 +915,16 @@ USA.
 (define (set-thread-event-block! block?)
   (without-interrupts
    (lambda ()
-     (let ((thread first-running-thread))
-       (if thread
-	   (set-thread/block-events?! thread block?)))
-     unspecific)))
+     (%set-thread-event-block! block?))))
+
+(define (%set-thread-event-block! block?)
+  (let ((thread first-running-thread))
+    (if thread
+	(begin
+	  (if (not block?)
+	      (handle-thread-events thread))
+	  (set-thread/block-events?! thread block?))))
+  unspecific)
 
 (define (signal-thread-event thread event #!optional no-error?)
   (guarantee thread? thread 'signal-thread-event)