test: Set tsan enable_adaptive_delay=1 by default #36308

pull maflcko wants to merge 4 commits into bitcoin:master from maflcko:2609-tsan-test-adapt-delay changing 8 files +81 −29
  1. maflcko commented at 9:07 AM on September 21, 2026: member

    There are several small sanitizer option issues:

    • Using the project's default settings outside of CI is only possible manually
    • Modifying the project's default requires modifying three places (docs, ci, and the fuzz runner)

    Fix both issues by adding a with_sanitizer_env.py ...cmd... command wrapper that sets the default values.

    Also, enable https://clang.llvm.org/docs/ThreadSanitizer.html#adaptive-delay, which may help to find some kind of rare races more frequently.

  2. doc: Remove section on how to install libasan etc
    The package name depends on the system and it should be trivial for a
    developer to install it in any way.
    fad2fc5a0f
  3. DrahtBot renamed this:
    test: Set tsan enable_adaptive_delay=1 by default
    test: Set tsan enable_adaptive_delay=1 by default
    on Sep 21, 2026
  4. DrahtBot added the label Tests on Sep 21, 2026
  5. DrahtBot commented at 9:07 AM on September 21, 2026: 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/36308.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

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

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #36121 (fuzz: Corpus sharding in parallel runs by maflcko)
    • #35762 (test: optionally run functional tests via CTest by willcl-ark)

    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-->

  6. test: Add with_sanitizer_env.py
    To run a command with default sanitizer options.
    
    This allows to de-duplicate the various places where options are
    hard-coded.
    
    Also, it should be easier to run tests locally with the options applied.
    faf907fd13
  7. test: Set tsan enable_adaptive_delay=1 by default fafce3be39
  8. maflcko force-pushed on Sep 21, 2026
  9. DrahtBot added the label CI failed on Sep 21, 2026
  10. DrahtBot removed the label CI failed on Sep 21, 2026
  11. test: Use sanitizer env in functional tests by default
    This allows to run the functional tests without having to manually set
    the sanitizer env.
    fa0d826eac
  12. maflcko force-pushed on Sep 21, 2026
  13. DrahtBot added the label CI failed on Sep 21, 2026
  14. DrahtBot removed the label CI failed on Sep 21, 2026
  15. dergoegge approved
  16. dergoegge commented at 8:56 AM on September 22, 2026: member

    utACk fa0d826eac336238642428a14db9d122aa6ecd4d

Labels

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-09-23 13:51 UTC

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