Closed Bug 1119363 Opened 10 years ago Closed 10 years ago

Intermittent test_bug732665.xul | Chrome should be invokable from content script with an exhausted stack: 10

Categories

(Core :: XPConnect, defect)

x86_64
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla38
Tracking Status
firefox36 --- unaffected
firefox37 --- fixed
firefox38 --- fixed
firefox-esr31 --- unaffected

People

(Reporter: RyanVM, Assigned: shu)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

04:01:24 INFO - 1516 INFO TEST-START | js/xpconnect/tests/chrome/test_bug732665.xul 04:01:24 INFO - 1517 INFO TEST-PASS | js/xpconnect/tests/chrome/test_bug732665.xul | Chrome should be able to have at least 10 heavy frames more stack than content: 80, 69 04:01:24 INFO - 1518 INFO TEST-UNEXPECTED-FAIL | js/xpconnect/tests/chrome/test_bug732665.xul | Chrome should be invokable from content script with an exhausted stack: 10 - expected PASS 04:01:24 INFO - 1519 INFO MEMORY STAT vsize after test: 1080451072 04:01:24 INFO - 1520 INFO MEMORY STAT vsizeMaxContiguous after test: 8519033622528 04:01:24 INFO - 1521 INFO MEMORY STAT residentFast after test: 360374272 04:01:24 INFO - 1522 INFO MEMORY STAT heapAllocated after test: 107745478 04:01:24 INFO - 1523 INFO TEST-OK | js/xpconnect/tests/chrome/test_bug732665.xul | took 38ms
This means that the stack limits in XPCJSRuntime.cpp need to be adjusted. I recall shu being interested in this code - shu, do you have a few cycles to do something intelligently here?
Flags: needinfo?(shu)
Not this week, but I'll put it on my queue of things to look at.
Thanks shu.
Not sure what else is there to be done here. See the try push in comment 31. I forced PGO by adding mk_add_options MOZ_PGO=1. The opt m-oth tests are all green, and the intermittent oranges in the debug seem unrelated to stack exhaustion.
Attachment #8555473 - Flags: review?(bobbyholley)
Attachment #8555473 - Flags: review?(bobbyholley) → review+
Flags: needinfo?(shu)
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Assignee: nobody → shu
Please request Aurora approval on this when you get a chance.
Flags: needinfo?(shu)
Comment on attachment 8555473 [details] [diff] [review] Tweak trusted stack buffer to give Win64 PGO more room. Approval Request Comment [Feature/regressing bug #]: No clue. [User impact if declined]: Inability to call back into some chrome code when stack is exhausted. [Describe test coverage new/current, TreeHerder]: On central. [Risks and why]: Low. Lowers the amount of content-usable stack space on Win64, but that number is already different on each platform so nobody should depend on it. [String/UUID change made/needed]: None.
Flags: needinfo?(shu)
Attachment #8555473 - Flags: approval-mozilla-aurora?
Comment on attachment 8555473 [details] [diff] [review] Tweak trusted stack buffer to give Win64 PGO more room. Aurora+
Attachment #8555473 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: