cpp_thread_test/dgemv: cap concurrency to number of hw threads on small systems

... instead of (number of hw threads - 4) to avoid invalid numbers on
smaller systems. Currently, systems with 4 or fewer CPUs (e.g., small CI
VMs) would fail the test. Fixes one of the issues discussed in #2668

Signed-off-by: Marius Hillenbrand <mhillen@linux.ibm.com>

diff --git a/cpp_thread_test/dgemv_thread_safety.cpp b/cpp_thread_test/dgemv_thread_safety.cpp
index 5411fec..277594f 100644 (file)
--- a/cpp_thread_test/dgemv_thread_safety.cpp
+++ b/cpp_thread_test/dgemv_thread_safety.cpp
@@ -18,7 +18,7 @@ int main(int argc, char* argv[]){
        uint32_t maxHwThreads = omp_get_max_threads();
        
        if (maxHwThreads < 52)
-               numConcurrentThreads = maxHwThreads -4;
+               numConcurrentThreads = maxHwThreads;
        
        if (argc > 4){
                std::cout<<"ERROR: too many arguments for thread safety tester"<<std::endl;