MB-19204: Remove alarm() call from atomic_ptr_test, reduce iteration count 06/62906/2
authorDave Rigby <daver@couchbase.com>
Thu, 14 Apr 2016 11:14:39 +0000 (12:14 +0100)
committerChiyoung Seo <chiyoung@couchbase.com>
Fri, 22 Apr 2016 07:00:57 +0000 (07:00 +0000)
This test runs slower under ThreadSanitizer than normal. Given we
already have CTest enforcing timeouts, remove the explicit alarm calls
and handle the timeout at the CTest level.

Also reduce the iteration count by 10x, so the test runs in a more
resonable time.

Change-Id: Ia6914c10a3073f5fea121cb7e600568ca5081beb
Reviewed-on: http://review.couchbase.org/62906
Tested-by: buildbot <build@couchbase.com>
Well-Formed: buildbot <build@couchbase.com>
Reviewed-by: Chiyoung Seo <chiyoung@couchbase.com>
tests/module_tests/atomic_ptr_test.cc

index 075f802..c548b9f 100644 (file)
 #include "locks.h"
 #include "threadtests.h"
 
-#ifdef _MSC_VER
-#define alarm(a)
-#endif
-
 #define NUM_THREADS 50
-#define NUM_TIMES 100000
+#define NUM_TIMES 10000
 
 class Doodad : public RCValue {
 public:
@@ -135,7 +131,6 @@ static void testOperators() {
 }
 
 int main() {
-    alarm(60);
     testOperators();
     testAtomicPtr();
 }