On master and 0.17 steps to reproduce:
git checkout bitcoin/master && make distclean
./configure --with-sanitizers=thread CC=clang CXX=clang++
make -j 16 check
unit tests fail --with-sanitizers=thread #14058
issue MarcoFalke opened this issue on August 24, 2018-
MarcoFalke commented at 10:58 PM on August 24, 2018: member
- MarcoFalke added the label Tests on Aug 25, 2018
-
MarcoFalke commented at 1:24 PM on August 26, 2018: member
Also, I saw one travis job hit a segfault in this test (can't recall the job right now) @skeees Any guess on this?
-
skeees commented at 1:40 PM on August 26, 2018: contributor
oh, i should have remembered this. the BOOST_CHECK_* are not thread safe. If i change to asserts it resolves the issue, submitting now
-
MarcoFalke commented at 2:29 PM on August 26, 2018: member
Ah thanks.
Seems there is one warning left in the
processnewblock_signals_orderingtest. The reason for that warning is thatblock.fCheckedis not atomic, so one thread could write to it while another could read from it. The primitive fix would be to make the member std::atomic. - MarcoFalke referenced this in commit 1117283543 on Aug 26, 2018
- MarcoFalke renamed this:
singlethreadedscheduler_ordered unit test fails --with-sanitizers=thread
unit test fails --with-sanitizers=thread
on Aug 26, 2018 - MarcoFalke renamed this:
unit test fails --with-sanitizers=thread
unit tests fail --with-sanitizers=thread
on Aug 26, 2018 -
skeees commented at 3:14 PM on August 26, 2018: contributor
This one is less simple because making that an atomic will delete the automatically generated copy constructors / assignment from CBlock - which means they'll need to be manually declared. Any thoughts?
-
skeees commented at 3:37 PM on August 26, 2018: contributor
I have a branch with that change here: https://github.com/bitcoin/bitcoin/compare/master...skeees:fchecked-atomic?expand=1
- laanwj referenced this in commit 5f71eac634 on Aug 28, 2018
- MarcoFalke closed this on Nov 20, 2018
- laanwj referenced this in commit 5ab5341d13 on Dec 1, 2018
- PastaPastaPasta referenced this in commit 88a9682185 on Jun 27, 2021
- PastaPastaPasta referenced this in commit 2fc4775bfd on Jun 28, 2021
- PastaPastaPasta referenced this in commit a6945f06dc on Jun 29, 2021
- PastaPastaPasta referenced this in commit af66fc6ea3 on Jun 29, 2021
- PastaPastaPasta referenced this in commit 47361e1548 on Jun 29, 2021
- PastaPastaPasta referenced this in commit 29578a6148 on Jun 29, 2021
- PastaPastaPasta referenced this in commit d9df41355c on Jun 29, 2021
- Munkybooty referenced this in commit f547446a20 on Jun 30, 2021
- Munkybooty referenced this in commit 1ff26a39f1 on Jun 30, 2021
- Munkybooty referenced this in commit d1d35d0796 on Jul 1, 2021
- PastaPastaPasta referenced this in commit ee68324db5 on Jul 1, 2021
- Munkybooty referenced this in commit f60f1954de on Aug 17, 2021
- vijaydasmp referenced this in commit 845b6df19e on Sep 5, 2021
- vijaydasmp referenced this in commit 67893ff277 on Sep 6, 2021
- DrahtBot locked this on Sep 8, 2021
Contributors
Labels