Skip to content

Commit

Permalink
increase stack size when TSAN is enabled
Browse files Browse the repository at this point in the history
Reviewed By: meyering

Differential Revision: D5757469

fbshipit-source-id: ad70c47251993c79a502d30f312534e28c9241d4
  • Loading branch information
Igor Sugak authored and facebook-github-bot committed Sep 1, 2017
1 parent bb9a223 commit 555494f
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions folly/fibers/FiberManagerInternal-inl.h
Original file line number Diff line number Diff line change
Expand Up @@ -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. */
Expand Down

0 comments on commit 555494f

Please sign in to comment.