Closed
Bug 1227535
Opened 9 years ago
Closed 9 years ago
Add ZoneAllocPolicy and use it to attribute module memory usage to the zone
Categories
(Core :: JavaScript: GC, defect)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
mozilla45
| Tracking | Status | |
|---|---|---|
| firefox45 | --- | fixed |
People
(Reporter: jonco, Assigned: jonco)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
We currently don't have an alloc policy for malloc memory that updates the counters for a zone. This is useful malloc memory associated with a particular GC thing that will be freed when the GC thing dies.
Add ZoneAllocPolicy and use it for module hash tables.
Attachment #8691379 -
Flags: review?(terrence)
|
||
Comment 1•9 years ago
|
||
Comment on attachment 8691379 [details] [diff] [review]
use-zone-alloc
Review of attachment 8691379 [details] [diff] [review]:
-----------------------------------------------------------------
Nice!
Attachment #8691379 -
Flags: review?(terrence) → review+
I had to back this out for static build bustage: https://hg.mozilla.org/integration/mozilla-inbound/rev/8ea3da3e71db
https://treeherder.mozilla.org/logviewer.html#?job_id=17762228&repo=mozilla-inbound
Flags: needinfo?(jcoppeard)
|
Assignee | |
Updated•9 years ago
|
Flags: needinfo?(jcoppeard)
|
||
Comment 5•9 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in
before you can comment on or make changes to this bug.
Description
•