fuzz: Drop unnecessary mutexes #35723

pull marcofleon wants to merge 1 commits into bitcoin:master from marcofleon:2026/07/remove-unnecessary-mutex changing 2 files +5 −9

marcofleon commented at 6:28 PM on July 14, 2026: contributor

Quick cleanup addressing #35295 (review).

This follows the same logic as 48df0939e7250a2de8a0c336bc3ddbcbe0578812 from #35521.

fuzz: Drop unnecessary mutexes

Remove the `Mutex` from the `coins_view` and `coinscache_sim`
pool startup helpers. Fuzz targets are entered sequentially within a
process and parallel fuzzing uses separate processes/forks, which each
have their own copy of the global thread pool. Therefore, a mutex to
prevent two in-process callers from racing to start the pool isn't needed.

240d5f743a

DrahtBot added the label Fuzzing on Jul 14, 2026
DrahtBot commented at 6:28 PM on July 14, 2026: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35723.



Reviews

See the guideline and AI policy for information on the review process.

Type Reviewers

ACK maflcko, nervana21, l0rinc

If your review is incorrectly listed, please copy-paste <code></code> into the comment that the bot should ignore.
maflcko commented at 6:52 PM on July 14, 2026: member

lgtm ACK 240d5f743aab0ff4f3e5d7d1783b410aa3306ad3
nervana21 commented at 7:08 PM on July 14, 2026: contributor

ACK 240d5f743aab0ff4f3e5d7d1783b410aa3306ad3
l0rinc commented at 7:11 PM on July 14, 2026: contributor

code review ACK 240d5f743aab0ff4f3e5d7d1783b410aa3306ad3
fanquake merged this on Jul 14, 2026
fanquake closed this on Jul 14, 2026

Type	Reviewers
ACK	maflcko, nervana21, l0rinc

Contributors

Labels

Linked (view graph)

#35295 validation: fetch block input prevouts in parallel during ConnectBlock #35521 fuzz: Speed up `dbwrapper_concurrent_reads` harness

fuzz: Drop unnecessary mutexes #35723

Code Coverage & Benchmarks

Reviews