[tsan] Remove long sleeps from fork tests
authorKuba Brecka <kuba.brecka@gmail.com>
Thu, 7 Apr 2016 12:10:28 +0000 (12:10 +0000)
committerKuba Brecka <kuba.brecka@gmail.com>
Thu, 7 Apr 2016 12:10:28 +0000 (12:10 +0000)
On one of our testing machines, we're running the tests under heavy load, and especially in the fork-based TSan tests, we're seeing timeouts when a test uses sleep(10), assuming that calling fork() on another thread will finish sooner than that. This patch removes a timeout and makes another one longer.

Differential Revision: http://reviews.llvm.org/D18476

llvm-svn: 265666

compiler-rt/test/tsan/fork_deadlock.cc
compiler-rt/test/tsan/fork_multithreaded.cc

index 5dce990..32006e2 100644 (file)
@@ -12,18 +12,10 @@ static void *incrementer(void *p) {
   return 0;
 }
 
-static void *watchdog(void *p) {
-  sleep(100);  // is not intended to exit
-  fprintf(stderr, "timed out after 100 seconds\n");
-  exit(1);
-  return 0;
-}
-
 int main() {
   barrier_init(&barrier, 2);
-  pthread_t th1, th2;
+  pthread_t th1;
   pthread_create(&th1, 0, incrementer, 0);
-  pthread_create(&th2, 0, watchdog, 0);
   for (int i = 0; i < 10; i++) {
     switch (fork()) {
     default:  // parent
index 33eef93..faf407b 100644 (file)
@@ -6,7 +6,7 @@
 #include <sys/wait.h>
 
 static void *sleeper(void *p) {
-  sleep(10);  // not intended to exit during test
+  sleep(1000);  // not intended to exit during test
   return 0;
 }