MB-19204: hash_table_test: Fix TSan issues 05/62905/2
authorDave Rigby <daver@couchbase.com>
Tue, 6 Oct 2015 10:53:21 +0000 (10:53 +0000)
committerChiyoung Seo <chiyoung@couchbase.com>
Fri, 22 Apr 2016 06:59:51 +0000 (06:59 +0000)
commite91aef6640311454d7336c1de67881e6f20cb00c
treec6d6d2caa9c691a3bb475da76a6390b629962aee
parent6f7bfa3de30c21275687985a3ee6bcf92053f741
MB-19204: hash_table_test: Fix TSan issues

Fix issues with hash_table_test on 3.x:

* The default number of HashTable locks (193) causes problems for
  ThreadSanitizer as it exceeds the maximum number of acquired locks
  it can track. Given that the tests where we do not already set the
  lock count are single-threaded, change these to have 1 lock.

* Remove alarm() calls - the tests take longer when run under TSan,
  and given that CTest already enforeces a test-level timeout these
  are redundent inside the test functions.

* Fix data race on AccessGenerator::size test harness.

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