Skip to content

Commit 3a1d2d3

Browse files
authored
Merge pull request #545 from ocaml-multicore/testsuite-clarify
Clarify test suite recommendations in README
2 parents 822ccf1 + 5f05049 commit 3a1d2d3

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

README.md

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,9 +67,14 @@ Installation instructions, and running the tests
6767
The multicore test suite requires OCaml 5.x:
6868
```
6969
opam update
70-
opam switch create 5.0.0
70+
opam switch create 5.3.0
7171
```
7272

73+
It is designed to stress test OCaml 5's multicore features. As such, it requires
74+
a multicore machine to run successfully and will fail on a single-CPU setup. We
75+
recommend running the test suite with 3 or more (virtual) CPUs.
76+
77+
7378
Installing the libraries
7479
------------------------
7580

@@ -110,6 +115,11 @@ dune build
110115
dune runtest -j1 --no-buffer --display=quiet
111116
```
112117

118+
As some of the tests are negative, e.g., confirming known domain unsafety by
119+
finding a counterexample, we recommend running only one property-based test at a
120+
time (`-j1`) to prevent contention between the individual tests for CPU
121+
resources.
122+
113123
Individual tests can be run by invoking `dune exec`. For example:
114124
```
115125
$ dune exec src/atomic/stm_tests.exe -- -v
@@ -830,4 +840,4 @@ let () = Task.teardown_pool pool
830840

831841
---
832842

833-
This project has been created and is maintained by <a href="https://tarides.com/">Tarides</a>.
843+
This project has been created by <a href="https://tarides.com/">Tarides</a>.

0 commit comments

Comments
 (0)