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)
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•10 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•10 years ago
|
||
Not this week, but I'll put it on my queue of things to look at.
Comment 16•10 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•10 years ago
|
||
Trying 140k instead of the current 120k:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=348274581561
Assignee | ||
Comment 32•10 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•10 years ago
|
Attachment #8555473 -
Flags: review?(bobbyholley)
Updated•10 years ago
|
Attachment #8555473 -
Flags: review?(bobbyholley) → review+
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(shu)
Comment 33•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Reporter | ||
Updated•10 years ago
|
Assignee: nobody → shu
status-firefox36:
--- → unaffected
status-firefox37:
--- → affected
status-firefox38:
--- → fixed
status-firefox-esr31:
--- → unaffected
Reporter | ||
Comment 34•10 years ago
|
||
Please request Aurora approval on this when you get a chance.
Flags: needinfo?(shu)
Assignee | ||
Comment 35•10 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•10 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•10 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•