refactor: Replace RecursiveMutex with Mutex in rpc/server.cpp #19399

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:200628-warm-mx changing 1 files +7 −7

hebasto commented at 7:14 AM on June 28, 2020: member

The functions that could lock this mutex, i.e., SetRPCWarmupStatus(), SetRPCWarmupFinished(), RPCIsInWarmup(), CRPCTable::execute(), do not call itself recursively, and do not call each other either directly or indirectly. Therefore, the g_rpc_warmup_mutex could be a non-recursive mutex.

Related to #19303.
refactor: Replace RecursiveMutex with Mutex in rpc/server.cpp 6fdfeebcc7
hebasto cross-referenced this on Jun 28, 2020 from issue Replace all of the RecursiveMutex instances with the Mutex ones by hebasto
fanquake added the label Refactoring on Jun 28, 2020
MarcoFalke commented at 11:20 AM on June 28, 2020: member

ACK 6fdfeebcc79df62c8bf1cf4b6e9e97d6aefb3eb3
laanwj commented at 5:06 PM on June 29, 2020: member

ACK 6fdfeebcc79df62c8bf1cf4b6e9e97d6aefb3eb3
laanwj merged this on Jun 29, 2020
laanwj closed this on Jun 29, 2020
hebasto deleted the branch on Jun 29, 2020
Fabcien referenced this in commit 19cb47be44 on Apr 27, 2021
bitcoin locked this on Feb 15, 2022

Contributors

Labels