Summary: | Remove the Baker barrier from JSC | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||||||
Component: | JavaScriptCore | Assignee: | Filip Pizlo <fpizlo> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | commit-queue, ryanhaddad | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 149432 | ||||||||||
Attachments: |
|
Description
Filip Pizlo
2016-03-14 17:34:17 PDT
Created attachment 274059 [details]
work in progress
Created attachment 274076 [details]
the patch
Attachment 274076 [details] did not pass style-queue:
ERROR: Source/JavaScriptCore/runtime/JSObject.h:872: Multi line control clauses should use braces. [whitespace/braces] [4]
Total errors found: 1 in 22 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 274077 [details]
the patch
Attachment 274077 [details] did not pass style-queue:
ERROR: Source/JavaScriptCore/runtime/JSObject.h:872: Multi line control clauses should use braces. [whitespace/braces] [4]
Total errors found: 1 in 22 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 274077 [details]
the patch
r=me
Landed in http://trac.webkit.org/changeset/198212 This change may have introduced a new JSC test failure <https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20JSC%20%28Tests%29/builds/2125> stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: Timed out after 339.000000 seconds! stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 1 0x10afd8a40 WTFCrash stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 2 0x109ede0aa timeoutThreadMain(void*) stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 3 0x10b043c39 WTF::createThread(void (*)(void*), void*, char const*)::$_0::operator()() const stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 4 0x10b043c0d 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/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 5 0x10b043bac 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/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 6 0x10a8e1eda std::__1::function<void ()>::operator()() const stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 7 0x10b04290e WTF::threadEntryPoint(void*) stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 8 0x10b0441a1 WTF::wtfThreadEntryPoint(void*) stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 9 0x7fff9aefac13 _pthread_body stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 10 0x7fff9aefab90 _pthread_body stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 11 0x7fff9aef8375 thread_start stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: test_script_15145: line 2: 39785 Segmentation fault: 11 ( "$@" ../../.vm/JavaScriptCore.framework/Resources/jsc --useFTLJIT\=false --useFunctionDotArguments\=true --jitMemoryReservationSize\=50000 --useFTLJIT\=true --useConcurrentJIT\=false --thresholdForJITAfterWarmUp\=100 materialize-past-butterfly-allocation.js ) stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: ERROR: Unexpected exit code: 139 FAIL: stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool (In reply to comment #8) > This change may have introduced a new JSC test failure > <https://build.webkit.org/builders/ > Apple%20El%20Capitan%20Debug%20JSC%20%28Tests%29/builds/2125> > > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: > Timed out after 339.000000 seconds! > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 1 > 0x10afd8a40 WTFCrash > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 2 > 0x109ede0aa timeoutThreadMain(void*) > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 3 > 0x10b043c39 WTF::createThread(void (*)(void*), void*, char > const*)::$_0::operator()() const > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 4 > 0x10b043c0d 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/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 5 > 0x10b043bac 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/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 6 > 0x10a8e1eda std::__1::function<void ()>::operator()() const > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 7 > 0x10b04290e WTF::threadEntryPoint(void*) > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 8 > 0x10b0441a1 WTF::wtfThreadEntryPoint(void*) > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 9 > 0x7fff9aefac13 _pthread_body > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 10 > 0x7fff9aefab90 _pthread_body > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: 11 > 0x7fff9aef8375 thread_start > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: > test_script_15145: line 2: 39785 Segmentation fault: 11 ( "$@" > ../../.vm/JavaScriptCore.framework/Resources/jsc --useFTLJIT\=false > --useFunctionDotArguments\=true --jitMemoryReservationSize\=50000 > --useFTLJIT\=true --useConcurrentJIT\=false > --thresholdForJITAfterWarmUp\=100 materialize-past-butterfly-allocation.js ) > stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool: > ERROR: Unexpected exit code: 139 > FAIL: stress/materialize-past-butterfly-allocation.js.ftl-no-cjit-small-pool Extremely unlikely. This pitch just kills dead code. (In reply to comment #9) > Extremely unlikely. This pitch just kills dead code. In that case, apologies for the false alarm. This was the only JSC change in the failing run on Yosemite and El Capitan. I filed https://bugs.webkit.org/show_bug.cgi?id=155526 to track the failure. (In reply to comment #10) > (In reply to comment #9) > > Extremely unlikely. This pitch just kills dead code. > > In that case, apologies for the false alarm. This was the only JSC change in > the failing run on Yosemite and El Capitan. > > I filed https://bugs.webkit.org/show_bug.cgi?id=155526 to track the failure. Looks like it's actually likely that this change did it, given that it's been failing consistently (according to https://bugs.webkit.org/show_bug.cgi?id=155526#c1). I think that since this times out in debug but not release, the most likely reason for the failure is that this is just a super inefficient test. That means that even if it wasn't timing out, it would be taking up a disproportionate amount of time. It's best to remove the test. |