Allow running of SPS profiling during capture

RESOLVED FIXED

Status

Testing Graveyard
Eideticker
RESOLVED FIXED
6 years ago
7 months ago

People

(Reporter: wlach, Unassigned)

Tracking

Trunk
All
Android
Dependency tree / graph

Details

(Whiteboard: [eideticker:p1])

BenWa wrote a lightweight profiler called SPS, which also has a mobile variant:

https://github.com/bgirard/SPS_helper

It would be useful for eideticker to gather this information out-of-band so we could correlate it with the capture.
OS: Linux → Android
Hardware: x86_64 → All
Whiteboard: [eideticker:p1]
Here's a sample of what mashing these two tools together would look like:
http://people.mozilla.org/~bgirard/cleopatra/?report=3f2f2196cc110a813b8c2dd30145133d78d57c6d

I would keep the dashboard the same with a new 'View profile' link if the profile data could be extracted. If it can't for whatever we reason eideticker would ignore profiling and keep working as it normally does.
Here's a profile collected using my latest posted patch. The result is a 124KB zip per run.

William can you help me test this tomorrow with the capture. Once we get a few real runs with capture + profiling I'll do the follow up tasks:
1) Add a profile link to the dashboard
2) Add the capture video inside the profile like shown in Comment 1.

Updated

6 years ago
Blocks: 788400

Updated

6 years ago
Depends on: 788680
Here's a real profile pair:
http://people.mozilla.com/~bgirard/cleopatra/?zippedProfile=profiles/sps-profile-1346882045.2.zip&videoCapture=videos/video-1346882045.19.webm

Note that we're still missing a few patches to sync them up properly and I will likely add some video controls.
Let's get the video synchronization (bug 789597) done, then mark this as finished.
Depends on: 789597
Video synchronization is working (but will need more UI love to support zooming)!

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Updated

7 months ago
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.