ci: Doc ASLR workaround for sanitizer tasks #33674

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2510-ci-doc-aslr changing 1 files +8 −0

maflcko commented at 7:15 AM on October 22, 2025: member

Fixes https://github.com/bitcoin/bitcoin/issues/30674
ci: Doc ASLR workaround for sanitizer tasks fa0e36156c
DrahtBot added the label Tests on Oct 22, 2025
DrahtBot commented at 7:15 AM on October 22, 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/33674.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK fanquake

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
maflcko requested review from Sjors on Oct 22, 2025
fanquake commented at 7:18 AM on October 23, 2025: member

ACK fa0e36156cba535846ae2ecfaaa554d7f14fcdfd
fanquake merged this on Oct 23, 2025
fanquake closed this on Oct 23, 2025
maflcko deleted the branch on Oct 23, 2025

Type	Reviewers
ACK	fanquake

in ci/README.md:28 in fa0e36156c

  19 | @@ -20,6 +20,14 @@ requires `bash`, `docker`, and `python3` to be installed. To run on different ar
  20 |  sudo apt install bash docker.io python3 qemu-user-static
  21 |  ```
  22 |  
  23 | +For some sanitizer builds, the kernel's address-space layout randomization
  24 | +(ASLR) entropy can cause sanitizer shadow memory mappings to fail. When running
  25 | +the CI locally you may need to reduce that entropy by running:
  26 | +
  27 | +```
  28 | +sudo sysctl -w vm.mmap_rnd_bits=28

Sjors commented at 10:40 AM on October 24, 2025:

What does -w do?

Sjors commented at 10:41 AM on October 24, 2025:

       -w, --write
              Force all arguments to be write arguments and print an error if they cannot be parsed this way.

maflcko commented at 12:54 PM on October 24, 2025:

Yeah, seems preferable to always use the long option name for self-documenting code to avoid confusion. Can be changed in a follow-up, as this is merged.

Sjors commented at 10:41 AM on October 24, 2025: member

Post merge ACK
sedited referenced this in commit f50d90fcbb on Oct 29, 2025
stickies-v referenced this in commit a785fe6759 on Nov 5, 2025
stringintech referenced this in commit b82132c4d1 on Nov 6, 2025
yuvicc referenced this in commit f620a2206b on Nov 16, 2025
Kino1994 referenced this in commit 0933b241f1 on Jun 28, 2026
BigcoinBGC referenced this in commit 10fbff69cd on Jun 30, 2026
Kino1994 referenced this in commit 42f864d302 on Aug 19, 2026
bitcoin locked this on Oct 24, 2026

Contributors

Labels

Linked (view graph)

#30674 TSAN/MSAN fails with vm.mmap_rnd_bits=32 even with llvm 18.1.3

ci: Doc ASLR workaround for sanitizer tasks #33674

Code Coverage & Benchmarks

Reviews