Closed
Bug 1259205
Opened 8 years ago
Closed 2 years ago
Self Samples, Self Cost and Self Time columns are missing data in performance profiler Call tree view.
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect, P2)
DevTools
Performance Tools (Profiler/Timeline)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: jujjyl, Unassigned)
Details
(Whiteboard: [btpp-fix-later])
Attachments
(1 file)
369.74 KB,
image/png
|
Details |
STR: 1. Visit https://dl.dropboxusercontent.com/u/40949268/emcc/10kCubes/10kCubes.html 2. Run a performance profile. 3. Open the "Call Tree" view. 4. Expand some of the frames bottom up. Observed: All non-leaf nodes have zero in their "Self Samples", "Self Cost", and "Self Time" columns. See the attached screenshot with cells in light red background.
Comment 1•8 years ago
|
||
If none of those frames were ever sampled as the youngest stack, then they *should* have self = 0.
Comment 2•8 years ago
|
||
(In reply to Nick Fitzgerald [:fitzgen] [⏰PDT; UTC-7] from comment #1) > If none of those frames were ever sampled as the youngest stack, then they > *should* have self = 0. I'm looking at triaging this bug, this looks like expected behavior, correct?
Flags: needinfo?(nfitzgerald)
Reporter | ||
Comment 3•8 years ago
|
||
Nick, you are probably right. Coming back to this, trying to understand my source of confusion, I did a small test: My test application is as follows: https://dl.dropboxusercontent.com/u/40949268/emcc/simulate_work/simulate_work.cpp which is a requestAnimationFrame-based loop, where each tick calls in a linear fashion functions tick() -> foo() -> bar(), where foo() works for 50 milliseconds, and bar() works for 200 milliseconds (4x the time of foo()). Here's the compiled page: https://dl.dropboxusercontent.com/u/40949268/emcc/simulate_work/simulate_work.html Looking at the profiler in top-down mode shows as expected: https://dl.dropboxusercontent.com/u/40949268/emcc/simulate_work/simulate_work_top_down.png tick to foo to bar, where bar takes up 4x time of foo. However looking at the bottom up view in this page is a bit odd, not sure if this is intended? https://dl.dropboxusercontent.com/u/40949268/emcc/simulate_work/simulate_work_bottom_up.png The function bar() does have 31 self samples, and the function foo() does have 11 self samples, but they don't show up in the non-leaf nodes, but only when those functions are presented as leaf nodes. Nick, how does this look to you?
Comment 4•8 years ago
|
||
Yeah that does look funky -- there should be some self time there.
Flags: needinfo?(nfitzgerald)
Updated•8 years ago
|
Priority: -- → P2
Whiteboard: [btpp-fix-later]
Updated•6 years ago
|
Product: Firefox → DevTools
Comment 5•2 years ago
|
||
Now that we replaced the old performance panel by the Firefox Profiler (see Bug 1668219) we're closing bugs related to this. If you see similar issues on the new profiler, please file another bug, and we'll be happy to have a look.
Filter on MASSCLOSEOLDPERFTOOLBUGS.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•