diff options
| author |   Kostya Kortchinsky <kostyak@google.com> | 2019-01-24 15:56:54 +0000 |
|---|---|---|
| committer | Kostya Kortchinsky <kostyak@google.com> | 2019-01-24 15:56:54 +0000 |
| commit | f0fbeaf44ae55b12490eaf93184d9c9a17d34067 (patch) | |
| tree | e2ef650cc02e67d5a45f1be698cc43ddd58e00af /compiler-rt/lib/scudo/scudo_flags.inc | |
| parent | [TargetLowering] Rename getExpandedFixedPointMultiplication to expandFixedPoi... (diff) | |
| download | llvm-project-f0fbeaf44ae55b12490eaf93184d9c9a17d34067.tar.gz llvm-project-f0fbeaf44ae55b12490eaf93184d9c9a17d34067.tar.bz2 llvm-project-f0fbeaf44ae55b12490eaf93184d9c9a17d34067.zip | |
[scudo] Tuning changes based on feedback from current use
Summary:
This tunes several of the default parameters used within the allocator:
- disable the deallocation type mismatch on Android by default; this
was causing too many issues with third party libraries;
- change the default `SizeClassMap` to `Dense`, it caches less entries
and is way more memory efficient overall;
- relax the timing of the RSS checks, 10 times per second was too much,
lower it to 4 times (every 250ms), and update the test so that it
passes with the new default.
Reviewers: eugenis
Reviewed By: eugenis
Subscribers: srhines, delcypher, #sanitizers, llvm-commits
Differential Revision: https://reviews.llvm.org/D57116
llvm-svn: 352057
Diffstat (limited to 'compiler-rt/lib/scudo/scudo_flags.inc')
| -rw-r--r-- | compiler-rt/lib/scudo/scudo_flags.inc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/compiler-rt/lib/scudo/scudo_flags.inc b/compiler-rt/lib/scudo/scudo_flags.inc index 6dffcbaa5ae2..c124738c1f3a 100644 --- a/compiler-rt/lib/scudo/scudo_flags.inc +++ b/compiler-rt/lib/scudo/scudo_flags.inc @@ -36,7 +36,9 @@ SCUDO_FLAG(int, QuarantineChunksUpToSize, -1, "Size in bytes up to which chunks will be quarantined (if lower than" "or equal to). Defaults to 256 (32-bit) or 2048 (64-bit)") -SCUDO_FLAG(bool, DeallocationTypeMismatch, true, +// Disable the deallocation type check by default on Android, it causes too many +// issues with third party libraries. +SCUDO_FLAG(bool, DeallocationTypeMismatch, !SANITIZER_ANDROID, "Report errors on malloc/delete, new/free, new/delete[], etc.") SCUDO_FLAG(bool, DeleteSizeMismatch, true, |