summaryrefslogtreecommitdiff
path: root/includes/rts/IOManager.h
diff options
context:
space:
mode:
authorAndreas Voellmy <andreas.voellmy@gmail.com>2014-08-19 08:02:18 -0500
committerAustin Seipp <austin@well-typed.com>2014-08-19 08:03:34 -0500
commitf9f89b7884ccc8ee5047cf4fffdf2b36df6832df (patch)
tree2871a2b4bca35ed8f2cecdb71d72ce66885a2c54 /includes/rts/IOManager.h
parent51a0b60038fb1dddbb9793c0c069de6125f84bf1 (diff)
downloadhaskell-f9f89b7884ccc8ee5047cf4fffdf2b36df6832df.tar.gz
rts/base: Fix #9423
Summary: Fix #9423. The problem in #9423 is caused when code invoked by `hs_exit()` waits on all foreign calls to return, but some IO managers are in `safe` foreign calls and do not return. The previous design signaled to the timer manager (via its control pipe) that it should "die" and when the timer manager returned to Haskell-land, the Haskell code in timer manager then signalled to the IO manager threads that they should return from foreign calls and `die`. Unfortunately, in the shutdown sequence the timer manager is unable to return to Haskell-land fast enough and so the code that signals to the IO manager threads (via their control pipes) is never executed and the IO manager threads remain out in the foreign calls. This patch solves this problem by having the RTS signal to all the IO manager threads (via their control pipes; and in addition to signalling to the timer manager thread) that they should shutdown (in `ioManagerDie()` in `rts/Signals.c`. To do this, we arrange for each IO manager thread to register its control pipe with the RTS (in `GHC.Thread.startIOManagerThread`). In addition, `GHC.Thread.startTimerManagerThread` registers its control pipe. These are registered via C functions `setTimerManagerControlFd` (in `rts/Signals.c`) and `setIOManagerControlFd` (in `rts/Capability.c`). The IO manager control pipe file descriptors are stored in a new field of the `Capability_ struct`. Test Plan: See the notes on #9423 to recreate the problem and to verify that it no longer occurs with the fix. Auditors: simonmar Reviewers: simonmar, edsko, ezyang, austin Reviewed By: austin Subscribers: phaskell, simonmar, ezyang, carter, relrod Differential Revision: https://phabricator.haskell.org/D129 GHC Trac Issues: #9423, #9284
Diffstat (limited to 'includes/rts/IOManager.h')
-rw-r--r--includes/rts/IOManager.h3
1 files changed, 2 insertions, 1 deletions
diff --git a/includes/rts/IOManager.h b/includes/rts/IOManager.h
index 1c331b97af..7bf2cdf43c 100644
--- a/includes/rts/IOManager.h
+++ b/includes/rts/IOManager.h
@@ -26,7 +26,8 @@ void sendIOManagerEvent (HsWord32 event);
#else
-void setIOManagerControlFd (int fd);
+void setIOManagerControlFd (nat cap_no, int fd);
+void setTimerManagerControlFd(int fd);
void setIOManagerWakeupFd (int fd);
#endif