915857 - Check-in merge-profiles.py

Status: RESOLVED FIXED

(Core :: Gecko Profiler, defect)

Description

[Benoit Girard (:BenWa)]

Attachment: patch

Sampled merged profile:
http://people.mozilla.org/~bgirard/cleopatra/#report=65cfe559f92bf66da8aee49d147b2dfc26f29e40

Comment 1

[Vladan Djeric (:vladan)]

Comment on attachment 803977 [details] [diff] [review]
patch

- Add a comment to the top of the file about what the script does and how it does it

>+    filedata = []
>+    symtable = dict()

Nit: camel case

>+        match = re.match('profile_([0-9]*)_(.*)\.sym', fname)

I think you meant 'profile_([0-9]+)_(.+)\.sym'

>+        if match is None:
>+            raise Exception("Filename '" + fname + "' doesn't match expected pattern")

Nit: consistent string quotes
>+                    frame['location'] = pid + frame['location']

Hmm, will you be able to distinguish (pid=12, location=3) from (pid=1, location=23)

>+    result = dict()
>+    result['profileJSON'] = dict()
>+    result['profileJSON']['meta'] = meta
>+    result['profileJSON']['libs'] = libs
>+    result['profileJSON']['threads'] = threads
>+    result['symbolicationTable'] = symtable
>+    result['format'] = 'profileJSONWithSymbolicationTable,1'

You could also do this in one or two lines, e.g. result = {'profileJSON': {'meta': meta, 'libs': libs, 'threads': threads, .. }. Just a suggestion

>+print "Usage: merge-profile.py profile_....sym profile_....sym > merged.sym"

print "Usage: merge-profile.py profile_<pid1>_<pname1>.sym profile_<pid2>_<pname2>.sym > merged.sym"

Comment 2

[Benoit Girard (:BenWa)]

Attachment: patch

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/ed734d586569

Comment 4

[Mike Habicher [:mikeh] (high bugzilla latency)]

Attachment: patch (b2g18)

merge-profiles.py, and enough of the changeset from bug 788022 to add 'startTime' logging to the profilers.

NOTE: This flag is now for security issues only. Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 914658
User impact if declined: unable to merge profiles
Testing completed: run on local builds
Risk to taking this patch (and alternatives if risky): minimal
String or UUID changes made by this patch: none