diff options
author | Simon Marlow <marlowsd@gmail.com> | 2016-08-04 15:57:37 +0100 |
---|---|---|
committer | Simon Marlow <marlowsd@gmail.com> | 2016-08-05 10:13:28 +0100 |
commit | ce13a9a9f57d61170837532948fed8bc1924a7ab (patch) | |
tree | f442352b3fa558e8b8de8328bc8351995ffd9cc9 /compiler/codeGen | |
parent | ca7e1ad346362ba68b430550110e74421b19193f (diff) | |
download | haskell-ce13a9a9f57d61170837532948fed8bc1924a7ab.tar.gz |
Fix an assertion that could randomly fail
Summary:
ASSERT_THREADED_CAPABILITY_INVARIANTS was testing properties of the
returning_tasks queue, but that requires cap->lock to access safely.
This assertion would randomly fail if stressed enough.
Instead I've removed it from the catch-all
ASSERT_PARTIAL_CAPABILITIY_INVARIANTS and made it a separate assertion
only called under cap->lock.
Test Plan:
```
cd testsuite/tests/concurrent/should_run
make TEST=setnumcapabilities001 WAY=threaded1 EXTRA_HC_OPTS=-with-rtsopts=-DS CLEANUP=0
while true; do ./setnumcapabilities001.run/setnumcapabilities001 4 9 2000 || break; done
```
Reviewers: niteria, bgamari, ezyang, austin, erikd
Subscribers: thomie
Differential Revision: https://phabricator.haskell.org/D2440
GHC Trac Issues: #10860
Diffstat (limited to 'compiler/codeGen')
0 files changed, 0 insertions, 0 deletions