Skip to content

Commit

Permalink
Merge bitcoin/bitcoin#28697: fuzz: Increase merge -rss_limit_mb
Browse files Browse the repository at this point in the history
fa21535 fuzz: Increase merge -rss_limit_mb (MarcoFalke)

Pull request description:

  For some reason, the limit is hit. (Presumably due to `-set_cover_merge=1` eating more memory, or by simply having more fuzz inputs).

  Fix it by increasing it for the merge operation.

ACKs for top commit:
  dergoegge:
    ACK fa21535
  hebasto:
    ACK fa21535, considering the discussion in bitcoin-core/qa-assets#155.

Tree-SHA512: 4fed0f254eccc6fe0b53656bc345ff898b13811dc39387387317d34b521ab77cee03d82b0896dd92d253b7546b6a7e4bdcd478749f47064374ab44ad759ab9ff
  • Loading branch information
fanquake committed Oct 23, 2023
2 parents c1106cf + fa21535 commit 0f15db0
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions test/fuzz/test_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -277,6 +277,7 @@ def merge_inputs(*, fuzz_pool, corpus, test_list, src_dir, build_dir, merge_dirs
for t in test_list:
args = [
os.path.join(build_dir, 'src', 'test', 'fuzz', 'fuzz'),
'-rss_limit_mb=8000',
'-set_cover_merge=1',
# set_cover_merge is used instead of -merge=1 to reduce the overall
# size of the qa-assets git repository a bit, but more importantly,
Expand Down

0 comments on commit 0f15db0

Please sign in to comment.