[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
  2. DrahtBot commented at 8:09 am on October 23, 2025: contributor

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

    Code Coverage & Benchmarks

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

    Reviews

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

    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.

  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

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-11-02 18:12 UTC

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