[compiler-rt] r187408 - [asan] Raise thread stack size limit.

Evgeniy Stepanov eugeni.stepanov at gmail.com
Tue Jul 30 02:01:18 PDT 2013


Author: eugenis
Date: Tue Jul 30 04:01:18 2013
New Revision: 187408

URL: http://llvm.org/viewvc/llvm-project?rev=187408&view=rev
Log:
[asan] Raise thread stack size limit.

It's a sanity check, mostly, and we've seen threads with >256Mb stack.

Modified:
    compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux_libcdep.cc

Modified: compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux_libcdep.cc
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux_libcdep.cc?rev=187408&r1=187407&r2=187408&view=diff
==============================================================================
--- compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux_libcdep.cc (original)
+++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_linux_libcdep.cc Tue Jul 30 04:01:18 2013
@@ -29,7 +29,7 @@ namespace __sanitizer {
 
 void GetThreadStackTopAndBottom(bool at_initialization, uptr *stack_top,
                                 uptr *stack_bottom) {
-  static const uptr kMaxThreadStackSize = 256 * (1 << 20);  // 256M
+  static const uptr kMaxThreadStackSize = 1 << 30;  // 1Gb
   CHECK(stack_top);
   CHECK(stack_bottom);
   if (at_initialization) {
@@ -70,9 +70,9 @@ void GetThreadStackTopAndBottom(bool at_
   pthread_attr_getstack(&attr, &stackaddr, (size_t*)&stacksize);
   pthread_attr_destroy(&attr);
 
+  CHECK_LE(stacksize, kMaxThreadStackSize);  // Sanity check.
   *stack_top = (uptr)stackaddr + stacksize;
   *stack_bottom = (uptr)stackaddr;
-  CHECK(stacksize < kMaxThreadStackSize);  // Sanity check.
 }
 
 // Does not compile for Go because dlsym() requires -ldl





More information about the llvm-commits mailing list