summaryrefslogtreecommitdiff
path: root/lib/scudo/scudo_allocator.cpp
diff options
context:
space:
mode:
authorKostya Kortchinsky <kostyak@google.com>2016-12-15 18:06:55 +0000
committerKostya Kortchinsky <kostyak@google.com>2016-12-15 18:06:55 +0000
commitcf34efc5911e10aa88bf2176857f207289876f1f (patch)
treed29e446b7d3141385c5fdb183f2e68363c7a3465 /lib/scudo/scudo_allocator.cpp
parent3c1161f95436df4fbd8482fda1776c8c1b7968ff (diff)
[scudo] Use DefaultSizeClassMap for 32-bit
Summary: With the recent changes to the Secondary, we use less bits for UnusedBytes, which allows us in return to increase the bits used for Offset. That means that we can use a Primary SizeClassMap allowing for a larger maximum size. Reviewers: kcc, alekseyshl Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D27816 git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@289838 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/scudo/scudo_allocator.cpp')
-rw-r--r--lib/scudo/scudo_allocator.cpp12
1 files changed, 6 insertions, 6 deletions
diff --git a/lib/scudo/scudo_allocator.cpp b/lib/scudo/scudo_allocator.cpp
index 4f3b05ffb..96cfbdbc1 100644
--- a/lib/scudo/scudo_allocator.cpp
+++ b/lib/scudo/scudo_allocator.cpp
@@ -68,7 +68,7 @@ typedef FlatByteMap<NumRegions> ByteMap;
# elif SANITIZER_WORDSIZE == 64
typedef TwoLevelByteMap<(NumRegions >> 12), 1 << 12> ByteMap;
# endif // SANITIZER_WORDSIZE
-typedef SizeClassMap<3, 4, 8, 16, 64, 14> SizeClassMap;
+typedef DefaultSizeClassMap SizeClassMap;
typedef SizeClassAllocator32<0, SANITIZER_MMAP_RANGE_SIZE, 0, SizeClassMap,
RegionSizeLog, ByteMap> PrimaryAllocator;
#endif // SANITIZER_CAN_USE_ALLOCATOR64
@@ -354,11 +354,11 @@ struct Allocator {
"header\n");
}
// Verify that we can fit the maximum amount of unused bytes in the header.
- // The worst case scenario would be when allocating 1 byte on a MaxAlignment
- // alignment. Since the combined allocator currently rounds the size up to
- // the alignment before passing it to the secondary, we end up with
- // MaxAlignment - 1 extra bytes.
- uptr MaxUnusedBytes = MaxAlignment - 1;
+ // Given that the Secondary fits the allocation to a page, the worst case
+ // scenario happens in the Primary. It will depend on the second to last
+ // and last class sizes, as well as the dynamic base for the Primary. The
+ // following is an over-approximation that works for our needs.
+ uptr MaxUnusedBytes = SizeClassMap::kMaxSize - 1 - AlignedChunkHeaderSize;
Header.UnusedBytes = MaxUnusedBytes;
if (Header.UnusedBytes != MaxUnusedBytes) {
dieWithMessage("ERROR: the maximum possible unused bytes doesn't fit in "