fuzz: Run in parallel #18688

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2004-fuzzPar changing 1 files +50 −23

MarcoFalke commented at 7:37 PM on April 17, 2020: member

Can be reviewed with --ignore-all-space
MarcoFalke renamed this:
~~fuzz: Merge in parallel~~
fuzz: Run in parallel
on Apr 17, 2020
fuzz: Run in parallel fa66280396
MarcoFalke force-pushed on Apr 17, 2020

MarcoFalke commented at 8:55 PM on April 17, 2020: member

Locally:

$ time ./test/fuzz/test_runner.py ../btc_qa_assets/fuzz_seed_corpus --par=9 -l DEBUG
...
real	1m44.747s
user	9m17.108s
sys	0m32.433s


$ time ./test/fuzz/test_runner.py ../btc_qa_assets/fuzz_seed_corpus --par=1 -l DEBUG
...
real	5m53.437s
user	5m25.461s
sys	0m19.031s

$ # current master
$ time ./test/fuzz/test_runner.py ../btc_qa_assets/fuzz_seed_corpus -l DEBUG
...
real	6m1.995s
user	5m35.519s
sys	0m18.982s

DrahtBot added the label Tests on Apr 17, 2020
practicalswift commented at 5:05 PM on April 18, 2020: contributor

ACK fa66280396890bf616cc75432603111e1046d496 -- patch looks correct and the live demo in https://github.com/bitcoin-core/qa-assets/pull/11#issuecomment-615439226 convinced me: this is good!
MarcoFalke commented at 5:08 PM on April 18, 2020: member

It doesn't seem to speed up the run on travis, but it doesn't slow it down either. I guess I don't care about that as long as locally everything runs fast.
fanquake approved
fanquake commented at 9:20 AM on April 25, 2020: member

ACK fa66280396890bf616cc75432603111e1046d496 - this seems sane, and clearly provides some speedup. I'd post benchmarks but can't seem to get through a full run of ./test/fuzz/test_runner.py without hitting at least a few crashes; see #18763, 18762.
MarcoFalke merged this on Apr 25, 2020
MarcoFalke closed this on Apr 25, 2020
MarcoFalke deleted the branch on Apr 25, 2020
sidhujag referenced this in commit cd50c6fd6c on Apr 25, 2020
jasonbcox referenced this in commit 518fd0b8c2 on Nov 5, 2020
DrahtBot locked this on Feb 15, 2022

Contributors

Labels