No description provided.
[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-
maflcko commented at 8:08 AM on October 23, 2025: member
-
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-->
-
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.
maflcko marked this as a draft on Oct 23, 2025fanquake commented at 2:43 AM on October 24, 2025: member4h 53m for MSAN depends. MSAN Fuzz cancelled after 6 hours.
ci: run native fuzz with MSAN job 2649220612msan -O0 f5a0b148ddtimeout 364197815123467e8668test: Pass bench exe into test framework utils
This teaches the test framework about the bench executable, which is required for the next commit.
test: Run bench sanity checks in parallel with functional tests 85c77053f3stack overflow 3d1385381bmaflcko force-pushed on Oct 24, 2025fanquake commented at 11:00 AM on October 29, 2025: memberClose here now it's been added to the nightly repo (https://github.com/bitcoin/bitcoin/issues/33668#issuecomment-3458046048)?
maflcko commented at 12:47 PM on October 29, 2025: memberyeah, 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.
maflcko closed this on Oct 29, 2025maflcko deleted the branch on Oct 29, 2025
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-04-24 09:13 UTC
More mirrored repositories can be found on mirror.b10c.me