diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2022-02-08 15:53:45 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2022-02-08 15:53:49 +0100 |
| commit | 280a7777d3a368101d667a80ebc536e95abb2f8c (patch) | |
| tree | aa44f55c6f8bc89b1b0f42c319f3459ec7423ee4 /src/zmq/zmqutil.cpp | |
| parent | 8edb0416dd4a13f6fae0c7f2f2061dd6f9365a0e (diff) | |
| parent | 99de8068cd08ecc2ad5dfe603bf3c2cc5b8b33aa (diff) | |
Merge bitcoin/bitcoin#24235: validation: use stronger EXCLUSIVE_LOCKS_REQUIRED()
99de8068cd08ecc2ad5dfe603bf3c2cc5b8b33aa validation: use stronger EXCLUSIVE_LOCKS_REQUIRED() (Vasil Dimov)
Pull request description:
https://github.com/bitcoin/bitcoin/pull/24103 added annotations to
denote that the callers of `CChainState::ActivateBestChain()` and
`CChainState::InvalidateBlock()` must not own `m_chainstate_mutex` at
the time of the call.
Replace the added `LOCKS_EXCLUDED()` with a stronger
`EXCLUSIVE_LOCKS_REQUIRED()`, see
https://clang.llvm.org/docs/ThreadSafetyAnalysis.html#negative for the
difference between both.
ACKs for top commit:
hebasto:
ACK 99de8068cd08ecc2ad5dfe603bf3c2cc5b8b33aa.
jonatack:
ACK 99de8068cd08ecc2ad5dfe603bf3c2cc5b8b33aa. Tested with Debian clang version 13.0.1. Reproduced hebasto's results. Verified that `LoadExternalBlockFile()` needs the annotation added here.
Tree-SHA512: 59640d9ad472cdb5066ecde89cc0aff8632a351fc030f39bb43800d2c856fb1aed3576e4134212d32be161b18780f06dc5066ac71df7f7cd69e3f21f886e1542
Diffstat (limited to 'src/zmq/zmqutil.cpp')
0 files changed, 0 insertions, 0 deletions