Previous attempt: #20218
test: Add MaybeCompactWalletDB tsan suppression (take 2) #20558
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2012-testSanTsan changing 1 files +1 −0-
MarcoFalke commented at 1:02 PM on December 3, 2020: member
-
test: Add MaybeCompactWalletDB tsan suppression fa818a31e1
- fanquake added the label Tests on Dec 3, 2020
-
laanwj commented at 8:45 PM on December 9, 2020: member
Do we have any idea why this is needed? Why does this particular function raise a false positive?
-
MarcoFalke commented at 6:36 AM on December 10, 2020: member
I have no clue. If someone knows why or even how to track this down (which is difficult because it happens intermittently), please let me know.
-
MarcoFalke commented at 6:37 AM on December 10, 2020: member
Though, having ci fail on known issues whose fix is unknown only increases frustration having to reset the builds.
-
laanwj commented at 4:07 PM on December 16, 2020: member
Yeah. What I'm a little bit worried about is that it might be an actual problem and not a false positive, in which case it would be a really bad idea to silence it. But I don't know how to debug this either.
- DrahtBot added the label Needs rebase on Apr 14, 2021
-
DrahtBot commented at 9:08 AM on April 14, 2021: contributor
<!--cf906140f33d8803c4a75a2196329ecb-->
🐙 This pull request conflicts with the target branch and needs rebase.
<sub>Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft".</sub>
- hebasto cross-referenced this on Apr 14, 2021 from issue test: Use called_from_lib to point uninstrumented libs to TSan by hebasto
- MarcoFalke closed this on Apr 15, 2021
- MarcoFalke deleted the branch on Apr 15, 2021
- bitcoin locked this on Aug 16, 2022
