Bug 155790

Summary: [ARM] There are 400-600 more crashes on ARMv7 Thumb2 with GLIB timers
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Critical CC: cgarcia, clopez, ossy, ysuzuki, zan
Priority: P1    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 108645    

Csaba Osztrogonác
Reported 2016-03-23 06:14:49 PDT
On ARMv7 Thumb2 Linux there are - 26 failures with EFL port - (no available link, but I saved the log previously, see it below) - 138 failures with JSCOnly port - https://build.webkit.org/builders/JSCOnly%20Linux%20ARMv7%20Thumb2%20Release/builds/13 - 110 failures with GTK port - https://build.webkit.org/builders/GTK%20Linux%20ARM%20Release/builds/10621 ( these results are on the same revision - r198487 ) failures on EFL port: ---------------------- ** The following JSC stress test failures have been introduced: cdjs-tests.yaml/main.js.eager-no-cjit cdjs-tests.yaml/main.js.no-cjit jsc-layout-tests.yaml/js/script-tests/intl-collator.js.layout jsc-layout-tests.yaml/js/script-tests/intl-collator.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-collator.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-collator.js.layout-no-llint jsc-layout-tests.yaml/js/script-tests/intl-datetimeformat.js.layout jsc-layout-tests.yaml/js/script-tests/intl-datetimeformat.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-datetimeformat.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-datetimeformat.js.layout-no-llint jsc-layout-tests.yaml/js/script-tests/intl-numberformat.js.layout jsc-layout-tests.yaml/js/script-tests/intl-numberformat.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-numberformat.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-numberformat.js.layout-no-llint jsc-layout-tests.yaml/js/script-tests/number-toLocaleString.js.layout jsc-layout-tests.yaml/js/script-tests/number-toLocaleString.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/number-toLocaleString.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/number-toLocaleString.js.layout-no-llint mozilla-tests.yaml/js1_5/Array/regress-157652.js.mozilla mozilla-tests.yaml/js1_5/Array/regress-157652.js.mozilla-baseline mozilla-tests.yaml/js1_5/Array/regress-157652.js.mozilla-dfg-eager-no-cjit-validate-phases stress/arrowfunction-lexical-bind-superproperty.js.dfg-eager stress/arrowfunction-lexical-bind-superproperty.js.dfg-eager-no-cjit-validate stress/arrowfunction-lexical-bind-superproperty.js.no-llint stress/op_mod.js.no-cjit-validate-phases stress/op_mul.js.no-cjit-validate-phases Results for JSC stress tests: 26 failures found.
Attachments
Csaba Osztrogonác
Comment 1 2016-03-23 06:16:06 PDT
note: There are no similar issue on ARMv7 traditional and AArch64. (at least with JSCOnly port)
Csaba Osztrogonác
Comment 2 2016-04-05 09:30:48 PDT
There are only 25 failures on EFL Thumb2 build, but ~600 on JSCOnly Thumb2 and ~400 on GTK Thumb2: - https://build.webkit.org/builders/JSCOnly%20Linux%20ARMv7%20Thumb2%20Release/builds/167 - https://build.webkit.org/builders/GTK%20Linux%20ARM%20Release/builds/10782 - EFL results: ** The following JSC stress test failures have been introduced: jsc-layout-tests.yaml/fast/regex/script-tests/pcre-test-1.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/fast/regex/script-tests/pcre-test-1.js.layout-no-llint jsc-layout-tests.yaml/js/script-tests/intl-collator.js.layout jsc-layout-tests.yaml/js/script-tests/intl-collator.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-collator.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-collator.js.layout-no-llint jsc-layout-tests.yaml/js/script-tests/intl-datetimeformat.js.layout jsc-layout-tests.yaml/js/script-tests/intl-datetimeformat.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-datetimeformat.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-datetimeformat.js.layout-no-llint jsc-layout-tests.yaml/js/script-tests/intl-numberformat.js.layout jsc-layout-tests.yaml/js/script-tests/intl-numberformat.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-numberformat.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/intl-numberformat.js.layout-no-llint jsc-layout-tests.yaml/js/script-tests/number-toLocaleString.js.layout jsc-layout-tests.yaml/js/script-tests/number-toLocaleString.js.layout-dfg-eager-no-cjit jsc-layout-tests.yaml/js/script-tests/number-toLocaleString.js.layout-no-cjit jsc-layout-tests.yaml/js/script-tests/number-toLocaleString.js.layout-no-llint mozilla-tests.yaml/js1_5/Array/regress-157652.js.mozilla-baseline mozilla-tests.yaml/js1_5/Array/regress-157652.js.mozilla-llint stress/arrowfunction-lexical-bind-superproperty.js.dfg-eager stress/arrowfunction-lexical-bind-superproperty.js.dfg-eager-no-cjit-validate stress/arrowfunction-lexical-bind-superproperty.js.no-llint stress/sparse-array-entry-update-144067.js.always-trigger-copy-phase stress/v8-earley-boyer-strict.js.no-llint Results for JSC stress tests: 25 failures found.
Csaba Osztrogonác
Comment 3 2016-04-05 09:31:19 PDT
It seems there is a serious bug near the GLIB timer or in the GC.
Csaba Osztrogonác
Comment 4 2016-04-05 09:32:27 PDT
*** Bug 156160 has been marked as a duplicate of this bug. ***
Csaba Osztrogonác
Comment 5 2016-04-05 09:32:56 PDT
*** Bug 155784 has been marked as a duplicate of this bug. ***
Csaba Osztrogonác
Comment 6 2016-04-05 09:34:00 PDT
*** Bug 156163 has been marked as a duplicate of this bug. ***
Yusuke Suzuki
Comment 7 2016-04-05 10:08:54 PDT
So many test failures report that there are "Illegal instruction". I think there is some bug in online assembler. But I cannot investigate it b/c I didn't have any ARM Thumb2 env.......
Zan Dobersek
Comment 8 2016-04-05 11:02:20 PDT
Bug #150083 has a WIP patch that addresses two places where tail call linking wasn't properly working on Thumb2 platforms on Linux. Bug #154857 probably has more in common with that problem, but I was not experiencing a larger set of JSC test failures with GLib timers when testing with r198847 in release or debug builds, apart from a few failures due to internationalization and timezone settings in the testing environment.
Csaba Osztrogonác
Comment 9 2016-04-12 01:39:14 PDT
*** This bug has been marked as a duplicate of bug 150083 ***
Note You need to log in before you can comment on or make changes to this bug.