Closed
Bug 995165
Opened 11 years ago
Closed 11 years ago
ARM v6 GC settings should be changed to prevent triggering a loop of multiple GCs.
Categories
(Core :: JavaScript: GC, defect)
Tracking
()
RESOLVED
FIXED
mozilla31
Tracking | Status | |
---|---|---|
firefox31 | --- | affected |
People
(Reporter: nbp, Assigned: nbp)
References
Details
Attachments
(1 file)
2.58 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
The current GC settings which are used on low-memory Android (armv6 devices) [1] are damn to low and they are more likely to cause GC-loops.
The reason being that if all the heap is live, we would push the GC trigger by these multipliers, but the factor used to set the trigger is based on the quantity of live objects to which we apply a multiplier of 0.85 / 0.9 [2]. This trigger multiplier implies that:
javascript.options.mem.gc_low_frequency_heap_growth
should be higher than 112 (> 1 / 0.9)
javascript.options.mem.gc_high_frequency_heap_growth_max &
javascript.options.mem.gc_high_frequency_heap_growth_min
should be higher than 118 (> 1 / 0.85)
As we are changing these GC settings we should probably assert/warn about this issue.
[1] http://dxr.mozilla.org/mozilla-central/source/mobile/android/app/mobile.js?from=mobile.js#415,418
[2] http://dxr.mozilla.org/mozilla-central/source/js/src/jsgc.cpp#2247
Assignee | ||
Updated•11 years ago
|
tracking-fennec: --- → ?
status-firefox31:
--- → affected
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8405446 -
Flags: review?(terrence)
Assignee | ||
Updated•11 years ago
|
Blocks: 783912
Summary: ARm v6 GC settings should be changed to prevent triggering a multiple GCs in a loop. → ARM v6 GC settings should be changed to prevent triggering a loop of multiple GCs.
Comment 2•11 years ago
|
||
Comment on attachment 8405446 [details] [diff] [review]
Fix Armv6 GC settings
Review of attachment 8405446 [details] [diff] [review]:
-----------------------------------------------------------------
r=me Great catch!
Attachment #8405446 -
Flags: review?(terrence) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Comment 5•11 years ago
|
||
We don't need to track for fennec, but if you want to uplift ask for approval.
tracking-fennec: ? → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•