From fa0b766f43df2ead7413363941d74244a55f090c Mon Sep 17 00:00:00 2001 From: MarcoFalke <*~=`'#}+{/-|&$^_@721217.xyz> Date: Mon, 9 Jun 2025 16:30:50 +0200 Subject: [PATCH] test: Remove intermittent and presumed fixed tsan race suppressions LoadWallet was added in commit d77170d52687c3d708daffb7976c3e0d345ff7dc, which points to a traceback with BerkeleyBatch in it. Now that BDB is removed, this can be removed as well. The race in DatabaseBatch was added in commit fd59670642f511523570607b752ae409b5e04e7d, which does not point to a traceback. This was likely also fixed with the BDB removal. If not, the suppressions should be added back, mentioning that they are intermittent and including a traceback and possibly steps to reproduce. --- test/sanitizer_suppressions/tsan | 4 ---- 1 file changed, 4 deletions(-) diff --git a/test/sanitizer_suppressions/tsan b/test/sanitizer_suppressions/tsan index 06cc63510fa..ab50e837082 100644 --- a/test/sanitizer_suppressions/tsan +++ b/test/sanitizer_suppressions/tsan @@ -3,10 +3,6 @@ # # https://github.com/google/sanitizers/wiki/ThreadSanitizerSuppressions -# race (TODO fix) -race:LoadWallet -race:DatabaseBatch - # deadlock (TODO fix) # To reproduce, see: # https://github.com/bitcoin/bitcoin/issues/19303#issuecomment-1514926359