[compiler-rt] 11539ed - [NFC][LSAN] Limit the number of concurrent threads is the test

[Vitaly Buka via llvm-commits]

Author: Vitaly Buka
Date: 2021-06-07T17:38:12-07:00
New Revision: 11539edf52d53ccbd42040a30e29d117600cfd6b

URL: https://github.com/llvm/llvm-project/commit/11539edf52d53ccbd42040a30e29d117600cfd6b
DIFF: https://github.com/llvm/llvm-project/commit/11539edf52d53ccbd42040a30e29d117600cfd6b.diff

LOG: [NFC][LSAN] Limit the number of concurrent threads is the test

Test still fails with D88184 reverted.

The test was flaky on https://bugs.chromium.org/p/chromium/issues/detail?id=1206745 and
https://lab.llvm.org/buildbot/#/builders/sanitizer-x86_64-linux

Reviewed By: morehouse

Differential Revision: https://reviews.llvm.org/D102218

Added: 
    

Modified: 
    compiler-rt/test/lsan/TestCases/many_threads_detach.cpp

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/lsan/TestCases/many_threads_detach.cpp b/compiler-rt/test/lsan/TestCases/many_threads_detach.cpp
index 6909a483aa4ce..e004dfeff3d8e 100644
--- a/compiler-rt/test/lsan/TestCases/many_threads_detach.cpp
+++ b/compiler-rt/test/lsan/TestCases/many_threads_detach.cpp
@@ -3,6 +3,8 @@
 // Use `-pthread` so that its driver will DTRT (ie., ignore it).
 // RUN: %clangxx_lsan %s -o %t -pthread && %run %t
 
+#include <assert.h>
+#include <dirent.h>
 #include <pthread.h>
 #include <stdlib.h>
 #include <unistd.h>
@@ -12,15 +14,21 @@
 // reused.
 static const size_t kTestThreads = 10000;
 
+// Limit the number of simultaneous threads to avoid reaching the limit.
+static const size_t kTestThreadsBatch = 100;
+
 void *null_func(void *args) {
   return NULL;
 }
 
 int main(void) {
-  for (size_t i = 0; i < kTestThreads; i++) {
-    pthread_t thread;
-    if (pthread_create(&thread, NULL, null_func, NULL) == 0)
-      pthread_detach(thread);
+  for (size_t i = 0; i < kTestThreads; i += kTestThreadsBatch) {
+    pthread_t thread[kTestThreadsBatch];
+    for (size_t j = 0; j < kTestThreadsBatch; ++j)
+      assert(pthread_create(&thread[j], NULL, null_func, NULL) == 0);
+
+    for (size_t j = 0; j < kTestThreadsBatch; ++j)
+      assert(pthread_join(thread[j], NULL) == 0);
   }
   return 0;
 }