Closed Bug 1308647 Opened 6 years ago Closed 6 years ago

Intermittent tests\jit-test\jit-test\tests\basic\bug1296249.js | Timeout (code 1, args "--ion-eager --ion-offthread-compile=off --non-writable-jitcode --ion-check-range-analysis --ion-extra-checks --no-sse3 --no-threads")


(Core :: JavaScript Engine: JIT, defect, P1)




Tracking Status
firefox50 --- unaffected
firefox51 --- fixed
firefox52 --- fixed


(Reporter: intermittent-bug-filer, Assigned: sandervv)



(Keywords: intermittent-failure)


(1 file, 1 obsolete file)

It seems that the testcase added in bug 1296249 takes a lot of time.
Can we make it quicker by decreasing the number of items it needs to allocate and still have the testcase triggering on the issue that was fixed by that bug?
Blocks: 1296249
Flags: needinfo?(sandervv)
Priority: -- → P1
Since the bug happens on windows8-64 only, and the link you gave suggests that it only a problem on the debug builds, I found this info about malloc on windows8 64-bit and debug builds:

"The debug heap also fills new memory blocks with a known value." (Note the debug heap is used when _DEBUG is not defined.)

I think that memset()ing the huge amount of memory during debug mode is causing the timeout. What do you think?

I think that the fix is to disable memset()ing huge memory allocations by using _IGNORE_BLOCK during _malloc_dbg() while in debug mode on windows 8 64-bit:
Flags: needinfo?(sandervv) → needinfo?(hv1989)
(In reply to Sander Mathijs van Veen from comment #4)
> I think that memset()ing the huge amount of memory during debug mode is
> causing the timeout. What do you think?

That totally makes sense. The extra things in debug builds probably takes a lot more time. Not sure we should disable those extra "checks". If we could decrease the runtime while still testing the "bug" this revision fixed, we would have both. Else we might just add it to the long-running list.
Flags: needinfo?(hv1989)
Added js function that we discussed on IRC.

Try job results are good:
Although I'm not sure if I've selected the right try job options to execute the jit tests on windows 8.
Assignee: nobody → sandervv
Attachment #8800768 - Flags: review?(hv1989)
Comment on attachment 8800768 [details] [diff] [review]

Review of attachment 8800768 [details] [diff] [review]:

Attachment #8800768 - Flags: review?(hv1989) → review+
Win8 debug on that Try push doesn't look amused.
Marked test case as slow, as discussed on IRC.
Attachment #8800768 - Attachment is obsolete: true
Attachment #8801661 - Flags: review?(hv1989)
Attachment #8801661 - Flags: review?(hv1989) → review+
Keywords: checkin-needed
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Whiteboard: [checkin-needed-aurora]
You need to log in before you can comment on or make changes to this bug.