Intermittent SM rootanalysis TEST-UNEXPECTED-FAIL | jit_test.py --ion-eager | jit-test/tests/sunspider/check-crypto-aes.js | [check-jit-test] Error 2

RESOLVED FIXED in mozilla22

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: RyanVM, Assigned: sfink)

Tracking

({crash, intermittent-failure})

Trunk
mozilla22
x86_64
Linux
crash, intermittent-failure
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
*Very* useful log on this one...

https://tbpl.mozilla.org/php/getParsedLog.php?id=20264155&tree=Mozilla-Inbound

Linux x86-64 mozilla-inbound leak test spidermonkey_mozilla-inbound-rootanalysis build on 2013-03-02 14:23:47 PST for push 5a0004e62900
slave: bld-linux64-ec2-341

FAIL - /builds/slave/m-in_l64-d_sm-rootanalysis-000/src/js/src/jit-test/tests/sunspider/check-crypto-aes.js
TEST-UNEXPECTED-FAIL | jit_test.py --no-ion --no-ti| /builds/slave/m-in_l64-d_sm-rootanalysis-000/src/js/src/jit-test/tests/sunspider/check-crypto-aes.js
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Comment 3

5 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=20387091&tree=Mozilla-Inbound
Summary: Intermittent SM rootanalysis TEST-UNEXPECTED-FAIL | jit_test.py --no-ion --no-ti | jit-test/tests/sunspider/check-crypto-aes.js | [check-jit-test] Error 2 → Intermittent SM rootanalysis TEST-UNEXPECTED-FAIL | jit_test.py | jit-test/tests/sunspider/check-crypto-aes.js | [check-jit-test] Error 2
(Reporter)

Comment 4

5 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=20389315&tree=Mozilla-Inbound
(Reporter)

Comment 5

5 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=20393926&tree=Mozilla-Inbound
(Assignee)

Updated

5 years ago
Assignee: general → sphink
(Assignee)

Comment 6

5 years ago
Created attachment 721935 [details] [diff] [review]
Avoid using unstable chars

'chars' was being held across calls that could GC. It's still getting passed into RegexpShared::execute(), which can GC, but I'm not sure if it can GC on non-error paths.

I also removed a mysterious redundant 'length' variable, which was getting its length from an unrooted parameter.
Attachment #721935 - Flags: review?(terrence)
Comment on attachment 721935 [details] [diff] [review]
Avoid using unstable chars

Review of attachment 721935 [details] [diff] [review]:
-----------------------------------------------------------------

This appears to be fallout from Bug 837845: glad it was so minor.
Attachment #721935 - Flags: review?(terrence) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://tbpl.mozilla.org/php/getParsedLog.php?id=20407746&tree=Mozilla-Inbound

Updated

5 years ago
Summary: Intermittent SM rootanalysis TEST-UNEXPECTED-FAIL | jit_test.py | jit-test/tests/sunspider/check-crypto-aes.js | [check-jit-test] Error 2 → Intermittent SM rootanalysis TEST-UNEXPECTED-FAIL | jit_test.py --ion-eager | jit-test/tests/sunspider/check-crypto-aes.js | [check-jit-test] Error 2

Comment 9

5 years ago
https://tbpl.mozilla.org/php/getParsedLog.php?id=20414342&tree=Mozilla-Inbound
Comment hidden (Treeherder Robot)
https://hg.mozilla.org/integration/mozilla-inbound/rev/b87d56cf0cb1
Keywords: checkin-needed
(Reporter)

Comment 12

5 years ago
https://hg.mozilla.org/mozilla-central/rev/b87d56cf0cb1
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.