summaryrefslogtreecommitdiff
path: root/rts/RtsAPI.c
diff options
context:
space:
mode:
authorDylan Yudaken <dylany@fb.com>2019-12-10 15:05:40 +0000
committerMarge Bot <ben+marge-bot@smart-cactus.org>2020-02-08 10:17:17 -0500
commit7d452be454857549679b93a0682a3f6fedf5d7c1 (patch)
tree73bc0df8dd1370dd5f07eb6a1ac0835045b31c61 /rts/RtsAPI.c
parent7755ffc2920facb7ed74efe379ad825feeaf1024 (diff)
downloadhaskell-7d452be454857549679b93a0682a3f6fedf5d7c1.tar.gz
Fix hs_try_putmvar losing track of running cap
If hs_try_putmvar was called through an unsafe import, it would lose track of the running cap causing a deadlock
Diffstat (limited to 'rts/RtsAPI.c')
-rw-r--r--rts/RtsAPI.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/rts/RtsAPI.c b/rts/RtsAPI.c
index 26433ac209..51a1f2b7cf 100644
--- a/rts/RtsAPI.c
+++ b/rts/RtsAPI.c
@@ -682,6 +682,7 @@ void hs_try_putmvar (/* in */ int capability,
{
Task *task = getTask();
Capability *cap;
+ Capability *task_old_cap USED_IF_THREADS;
if (capability < 0) {
capability = task->preferred_capability;
@@ -702,6 +703,7 @@ void hs_try_putmvar (/* in */ int capability,
// If the capability is free, we can perform the tryPutMVar immediately
if (cap->running_task == NULL) {
cap->running_task = task;
+ task_old_cap = task->cap;
task->cap = cap;
RELEASE_LOCK(&cap->lock);
@@ -712,6 +714,7 @@ void hs_try_putmvar (/* in */ int capability,
// Wake up the capability, which will start running the thread that we
// just awoke (if there was one).
releaseCapability(cap);
+ task->cap = task_old_cap;
} else {
PutMVar *p = stgMallocBytes(sizeof(PutMVar),"hs_try_putmvar");
// We cannot deref the StablePtr if we don't have a capability,