Closed
Bug 1119363
Opened 9 years ago
Closed 9 years ago
Intermittent test_bug732665.xul | Chrome should be invokable from content script with an exhausted stack: 10
Categories
(Core :: XPConnect, defect)
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)
1.48 KB,
patch
|
bholley
:
review+
lmandel
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
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
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 14•9 years ago
|
||
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)
Assignee | ||
Comment 15•9 years ago
|
||
Not this week, but I'll put it on my queue of things to look at.
Comment 16•9 years ago
|
||
Thanks shu.
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 31•9 years ago
|
||
Trying 140k instead of the current 120k: https://treeherder.mozilla.org/#/jobs?repo=try&revision=348274581561
Assignee | ||
Comment 32•9 years ago
|
||
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.
Assignee | ||
Updated•9 years ago
|
Attachment #8555473 -
Flags: review?(bobbyholley)
Updated•9 years ago
|
Attachment #8555473 -
Flags: review?(bobbyholley) → review+
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(shu)
Comment 33•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fe0a5e20d1f4
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Reporter | ||
Updated•9 years ago
|
Assignee: nobody → shu
status-firefox36:
--- → unaffected
status-firefox37:
--- → affected
status-firefox38:
--- → fixed
status-firefox-esr31:
--- → unaffected
Reporter | ||
Comment 34•9 years ago
|
||
Please request Aurora approval on this when you get a chance.
Flags: needinfo?(shu)
Assignee | ||
Comment 35•9 years ago
|
||
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 36•9 years ago
|
||
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+
Reporter | ||
Comment 37•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/220148363987
You need to log in
before you can comment on or make changes to this bug.
Description
•