From 555494f0af80e57630592b1829f5899507657438 Mon Sep 17 00:00:00 2001 From: Igor Sugak Date: Fri, 1 Sep 2017 13:23:11 -0700 Subject: [PATCH] increase stack size when TSAN is enabled Reviewed By: meyering Differential Revision: D5757469 fbshipit-source-id: ad70c47251993c79a502d30f312534e28c9241d4 --- folly/fibers/FiberManagerInternal-inl.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/folly/fibers/FiberManagerInternal-inl.h b/folly/fibers/FiberManagerInternal-inl.h index 0b423a13..20aa1d9d 100644 --- a/folly/fibers/FiberManagerInternal-inl.h +++ b/folly/fibers/FiberManagerInternal-inl.h @@ -37,8 +37,9 @@ namespace fibers { namespace { inline FiberManager::Options preprocessOptions(FiberManager::Options opts) { -#if defined(FOLLY_SANITIZE_ADDRESS) || defined(UNDEFINED_SANITIZER) - /* ASAN/UBSAN needs a lot of extra stack space. +#if defined(FOLLY_SANITIZE_ADDRESS) || defined(UNDEFINED_SANITIZER) || \ + defined(FOLLY_SANITIZE_THREAD) + /* Sanitizers need a lot of extra stack space. 16x is a conservative estimate, 8x also worked with tests where it mattered. Note that overallocating here does not necessarily increase RSS, since unused memory is pretty much free. */ -- 2.34.1