diff options
| author | Samuel Dobson <dobsonsa68@gmail.com> | 2021-11-27 21:46:21 +1300 |
|---|---|---|
| committer | Samuel Dobson <dobsonsa68@gmail.com> | 2021-11-27 22:30:46 +1300 |
| commit | 200d97faf2ed0bd0cbef93dd4bbbe77cf8fe5d13 (patch) | |
| tree | e64061a766d92aeb6a8549ea2db007bf4f07029f /src/noui.h | |
| parent | 111c3e06b35b7867f2e0f740e988f648ac6c325d (diff) | |
| parent | f13a22a631efe01e1fbae4ae78a4901d14ebda3c (diff) | |
Merge bitcoin/bitcoin#22868: wallet: Call load handlers without cs_wallet locked
f13a22a631efe01e1fbae4ae78a4901d14ebda3c wallet: Call load handlers without cs_wallet locked (João Barbosa)
Pull request description:
Don't have `cs_wallet` locked while calling each `context.wallet_load_fns`. A load handler can always lock `cs_wallet` if needed.
The lock was added in 1c7e25db0c to satisfy TSAN. With 44c430ffac most of the code requiring the lock is in `CWallet::AttachChain`. A comment is added to warn about wallets_mutex and cs_wallet lock ordering.
ACKs for top commit:
meshcollider:
re-utACK f13a22a631efe01e1fbae4ae78a4901d14ebda3c
ryanofsky:
Code review ACK f13a22a631efe01e1fbae4ae78a4901d14ebda3c. Only change since last review is adding a lock order comment
jonatack:
ACK f13a22a631efe01e1fbae4ae78a4901d14ebda3c
Tree-SHA512: d51976c3aae4bebc2d1997c88edff712d21fc5523801f5614062a10f826e164579973aeb1981bb1cbc243ecff6af3250362f544c02a79e5d135cbbca1704be62
Diffstat (limited to 'src/noui.h')
0 files changed, 0 insertions, 0 deletions