Closed
Bug 1492263
Opened 6 years ago
Closed 6 years ago
Add memory reporters for large unclassified allocations reported in bug 1425756
Categories
(Core :: Graphics: WebRender, enhancement, P2)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: bholley, Assigned: bholley)
References
Details
Attachments
(1 file)
In bug 1425756 comment 16, Asif provided a set of seven urls that, when loaded in different windows, generate > 100 megabytes of additional heap-unclassified. We should add memory reporters to get that number down.
Updated•6 years ago
|
Blocks: stage-wr-trains
Priority: -- → P1
Assignee | ||
Updated•6 years ago
|
Priority: P1 → P2
Assignee | ||
Comment 1•6 years ago
|
||
MozReview-Commit-ID: JGV7alMaCy0
Assignee | ||
Comment 2•6 years ago
|
||
The TextureCache free list showed up at ~1MB, but it would have added a fair amount of boilerplate to measure so I skipped it. All the other decent-sized WebRender allocations are either covered by these reporters or not accessible (like the shader program binaries or OS glyph rasterization caches).
Comment 3•6 years ago
|
||
Comment on attachment 9010507 [details]
Add more memory reporters.
Glenn Watson [:gw] has approved the revision.
Attachment #9010507 -
Flags: review+
Updated•6 years ago
|
Depends on: 1493097
See Also: → https://github.com/servo/webrender/pull/3097
Pushed by bholley@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/be7d67d74770
Add more memory reporters. r=gw
Comment 5•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•