959164 - Jit-test basic\bug710947.js fails on Windows 8 test machines

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Dan Minor [:dminor]]

This test fails on Cedar for Windows 8 (debug):

Appears to timeout:
WARNING -  TEST-UNEXPECTED-FAIL | tests\jit-test\jit-test\tests\basic\bug710947.js | --ion-eager --ion-parallel-compile=off --ion-check-range-analysis --no-sse3
10:04:11     INFO -  INFO exit-status     : -1
10:04:11     INFO -  INFO timed-out       : True
10:04:11     INFO -  INFO stdout          >
10:04:11     INFO -  INFO stderr         2>
10:04:11     INFO -  TIMEOUT - basic\bug710947.js

Comment 1

[Dan Minor [:dminor]]

Like bug 959156 this one uses a large ArrayBuffer that is causing problems on the Windows test machines:

// |jit-test| allow-oom;
function startTest() {
 if (typeof document != "object" 
    || !document.location.href.match(/jsreftest.html/))  {}
};
gczeal(4);
startTest();
ArrayBuffer( 946684800000 );

Can we just remove this one from the testsuite as well? It seems like an intermittent timeout rather than an OOM, so possibly this needs further investigation.

Comment 2

[Terrence Cole [:terrence]]

Unlike the previous test, this is catching a fairly subtle regression. I think any sized allocation larger than 16 * sizeof(Value) should work to trigger it. If we change the last line to |ArrayBuffer( 8192 );| does that fix the timeout?

Comment 3

[Dan Minor [:dminor]]

Attachment: Change ArrayBuffer size to make windows tests machines happy.

I ran this on the loaner test machine and on try and this seems to fix the problem. Try run is here: https://tbpl.mozilla.org/?tree=Try&rev=da83119b547b&showall=1

Thanks for the suggestion!

Comment 4

[Terrence Cole [:terrence]]

Comment on attachment 8367890 [details] [diff] [review]
Change ArrayBuffer size to make windows tests machines happy.

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

Great! r=me

Comment 5

[Dan Minor [:dminor]]

Thanks, pushed to https://hg.mozilla.org/integration/mozilla-inbound/rev/a535989de2b9

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a535989de2b9