Enable drd and helgrind in the valgrind jobs

Signed-off-by: Lukasz Dorau <[email protected]>

Author: ldorau

.github/workflows/nightly.yml

@@ -16,7 +16,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-          tool: ['memcheck'] # TODO: enable 'drd' and 'helgrind' when all issues are fixed
+          tool: ['memcheck', 'drd', 'helgrind']
     runs-on: ubuntu-latest
 
     steps:

.github/workflows/valgrind.yml

@@ -11,7 +11,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-          tool: ['memcheck'] # TODO: enable 'drd' and 'helgrind' when all issues are fixed
+        tool: ['memcheck', 'drd', 'helgrind']
     runs-on: ubuntu-latest
 
     steps:

test/supp/drd-umf_test-jemalloc_pool.supp

@@ -0,0 +1,6 @@
+{
+   Conflicting Access in libjemalloc.so*
+   drd:ConflictingAccess
+   obj:/usr/lib/x86_64-linux-gnu/libjemalloc.so*
+   ...
+}

test/supp/drd-umf_test-scalable_pool.supp

@@ -0,0 +1,35 @@
+{
+   Conflicting Access in libtbbmalloc.so*
+   drd:ConflictingAccess
+   obj:*libtbbmalloc.so*
+   ...
+}
+
+{
+   Conflicting Access in memset in pow2AlignedAllocHelper
+   drd:ConflictingAccess
+   fun:memset
+   fun:*pow2AlignedAllocHelper*
+   ...
+}
+
+{
+   ConflictingAccess in tbb_realloc in multiThreadedReallocFree test
+   drd:ConflictingAccess
+   fun:memmove
+   obj:*libtbbmalloc.so*
+   fun:tbb_realloc
+   fun:umfPoolRealloc
+   fun:*umfPoolTest_multiThreadedReallocFree*
+   ...
+}
+
+{
+   ConflictingAccess in memset in multiThreadedCallocFree test
+   drd:ConflictingAccess
+   fun:memset
+   fun:tbb_calloc
+   fun:umfPoolCalloc
+   fun:*umfPoolTest_multiThreadedCallocFree*
+   ...
+}

test/supp/helgrind-umf_test-disjointPool.supp

@@ -0,0 +1,15 @@
+{
+   pthread_mutex_lock with a pthread_rwlock_t* argument
+   Helgrind:Misc
+   obj:*vgpreload_helgrind-amd64-linux.so
+   fun:*gthread_mutex_lock*pthread_mutex_t
+   ...
+}
+
+{
+   pthread_mutex_unlock with a pthread_rwlock_t* argument
+   Helgrind:Misc
+   obj:*vgpreload_helgrind-amd64-linux.so
+   fun:*gthread_mutex_unlock*pthread_mutex_t
+   ...
+}

test/supp/helgrind-umf_test-jemalloc_pool.supp

@@ -0,0 +1,6 @@
+{
+   Race in libjemalloc.so*
+   Helgrind:Race
+   obj:/usr/lib/x86_64-linux-gnu/libjemalloc.so*
+   ...
+}

test/supp/helgrind-umf_test-scalable_pool.supp

@@ -0,0 +1,35 @@
+{
+   Race in libtbbmalloc.so*
+   Helgrind:Race
+   obj:/usr/lib/x86_64-linux-gnu/libtbbmalloc.so*
+   ...
+}
+
+{
+   Race Access in memset in pow2AlignedAllocHelper
+   Helgrind:Race
+   fun:memset
+   fun:*pow2AlignedAllocHelper*
+   ...
+}
+
+{
+   Race in tbb_realloc in multiThreadedReallocFree test
+   Helgrind:Race
+   fun:memmove
+   obj:*libtbbmalloc.so*
+   fun:tbb_realloc
+   fun:umfPoolRealloc
+   fun:*umfPoolTest_multiThreadedReallocFree*
+   ...
+}
+
+{
+   Race in memset in multiThreadedCallocFree test
+   Helgrind:Race
+   fun:memset
+   fun:tbb_calloc
+   fun:umfPoolCalloc
+   fun:*umfPoolTest_multiThreadedCallocFree*
+   ...
+}

test/supp/umf_test-jemalloc_pool.supp renamed to test/supp/memcheck-umf_test-jemalloc_pool.supp

@@ -76,9 +76,9 @@ for test in $(ls -1 ./umf_test-*); do
 	echo -n "$test "
 	LOG=${test}.log
 	ERR=${test}.err
-	SUP="${WORKSPACE}/test/supp/${test}.supp"
+	SUP="${WORKSPACE}/test/supp/${TOOL}-${test}.supp"
 	OPT_SUP=""
-	[ -f ${SUP} ] && OPT_SUP="--suppressions=${SUP}"
+	[ -f ${SUP} ] && OPT_SUP="--suppressions=${SUP}" && echo -n "(${TOOL}-${test}.supp) "
 
 	# skip tests incompatible with valgrind
 	case $test in