diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2021-05-03 11:13:38 +0200 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2021-05-03 11:13:41 +0200 |
| commit | b8593616dc2ab5b8f81edd8b2408d400e3b696cd (patch) | |
| tree | acd254f757328cee41236fc7d58095b873578974 /src/qt/optionsmodel.cpp | |
| parent | 320e518b9062ea5510abd7876d22dc5bd3352535 (diff) | |
| parent | fac96d026511f22f0202ce3631a38be0e990555f (diff) | |
| download | bitcoin-b8593616dc2ab5b8f81edd8b2408d400e3b696cd.tar.xz | |
Merge bitcoin/bitcoin#21775: p2p: Limit m_block_inv_mutex
fac96d026511f22f0202ce3631a38be0e990555f p2p: Limit m_block_inv_mutex (MarcoFalke)
Pull request description:
Keeping the lock longer than needed is confusing to reviewers and thread analysis. For example, keeping the lock while appending tx-invs, which requires the mempool lock, will tell thread analysis tools an incorrect lock order of `(1) m_block_inv_mutex, (2) pool.cs`.
ACKs for top commit:
Crypt-iQ:
crACK fac96d026511f22f0202ce3631a38be0e990555f
jnewbery:
utACK fac96d026511f22f0202ce3631a38be0e990555f
theStack:
Code-Review ACK fac96d026511f22f0202ce3631a38be0e990555f
Tree-SHA512: fcfac0f1f8b16df7522513abf716b2eed3d2fc9153f231c8cb61f451e342f29c984a5c872deca6bab3e601e5d651874cc229146c9370e46811b4520747a21f2b
Diffstat (limited to 'src/qt/optionsmodel.cpp')
0 files changed, 0 insertions, 0 deletions