Closed Bug 623072 Opened 10 years ago Closed 10 years ago
_MAX _MALLOC _BYTES usage
Recently, JSGC_MAX_MALLOC_BYTES was increased to 80mb. I added support to make this configurable. The original value was 32mb. The difference between 40 and 20 makes little difference in terms of memory usage, but does slow down ss. The following patch sets JSGC_MAX_MALLOC_BYTES to 40mbs.
Comment on attachment 501194 [details] [diff] [review] patch v.1 this patch will cause perf regressions over benchmarks due to more frequent gc as memory rises.
Attachment #501194 - Flags: review?(mark.finkle)
Comment on attachment 501194 [details] [diff] [review] patch v.1 So if I understand correctly, we are currently running with a high_water_mark = 32 and I don't see any negative effect on the Tsunspider talos test. Why not just keep it at 32?
default is 80.
(In reply to comment #4) > default is 80. high_water_mark? What about http://mxr.mozilla.org/mozilla-central/source/modules/libpref/src/init/all.js#599
mfinkle, the existing code, when 32 was set, would force the watermark to actually be 80 and the max allocation to be -1 (inf).
show should we just set this to 32 and try it?
Sounds fine. r+ on that change?
Comment on attachment 501194 [details] [diff] [review] patch v.1 r+ for setting it to 32 to start with
Attachment #501194 - Flags: review?(mark.finkle) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
doug, anything we need to look out for when testing this fix? stability? oom? perf?
memory usage should be lower when compared to builds without this change.
Ran a handful of startup tests, open tabs, and running extensions on on the 1/10 nightly vs fennec beta 3. The nightly seemed to consistently display lower memory usage (using process manager app). marking verified on Mozilla/5.0 (Android; Linux armv71; rv:2.0b9pre) Gecko/20110110 Firefox/4.0b9pre Fennec/4.0b4pre Can someone verify this against maemo builds?
How can be verified this on maemo?
You need to log in before you can comment on or make changes to this bug.