Closed
Bug 1204167
Opened 9 years ago
Closed 9 years ago
Make a generic RAII class for pushing and popping sps psuedo frames
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: fitzgen, Assigned: fitzgen)
References
Details
Attachments
(1 file)
(deleted),
patch
|
djvj
:
review+
|
Details | Diff | Splinter Review |
So that we can add psuedo frames for a bunch of spidermonkey/js stuff.
SPSEntryMarker is close to what we want, but currently is specialized for js::RunScript
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8660282 -
Flags: review?(kvijayan)
Assignee | ||
Comment 2•9 years ago
|
||
Kannan, one issue I have with this patch is that the category doesn't seem to be picked up by the time the devtools get their hands on the samples and the category is always "other". Any ideas why this isn't making it all the way through the pipeline?
Comment 3•9 years ago
|
||
The category is getting dropped when the pseudostack entries are recorded into the sample buffer. See "tools/profiler/core/GeckoSampler.cpp" line 1289. That file adds a pseudo-stack entry to the sample buffer.
The format of a pseudostack entry on the profile buffer doesn't include the category information. That formatting needs to be changed to allow for category info to be embedded, then the code above needs to be changed to emit the category info into the profile buffer, and lastly, "tools/profiler/core/ProfileEntry.cpp" line 610 (method ProfileBuffer::StreamSamplesToJSON) needs to be fixed up to parse that and emit the appropriate JSON for it.
Comment 4•9 years ago
|
||
If you want, I can take you through that code in more detail over videochat.
Updated•9 years ago
|
Attachment #8660282 -
Flags: review?(kvijayan) → review+
Assignee | ||
Comment 5•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•