Skip to content

Commit f57a675

Browse files
committed
Merge bitcoin/bitcoin#30826: fuzz: reduce number of iterations in crypto_aeadchacha20poly1305 target
f482d0e fuzz: reduce number of iterations in `crypto_aeadchacha20poly1305` target (brunoerg) Pull request description: By reducing the number of iterations we improve the performance of this target and may increase coverage. Running with `-runs=100000` from qa-assets I noticed a significant performance improvement and an increase on cov: master: ``` #100000 DONE cov: 567 ft: 4078 corp: 124/33Kb lim: 4096 exec/s: 793 rss: 499Mb ``` PR: ``` #100000 DONE cov: 568 ft: 3833 corp: 113/15188b lim: 1746 exec/s: 1250 rss: 544Mb ``` ACKs for top commit: achow101: ACK f482d0e marcofleon: Tested ACK f482d0e. Saw the same slight increase in coverage. Executed 100,000 runs several times and total time went from 30-35 sec to 20-25 sec. stratospher: ACK f482d0e. saw similar coverage stats Tree-SHA512: 1a96dbc22a0aed396b7f8cc9b13534b7f20a461f64f167c69c650529d535e360499f1a501abc1f957f7541ee1860b36a5580aa488a1edbfa0270c9ed83ef741d
2 parents 48c20db + f482d0e commit f57a675

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/test/fuzz/crypto_chacha20poly1305.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ FUZZ_TARGET(crypto_aeadchacha20poly1305)
3939
// data).
4040
InsecureRandomContext rng(provider.ConsumeIntegral<uint64_t>());
4141

42-
LIMITED_WHILE(provider.ConsumeBool(), 10000)
42+
LIMITED_WHILE(provider.ConsumeBool(), 100)
4343
{
4444
// Mode:
4545
// - Bit 0: whether to use single-plain Encrypt/Decrypt; otherwise use a split at prefix.

0 commit comments

Comments
 (0)