Closed Bug 1095282 Opened 8 years ago Closed 8 years ago

jit-test/tests/atomics/basic-tests.js is going to permafail when Gecko 36 merges to Aurora

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
critical

Tracking

()

VERIFIED FIXED
mozilla36
Tracking Status
firefox36 + verified

People

(Reporter: RyanVM, Assigned: lth)

References

Details

Attachments

(1 file)

[Tracking Requested - why for this release]: Permafailing tests when Gecko 36 merges to Aurora.

https://treeherder.mozilla.org/ui/logviewer.html#?job_id=2981184&repo=try

00:29:55 INFO - FAIL - atomics/basic-tests.js
00:29:55 WARNING - TEST-UNEXPECTED-FAIL | tests/jit-test/jit-test/tests/atomics/basic-tests.js | Unknown (code -11, args "")
00:29:55 INFO - INFO exit-status : -11
00:29:55 INFO - INFO timed-out : False
00:29:55 INFO - FAIL - atomics/basic-tests.js
00:29:55 WARNING - TEST-UNEXPECTED-FAIL | tests/jit-test/jit-test/tests/atomics/basic-tests.js | Unknown (code -11, args "--ion-eager --ion-offthread-compile=off")
00:29:55 INFO - INFO exit-status : -11
00:29:55 INFO - INFO timed-out : False
00:29:55 INFO - FAIL - atomics/basic-tests.js
00:29:55 WARNING - TEST-UNEXPECTED-FAIL | tests/jit-test/jit-test/tests/atomics/basic-tests.js | Unknown (code -11, args "--ion-eager --ion-offthread-compile=off --ion-check-range-analysis --no-sse3 --no-threads")
00:29:55 INFO - INFO exit-status : -11
00:29:55 INFO - INFO timed-out : False
00:29:55 INFO - FAIL - atomics/basic-tests.js
00:29:55 WARNING - TEST-UNEXPECTED-FAIL | tests/jit-test/jit-test/tests/atomics/basic-tests.js | Unknown (code -11, args "--baseline-eager")
00:29:55 INFO - INFO exit-status : -11
00:29:55 INFO - INFO timed-out : False
00:29:55 INFO - FAIL - atomics/basic-tests.js
00:29:55 WARNING - TEST-UNEXPECTED-FAIL | tests/jit-test/jit-test/tests/atomics/basic-tests.js | Unknown (code -11, args "--baseline-eager --no-fpu")
00:29:55 INFO - INFO exit-status : -11
00:29:55 INFO - INFO timed-out : False
00:29:55 INFO - FAIL - atomics/basic-tests.js
00:29:55 WARNING - TEST-UNEXPECTED-FAIL | tests/jit-test/jit-test/tests/atomics/basic-tests.js | Unknown (code -11, args "--no-baseline --no-ion")
00:29:55 INFO - INFO exit-status : -11
00:29:55 INFO - INFO timed-out : False
Flags: needinfo?(lhansen)
That's extremely curious since the tests actually contain the appropriate feature guards, but I guess I'll have a look.
Flags: needinfo?(lhansen)
The problem appears to be that the Atomics atom is defined eagerly but is not backed by a class, and so we get an NPE in JS_ResolveStandardClass which assumes otherwise, probably rightly so.  The correct fix is to make the existence of that atom dependent on the same #define as the class definition is dependent on (ENABLE_SHARED_ARRAY_BUFFER).
Assignee: nobody → lhansen
Status: NEW → ASSIGNED
Simple fix: Only eagerly define the Atomics symbol when the class it references will also be defined.
Attachment #8518820 - Flags: review?(jdemooij)
Attachment #8518820 - Flags: review?(jdemooij) → review+
Blocks: 1095276
https://hg.mozilla.org/mozilla-central/rev/97982a1129a2
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Verified, thanks :)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.