[wip] [nomerge] [draft] 2510 msan zero #33686

pull maflcko wants to merge 6 commits into bitcoin:master from maflcko:2510-msan-zero changing 9 files +99 −10
  1. maflcko commented at 8:08 AM on October 23, 2025: member

    No description provided.

  2. DrahtBot commented at 8:09 AM on October 23, 2025: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

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

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #33626 (ci: run native fuzz with MSAN job by fanquake)
    • #33483 (CMake: Add dynamic test discovery by purpleKarrot)
    • #33455 (CPack by purpleKarrot)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  3. in .github/workflows/ci.yml:536 in 116fea537a 
     529 | @@ -530,11 +530,23 @@ jobs:
 530 |              timeout-minutes: 120
 531 |              file-env: './ci/test/00_setup_env_native_tsan.sh'
 532 |  
 533 | +          - name: 'MSan, fuzz'
 534 | +            cirrus-runner: 'ghcr.io/cirruslabs/ubuntu-runner-amd64:24.04-lg'
 535 | +            fallback-runner: 'ubuntu-24.04'
 536 | +            timeout-minutes: 120
    fanquake commented at 9:14 AM on October 23, 2025:

    Probably want to bump all the timeouts as far as possible.

  4. maflcko marked this as a draft on Oct 23, 2025
  5. fanquake commented at 2:43 AM on October 24, 2025: member

    4h 53m for MSAN depends. MSAN Fuzz cancelled after 6 hours.

  6. ci: run native fuzz with MSAN job 2649220612
  7. msan -O0 f5a0b148dd
  8. timeout 3641978151
  9. test: Pass bench exe into test framework utils
    This teaches the test framework about the bench executable, which is
required for the next commit.
    23467e8668
  10. test: Run bench sanity checks in parallel with functional tests 85c77053f3
  11. stack overflow 3d1385381b
  12. maflcko force-pushed on Oct 24, 2025
  13. fanquake commented at 11:00 AM on October 29, 2025: member

    Close here now it's been added to the nightly repo (https://github.com/bitcoin/bitcoin/issues/33668#issuecomment-3458046048)?

  14. maflcko commented at 12:47 PM on October 29, 2025: member

    yeah, closing for now. It is a bit faster with the fast machines from the CI here, but it isn't going to pass in 6 hours, even with them, so moving to dedicated hardware without timeouts seems the only option.

  15. maflcko closed this on Oct 29, 2025
  16. maflcko deleted the branch on Oct 29, 2025
  17. bitcoin locked this on Oct 29, 2026
Contributors
maflckoDrahtBotfanquake
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: 2026-10-30 09:41 UTC