From b9ce668437bf5695a1fb3d36619d2b5d8a992355 Mon Sep 17 00:00:00 2001 From: "palfia@homejinni.com" Date: Sat, 9 Nov 2013 00:14:21 +0000 Subject: [PATCH] Set at least PTHREAD_STACK_MIN stack size when creating threads. This patch makes sure, that the stack size of the new threads are never less than PTHREAD_STACK_MIN, otherwise the pthread_attr_setstacksize() function sets the default stack size (8MB), which leads to failure in the BootUpMemoryUse test. TEST=cctest/test-mark-compact/BootUpMemoryUse BUG= R=yangguo@chromium.org Review URL: https://codereview.chromium.org/63183003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17610 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/platform-posix.cc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/platform-posix.cc b/src/platform-posix.cc index 797557d..923cd87 100644 --- a/src/platform-posix.cc +++ b/src/platform-posix.cc @@ -564,6 +564,9 @@ Thread::Thread(const Options& options) : data_(new PlatformData), stack_size_(options.stack_size()), start_semaphore_(NULL) { + if (stack_size_ > 0 && stack_size_ < PTHREAD_STACK_MIN) { + stack_size_ = PTHREAD_STACK_MIN; + } set_name(options.name()); } -- 2.7.4