summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMitch Phillips <mitchp@google.com>2023-02-03 21:01:28 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2023-02-03 21:01:28 +0000
commitc07ef96e0d50c288a8cbb35b06085e718967d53a (patch)
tree41f13cc9133c62f687951ce0c99f061bad5c0127
parentc372de5c5aab96d43f2961e3442749a42b83a042 (diff)
parent910200d4fe24782870cba0f369516f48a2c81675 (diff)
downloadgwp_asan-c07ef96e0d50c288a8cbb35b06085e718967d53a.tar.gz
Re-land "[GWP-ASan] Remove thread clamping in tests."" am: 910200d4fe
Original change: https://android-review.googlesource.com/c/platform/external/gwp_asan/+/2420551 Change-Id: Iabf966906509ddee7115c5c40788e11cfd5857cc Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--gwp_asan/tests/recoverable.cpp3
-rw-r--r--gwp_asan/tests/thread_contention.cpp4
2 files changed, 0 insertions, 7 deletions
diff --git a/gwp_asan/tests/recoverable.cpp b/gwp_asan/tests/recoverable.cpp
index a4c5c3f..adc9731 100644
--- a/gwp_asan/tests/recoverable.cpp
+++ b/gwp_asan/tests/recoverable.cpp
@@ -182,9 +182,6 @@ void runInterThreadThrashingSingleAlloc(unsigned NumIterations,
std::atomic<bool> StartingGun{false};
std::vector<std::thread> Threads;
constexpr unsigned kNumThreads = 4;
- if (std::thread::hardware_concurrency() < kNumThreads) {
- GTEST_SKIP() << "Not enough threads to run this test";
- }
char *Ptr = static_cast<char *>(AllocateMemory(*GPA));
diff --git a/gwp_asan/tests/thread_contention.cpp b/gwp_asan/tests/thread_contention.cpp
index 0992b97..26ccd8e 100644
--- a/gwp_asan/tests/thread_contention.cpp
+++ b/gwp_asan/tests/thread_contention.cpp
@@ -44,12 +44,8 @@ void asyncTask(gwp_asan::GuardedPoolAllocator *GPA,
void runThreadContentionTest(unsigned NumThreads, unsigned NumIterations,
gwp_asan::GuardedPoolAllocator *GPA) {
-
std::atomic<bool> StartingGun{false};
std::vector<std::thread> Threads;
- if (std::thread::hardware_concurrency() < NumThreads) {
- NumThreads = std::thread::hardware_concurrency();
- }
for (unsigned i = 0; i < NumThreads; ++i) {
Threads.emplace_back(asyncTask, GPA, &StartingGun, NumIterations);