Step by step, going to replace all of the RecursiveMutex instances with the Mutex ones throughout the code base :)
Not sure if it is possible in all cases though...
This one is a low-hanging fruit.
refactor: Replace RecursiveMutex with Mutex in Shutdown() #19180
pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:200605-shutdown changing 1 files +5 −6-
hebasto commented at 7:10 PM on June 5, 2020: member
-
refactor: Replace RecursiveMutex with Mutex in Shutdown() 1a9ef1d398
- fanquake added the label Refactoring on Jun 5, 2020
- hebasto marked this as ready for review on Jun 5, 2020
-
MarcoFalke commented at 8:09 PM on June 5, 2020: member
ACK 1a9ef1d398dd14728b6bc67a89139cdf827c9753 Shutdown is not recursive, so the same thread can never lock twice (UB)
-
DrahtBot commented at 1:02 AM on June 6, 2020: member
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #19192 (doc: Extract net permissions doc by MarcoFalke)
- #19064 (refactor: Cleanup thread ctor calls by hebasto)
- #18077 (net: Add NAT-PMP port forwarding support by hebasto)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
-
vasild commented at 7:57 AM on June 8, 2020: member
ACK 1a9ef1d3 verified manually that
Shutdown()is not called from places that could be called from insideShutdown().Not sure if it is possible in all cases though...
I think it should be
- vasild approved
-
- MarcoFalke merged this on Jun 8, 2020
- MarcoFalke closed this on Jun 8, 2020
- hebasto deleted the branch on Jun 8, 2020
- MarcoFalke referenced this in commit 8496dbeba6 on Jun 8, 2020
- MarcoFalke referenced this in commit 374fd6fc8b on Jun 8, 2020
- MarcoFalke referenced this in commit 9ad6f14175 on Jun 9, 2020
- Fabcien referenced this in commit c9c716200e on Mar 1, 2021
- PastaPastaPasta referenced this in commit 40f173954b on Jun 27, 2021
- PastaPastaPasta referenced this in commit 6c258b1f13 on Jun 28, 2021
- PastaPastaPasta referenced this in commit b192adca4f on Jun 29, 2021
- PastaPastaPasta referenced this in commit 7eb2b2f910 on Sep 17, 2021
- PastaPastaPasta referenced this in commit 12b5cfcff0 on Sep 19, 2021
- thelazier referenced this in commit d9423ace44 on Sep 25, 2021
- DrahtBot locked this on Feb 15, 2022
Contributors
Labels
Linked (view graph)
#19189 refactor: Replace RecursiveMutex with Mutex in timedata.cpp#19190 refactor: Replace RecursiveMutex with Mutex in netbase.cpp#19213 util: Get rid of RecursiveMutex in Get{Blocks,Data}Dir#19220 refactor: Replace RecursiveMutex with Mutex in warnings.cpp#19303 Replace all of the RecursiveMutex instances with the Mutex ones