[sanitizers] Bump malloc limit to 1TB for MSAN, LSAN & DFSAN

[yingcong-wu]

We already have const uptr kMaxAllowedMallocSize = 1ULL << 40; set for ASAN, HWASAN, memprof, TSAN. This patch bumps the malloc limit for MSAN, LSAN and DFSAN to 1TB as well. 8GB is simply not enough nowadays.

[llvmbot]

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

Author: Wu Yingcong (yingcong-wu)

Changes

We already have const uptr kMaxAllowedMallocSize = 1ULL << 40; set for ASAN, HWASAN, memprof, TSAN. This patch bumps the malloca limit for MSAN, LSAN and DFSAN to 1TB as well. 8GB is not enough for now.

---

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

3 Files Affected:

• (modified) compiler-rt/lib/dfsan/dfsan_allocator.cpp (+1-1)
• (modified) compiler-rt/lib/lsan/lsan_allocator.cpp (+1-1)
• (modified) compiler-rt/lib/msan/msan_allocator.cpp (+1-1)

diff --git a/compiler-rt/lib/dfsan/dfsan_allocator.cpp b/compiler-rt/lib/dfsan/dfsan_allocator.cpp
index 63475f434cd100..682df8c6e0346f 100644
--- a/compiler-rt/lib/dfsan/dfsan_allocator.cpp
+++ b/compiler-rt/lib/dfsan/dfsan_allocator.cpp
@@ -45,7 +45,7 @@ const uptr kAllocatorSpace = 0xE00000000000ULL;
 #else
 const uptr kAllocatorSpace = 0x700000000000ULL;
 #endif
-const uptr kMaxAllowedMallocSize = 8UL << 30;
+const uptr kMaxAllowedMallocSize = 1ULL << 40;
 
 struct AP64 {  // Allocator64 parameters. Deliberately using a short name.
   static const uptr kSpaceBeg = kAllocatorSpace;
diff --git a/compiler-rt/lib/lsan/lsan_allocator.cpp b/compiler-rt/lib/lsan/lsan_allocator.cpp
index 12d579a9385b17..493bf5f9efc578 100644
--- a/compiler-rt/lib/lsan/lsan_allocator.cpp
+++ b/compiler-rt/lib/lsan/lsan_allocator.cpp
@@ -31,7 +31,7 @@ static const uptr kMaxAllowedMallocSize = 1ULL << 30;
 #elif defined(__mips64) || defined(__aarch64__)
 static const uptr kMaxAllowedMallocSize = 4ULL << 30;
 #else
-static const uptr kMaxAllowedMallocSize = 8ULL << 30;
+static const uptr kMaxAllowedMallocSize = 1ULL << 40;
 #endif
 
 static Allocator allocator;
diff --git a/compiler-rt/lib/msan/msan_allocator.cpp b/compiler-rt/lib/msan/msan_allocator.cpp
index b1bc5b9390f75b..8350106dc8175f 100644
--- a/compiler-rt/lib/msan/msan_allocator.cpp
+++ b/compiler-rt/lib/msan/msan_allocator.cpp
@@ -71,7 +71,7 @@ static const uptr kAllocatorSpace = 0x700000000000ULL;
 #else
 static const uptr kAllocatorSpace = 0x600000000000ULL;
 #endif
-static const uptr kMaxAllowedMallocSize = 8UL << 30;
+static const uptr kMaxAllowedMallocSize = 1ULL << 40;
 
 struct AP64 {  // Allocator64 parameters. Deliberately using a short name.
   static const uptr kSpaceBeg = kAllocatorSpace;

[yingcong-wu]

Hi @MaskRay @kcc , would you please help review this patch? Thank you!

[yingcong-wu]

Kindly ping @vitalybuka to review this as well.

[yingcong-wu]

Kindly ping @MaskRay @kcc @vitalybuka to review the patch. Thanks.

[yingcong-wu]

Kindly ping @MaskRay @kcc @vitalybuka to review the patch. Thanks.

[yingcong-wu]

Kindly ping @MaskRay @vitalybuka to review the patch. Thanks.

[yingcong-wu]

Hi @MaskRay , I don't have commit access, could you please help land this patch? Thanks.

[yingcong-wu]

Hi @MaskRay , I don't have commit access, could you please help land this patch? Thanks.