fuzz: Speed up mini_miner fuzz target? #32870

issue maflcko openend this issue on July 3, 2025
  1. maflcko commented at 3:39 pm on July 3, 2025: member

    Looks like it is somewhat slow (c.f. https://cirrus-ci.com/task/6602192649453568?logs=ci#L5401), so it would be nice to make it faster.

    Possibly with txgraph, see https://github.com/bitcoin-core/qa-assets/pull/227#issuecomment-3032434420

  2. maflcko added the label Tests on Jul 3, 2025
  3. enirox001 commented at 12:51 pm on July 7, 2025: contributor
    Interested in working on speeding up the mini_miner fuzz target. Are you already on this, or should I take a stab at it?
  4. maflcko commented at 1:17 pm on July 7, 2025: member
    I am not. If you want to work on something, you can just do it without asking. In case there are two pull requests for the same thing, the authors should be glad about it, because they should be qualified to review each others pull request and pick the one that is more suitable to be merged.
  5. warp-records commented at 7:29 pm on July 29, 2025: none

    Looks like it is somewhat slow (c.f. https://cirrus-ci.com/task/6602192649453568?logs=ci#L5401), so it would be nice to make it faster.

    Possibly with txgraph, see bitcoin-core/qa-assets#227 (comment)

    Can you close this now?

  6. fanquake commented at 1:13 pm on September 18, 2025: member
    No, this is still an issue. See here: #33425 (comment). mini_miner is running for ~half an hour. cc @murchandamus @glozow
  7. glozow commented at 1:34 pm on September 18, 2025: member

    Unsure what the best course of action is, since the plan is to reimplement MiniMiner anyway after #28676. If there’s a timeout/infinite loop then that’s a bug we should fix for current wallet users. But if it’s just a slow test, then I’d suggest leaving it as is and it will probably fix itself when mempool is more efficient at cluster calculation.

    I haven’t looked into the input itself (idk if @enirox001 or somebody wants to do it) but if the culprit is just too many potential outpoints then we can knock that down as an intermediate fix.

  8. fanquake referenced this in commit 953544d028 on Sep 20, 2025
  9. fanquake commented at 1:56 pm on September 20, 2025: member
    Think we can close this based on #33429.
  10. fanquake closed this on Sep 20, 2025


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-09-26 15:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me