Bug 158571

Summary: JSC Stress Test failing: jsair-tests.yaml/test.js.ftl-eager-no-cjit
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, fpizlo, rniwa, sbarati
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=158493
Attachments:
Description Flags
the patch keith_miller: review+

Description Ryan Haddad 2016-06-09 10:41:11 PDT
** The following JSC stress test failures have been introduced:
	jsair-tests.yaml/test.js.ftl-eager-no-cjit

https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20JSC%20%28Tests%29/builds/3087
https://build.webkit.org/builders/Apple%20El%20Capitan%2032-bit%20JSC%20%28BuildAndTest%29/builds/2604
Comment 1 Ryan Haddad 2016-06-09 10:43:44 PDT
jsair-tests.yaml/test.js.ftl-eager-no-cjit: Timed out after 339.000000 seconds!
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 1   0x10fe57f30 WTFCrash
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 2   0x10ec9eeaa timeoutThreadMain(void*)
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 3   0x10fec0689 WTF::createThread(void (*)(void*), void*, char const*)::$_0::operator()() const
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 4   0x10fec065d 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&&&)
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 5   0x10fec05fc 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()()
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 6   0x10f507e5a std::__1::function<void ()>::operator()() const
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 7   0x10febf25e WTF::threadEntryPoint(void*)
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 8   0x10fec0d01 WTF::wtfThreadEntryPoint(void*)
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 9   0x7fff9721099d _pthread_body
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 10  0x7fff9721091a _pthread_body
jsair-tests.yaml/test.js.ftl-eager-no-cjit: 11  0x7fff9720e351 thread_start
jsair-tests.yaml/test.js.ftl-eager-no-cjit: test_script_384: line 2: 73168 Segmentation fault: 11  ( "$@" ../../../.vm/JavaScriptCore.framework/Resources/jsc --useFTLJIT\=false --useFunctionDotArguments\=true --validateBytecode\=true --validateGraphAtEachPhase\=true --useFTLJIT\=true --useConcurrentJIT\=false --thresholdForJITAfterWarmUp\=100 --thresholdForJITAfterWarmUp\=10 --thresholdForJITSoon\=10 --thresholdForOptimizeAfterWarmUp\=20 --thresholdForOptimizeAfterLongWarmUp\=20 --thresholdForOptimizeSoon\=20 --thresholdForFTLOptimizeAfterWarmUp\=20 --thresholdForFTLOptimizeSoon\=20 --maximumEvalCacheableSourceLength\=150000 test.js )
jsair-tests.yaml/test.js.ftl-eager-no-cjit: ERROR: Unexpected exit code: 139
FAIL: jsair-tests.yaml/test.js.ftl-eager-no-cjit
Comment 2 Filip Pizlo 2016-06-09 12:20:18 PDT
Ooops.  I think we need to skip these tests in debug.  I'll skip them completely for now.
Comment 3 Filip Pizlo 2016-06-09 12:23:20 PDT
Skipped in http://trac.webkit.org/changeset/201878

I will keep this bug open to track the work of making the test not timeout in debug.
Comment 4 Filip Pizlo 2016-06-10 15:28:36 PDT
Created attachment 281053 [details]
the patch
Comment 5 Keith Miller 2016-06-10 15:49:06 PDT
Comment on attachment 281053 [details]
the patch

r=me.
Comment 6 Filip Pizlo 2016-06-10 15:54:59 PDT
Landed in http://trac.webkit.org/changeset/201947