Closed
Bug 1820136
Opened 3 years ago
Closed 3 years ago
Use same value for jemalloc moz_set_max_dirty_page_modifier in JS shell as for foreground content processes
Categories
(Core :: JavaScript Engine, task, P1)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
112 Branch
| Tracking | Status | |
|---|---|---|
| firefox112 | --- | fixed |
People
(Reporter: jandem, Assigned: jandem)
References
Details
(Whiteboard: [sp3])
Attachments
(1 file)
Olli's changes in bug 1815069 improved our browser performance a lot. We should use the same jemalloc settings for the JS shell so performance there is more similar to a foreground content process.
We could also add a testing function to the JS shell to allow changing this value, for both fuzzing and performance testing.
| Assignee | ||
Updated•3 years ago
|
Summary: Use same jemalloc setting for max dirty pages in JS shell as for foreground content processes → Use same jemalloc setting for moz_set_max_dirty_page_modifier in JS shell as for foreground content processes
| Assignee | ||
Updated•3 years ago
|
Summary: Use same jemalloc setting for moz_set_max_dirty_page_modifier in JS shell as for foreground content processes → Use same value for jemalloc moz_set_max_dirty_page_modifier in JS shell as for foreground content processes
| Assignee | ||
Comment 1•3 years ago
|
||
This matches the behavior for foreground content processes since bug 1815069.
Also adds a testing function for fuzzing and performance testing.
Updated•3 years ago
|
Whiteboard: [sp3]
Updated•3 years ago
|
See Also: → https://mozilla-hub.atlassian.net/browse/SP3-32
Pushed by jdemooij@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d929730d9d23
Use moz_set_max_dirty_page_modifier and jemalloc_free_dirty_pages for the JS shell to match the browser. r=smaug,jonco
Updated•3 years ago
|
Severity: -- → N/A
Priority: -- → P1
Comment 3•3 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox112:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•