Closed
Bug 967467
Opened 11 years ago
Closed 11 years ago
Jit-test basic\bug698584.js timesout on Windows XP test machines for debug builds
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla30
People
(Reporter: dminor, Assigned: dminor)
References
Details
+++ This bug was initially created as a clone of Bug #959155 +++
It turns out the failure on Windows XP debug was a timeout, not an out of memory, and needs to be dealt with separately.
Assignee | ||
Comment 1•11 years ago
|
||
Based on the try run here [1], this will run for at least 20 minutes and the test run will be killed due to 1200 seconds of no output.
[1] https://tbpl.mozilla.org/?tree=Try&rev=6f78e053742c&showall=1
Assignee | ||
Comment 2•11 years ago
|
||
Given more time, they will run to completion in 9776 seconds.
Assignee | ||
Comment 3•11 years ago
|
||
From what I can tell without having a profiler available, it looks like the allocations are just taking a really, really long time in this case.
I think the next step would be to ask releng to install development tools on the loaner test machine so that I and debug and/or profile to see exactly what is going wrong here.
Before I do that, I wanted to make sure that this test is worth spending more time on or if we should just remove it from the test suite or mark it as 'slow'.
Flags: needinfo?(terrence)
Comment 4•11 years ago
|
||
This test takes 0.18s to run locally and uses 696M of memory. The specific thing it is testing is a missing OOM check when JaegerMonkey stubs into our regex machinery. Given that JM is just gone, I can't see this regression test being particularly useful in future.
r=me to remove this test.
Flags: needinfo?(terrence)
Assignee | ||
Comment 5•11 years ago
|
||
Thanks! pushed to https://hg.mozilla.org/integration/mozilla-inbound/rev/7ec2f0199218
Comment 6•11 years ago
|
||
Assignee: nobody → dminor
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in
before you can comment on or make changes to this bug.
Description
•