[compiler-rt] [scudo] default disable allocation ring buffer for trusty (PR #72372)

via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 15 03:36:39 PST 2023


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Florian Mayer (fmayer)

<details>
<summary>Changes</summary>

this is currently done on AOSP trusty-main. it's easier to do it upstream and remove the divergence.

---
Full diff: https://github.com/llvm/llvm-project/pull/72372.diff


1 Files Affected:

- (modified) compiler-rt/lib/scudo/standalone/flags.inc (+1-1) 


``````````diff
diff --git a/compiler-rt/lib/scudo/standalone/flags.inc b/compiler-rt/lib/scudo/standalone/flags.inc
index f5a2bab5057ae13..03156eb92ba0e3c 100644
--- a/compiler-rt/lib/scudo/standalone/flags.inc
+++ b/compiler-rt/lib/scudo/standalone/flags.inc
@@ -46,6 +46,6 @@ SCUDO_FLAG(int, release_to_os_interval_ms, SCUDO_ANDROID ? INT32_MIN : 5000,
            "Interval (in milliseconds) at which to attempt release of unused "
            "memory to the OS. Negative values disable the feature.")
 
-SCUDO_FLAG(int, allocation_ring_buffer_size, 32768,
+SCUDO_FLAG(int, allocation_ring_buffer_size, SCUDO_TRUSTY ? 0 : 32768,
            "Entries to keep in the allocation ring buffer for scudo. "
            "Values less or equal to zero disable the buffer.")

``````````

</details>


https://github.com/llvm/llvm-project/pull/72372


More information about the llvm-commits mailing list