[Openmp-commits] [PATCH] D32033: Make test/parallel/omp_nested.c not use so many threads
Hal Finkel via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Thu Apr 13 10:37:11 PDT 2017
hfinkel created this revision.
Herald added a subscriber: mcrosier.
I've found it very difficult to get test/parallel/omp_nested.c to pass consistently across my build environments. The problem is that it creates N^2 threads (it is testing nested parallel regions), and that often exceeds the thread limits on systems with many cores. We do raise the process limits in lit, and that often helps, but if running lit with a smaller number of threads or on a system where we're otherwise resource constrained, this particular test tends to fail (because the runtime cannot create a sufficient number of threads).
This seems to work: if the maximum number of threads is more than some small number, then cap the number of threads used for the parallel region. The choice of 4 here is somewhat arbitrary.
@@ -9,6 +9,11 @@
+ if (omp_get_max_threads() > 4)
int counter = 0;
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 383 bytes
Desc: not available
More information about the Openmp-commits