Bug 158362

Summary: 32-bit JSC stress test failing: stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, commit-queue, fpizlo, keith_miller, mark.lam, msaboff, ryanhaddad, sbarati
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
proposed patch. msaboff: review+

Description Ryan Haddad 2016-06-03 14:52:10 PDT
JSC stress test failing: stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler

stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: Timed out after 339.000000 seconds!
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 1   0xe4f2dd WTFCrash
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 2   0x72caa timeoutThreadMain(void*)
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 3   0xebf479 WTF::createThread(void (*)(void*), void*, char const*)::$_0::operator()() const
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 4   0xebf459 void std::__1::__invoke_void_return_wrapper<void>::__call<WTF::createThread(void (*)(void*), void*, char const*)::$_0&>(WTF::createThread(void (*)(void*), void*, char const*)::$_0&&&)
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 5   0xebf408 std::__1::__function::__func<WTF::createThread(void (*)(void*), void*, char const*)::$_0, std::__1::allocator<WTF::createThread(void (*)(void*), void*, char const*)::$_0>, void ()>::operator()()
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 6   0x6ac41a std::__1::function<void ()>::operator()() const
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 7   0xebe128 WTF::threadEntryPoint(void*)
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 8   0xebfa7b WTF::wtfThreadEntryPoint(void*)
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 9   0x987c8780 _pthread_body
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 10  0x987c86f6 _pthread_body
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: 11  0x987c5f7a thread_start
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: test_script_18526: line 2: 58192 Segmentation fault: 11  ( "$@" ../../.vm/JavaScriptCore.framework/Resources/jsc --useFTLJIT\=false --useFunctionDotArguments\=true --validateGraph\=true --useSamplingProfiler\=true --useFTLJIT\=true --useConcurrentJIT\=false --thresholdForJITAfterWarmUp\=100 recursive-try-catch.js )
stress/recursive-try-catch.js.ftl-no-cjit-validate-sampling-profiler: ERROR: Unexpected exit code: 139

<https://build.webkit.org/builders/Apple%20El%20Capitan%2032-bit%20JSC%20%28BuildAndTest%29/builds/2545>
Comment 1 Mark Lam 2016-06-06 16:54:08 PDT
Created attachment 280648 [details]
proposed patch.
Comment 2 Michael Saboff 2016-06-06 16:55:39 PDT
Comment on attachment 280648 [details]
proposed patch.

r=me
Comment 3 Mark Lam 2016-06-06 17:00:28 PDT
Thanks for the review.  Landed in r201733: <http://trac.webkit.org/r201733>.