From f1e34517cfb679ed51a6530b7b9fa6bb97fdafc0 Mon Sep 17 00:00:00 2001 From: Bobby Holley Date: Sun, 17 Nov 2013 18:46:40 -0800 Subject: [PATCH] Bug 938429 - Bump the trusted script buffer for macosx64 a bit. r=luke --- js/xpconnect/src/XPCJSRuntime.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/js/xpconnect/src/XPCJSRuntime.cpp b/js/xpconnect/src/XPCJSRuntime.cpp index 1eca48f6c4b..5513a149f49 100644 --- a/js/xpconnect/src/XPCJSRuntime.cpp +++ b/js/xpconnect/src/XPCJSRuntime.cpp @@ -2951,6 +2951,7 @@ XPCJSRuntime::XPCJSRuntime(nsXPConnect* aXPConnect) // ordered by the effective categories in which they are grouped in the // JS_SetNativeStackQuota call (which predates this analysis). // + // (NB: These numbers may have drifted recently - see bug 938429) // OSX 64-bit Debug: 7MB stack, 636 stack frames => ~11.3k per stack frame // OSX64 Opt: 7MB stack, 2440 stack frames => ~3k per stack frame // @@ -2978,9 +2979,9 @@ XPCJSRuntime::XPCJSRuntime(nsXPConnect* aXPConnect) #if defined(XP_MACOSX) || defined(DARWIN) // MacOS has a gargantuan default stack size of 8MB. Go wild with 7MB, - // and give trusted script 120k extra. The stack is huge on mac anyway. + // and give trusted script 140k extra. The stack is huge on mac anyway. const size_t kStackQuota = 7 * 1024 * 1024; - const size_t kTrustedScriptBuffer = 120 * 1024; + const size_t kTrustedScriptBuffer = 140 * 1024; #elif defined(MOZ_ASAN) // ASan requires more stack space due to red-zones, so give it double the // default (2MB on 32-bit, 4MB on 64-bit). ASAN stack frame measurements