Closed
Bug 1148836
Opened 10 years ago
Closed 2 years ago
Allocations over time view
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect, P3)
DevTools
Performance Tools (Profiler/Timeline)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: jsantell, Unassigned)
References
Details
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review |
From bug 1148822, quick attempt at rendering allocations over time. Don't know if this is useful, or if it should be for performance tool or memory tool, but just putting it here.
Reporter | ||
Comment 1•10 years ago
|
||
and here is the hacky proof of concept
Comment 2•10 years ago
|
||
What are the units here? What is the x-axis? What is the y-axis?
We talked about adding an allocation heat map in London, but I don't know if we ever filed bugs.
Comment 3•10 years ago
|
||
What questions are you trying to answer with this graph?
Reporter | ||
Comment 4•10 years ago
|
||
Its number of allocations in time domain; afaict, can't get the size of these allocations currently with the memory actor. Its grouped in chunks based off of polling, currently at 200ms, so multiplying alloc counts by 5 we'd have allocs per second for allocation pressure, though might be better represented as KB/s.
What is it trying to answer? Heavy allocation actions, spawned from ideas in bug 1148822 -- I do not know if this is useful yet, or made redundant if we make the allocations flame chart/graph not in the time domain, but wanted a home for the feature regardless for now
Comment 5•10 years ago
|
||
I think we want to graph rate of allocations, not the allocations themselves. Its ok to alloc, but if you alloc too fast, that's what forces a blocking GC.
Should probably overlay GC markers as well once bug 1137527 lands.
Comment 6•10 years ago
|
||
Just to nitpick, i'd like to see this part of the same memory graph, not as a bar graph underneath. Too much wasted space.
Reporter | ||
Comment 7•10 years ago
|
||
(In reply to Nick Fitzgerald [:fitzgen] from comment #5)
> I think we want to graph rate of allocations, not the allocations
> themselves.
Same thing, just in line graph form, I think is what you're asking for?
(In reply to Victor Porof [:vporof][:vp] from comment #6)
> Just to nitpick, i'd like to see this part of the same memory graph, not as
> a bar graph underneath. Too much wasted space.
Agreed -- again this is a quick ugly hack. Especially if this was in a line graph, would be pretty straight forward to layer them both in the same graph (although the LineGraphWidget atm does not support that, and would need to do something about the legends/labels min/max marks too)
Reporter | ||
Comment 8•10 years ago
|
||
And probably should be rate of allocation size, not by count, right? What's more likely to cause a GC, lots of small objects, or a handful of large objects? Would you rather fight 100 duck-sized horses or one horse-sized duck?
Comment 9•10 years ago
|
||
(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #8)
> And probably should be rate of allocation size, not by count, right? What's
> more likely to cause a GC, lots of small objects, or a handful of large
> objects? Would you rather fight 100 duck-sized horses or one horse-sized
> duck?
Yes, but that will have to wait on bug 1068988
Reporter | ||
Updated•9 years ago
|
Blocks: perf-allocations
Reporter | ||
Updated•9 years ago
|
Reporter | ||
Comment 10•9 years ago
|
||
Is this valuable to have in realtime? If not, then we can move forward with bug 1196862 and serve allocations after the recording is finished, which would significantly increase speed, I think, as this is a lot of data going over the wire.
Flags: needinfo?(nfitzgerald)
Reporter | ||
Updated•9 years ago
|
No longer blocks: perf-tools-fx43
Reporter | ||
Updated•9 years ago
|
Assignee: jsantell → nobody
Status: ASSIGNED → NEW
Comment 12•9 years ago
|
||
Triaging. Filter on ADRENOCORTICOTROPIC (yes).
OS: Mac OS X → Unspecified
Priority: -- → P3
Hardware: x86 → Unspecified
Version: 37 Branch → unspecified
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•2 years ago
|
Severity: normal → S3
Comment 13•2 years ago
|
||
This report is related to the old DevTools profiler implementation, which has been removed. The Performance panel is now based on Firefox Profiler at https://profiler.firefox.com/
Closing as invalid
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•