Bug 131132 - [webkitpy] perf profiler does not understand --call-graph argument
Summary: [webkitpy] perf profiler does not understand --call-graph argument
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Manuel Rego Casasnovas
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-02 15:12 PDT by Manuel Rego Casasnovas
Modified: 2014-04-03 02:55 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.57 KB, patch)
2014-04-02 15:14 PDT, Manuel Rego Casasnovas
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Manuel Rego Casasnovas 2014-04-02 15:12:12 PDT
New version of perf profiler (3.13.6 in Debian testing) do not understand --call-graph argument.

So when running perf-tests enabling the profiler you get the following error:
callchain: Unknown --call-graph option value: --output

 usage: perf record [<options>] [<command>]
    or: perf record [<options>] -- <command> [<options>]

        --call-graph <mode[,dump_size]>
                          setup and enables call-graph (stack chain/backtrace) recording: fp dwarf

Using -g seems enough (from the manual):
       -g
           Enables call-graph (stack chain/backtrace) recording.
Comment 1 Manuel Rego Casasnovas 2014-04-02 15:14:59 PDT
Created attachment 228433 [details]
Patch
Comment 2 Zan Dobersek 2014-04-03 00:26:25 PDT
Looks OK.

Ryosuke, can you please give this a look?
Comment 3 Ryosuke Niwa 2014-04-03 01:37:46 PDT
Comment on attachment 228433 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=228433&action=review

> Tools/ChangeLog:8
> +        New versions of perf profiler have changed the argument from --call-graph to -g.

Should we check the version?  Or is -g backward compatible?
Comment 4 Manuel Rego Casasnovas 2014-04-03 02:25:10 PDT
(In reply to comment #3)
> (From update of attachment 228433 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=228433&action=review
> 
> > Tools/ChangeLog:8
> > +        New versions of perf profiler have changed the argument from --call-graph to -g.
> 
> Should we check the version?  Or is -g backward compatible?

-g has been there since a while ago, for example in previous versions you have:
       -g, --call-graph
           Do call-graph (stack chain/backtrace) recording.

So I hope it doesn't cause any issue.
Comment 5 WebKit Commit Bot 2014-04-03 02:55:26 PDT
Comment on attachment 228433 [details]
Patch

Clearing flags on attachment: 228433

Committed r166711: <http://trac.webkit.org/changeset/166711>
Comment 6 WebKit Commit Bot 2014-04-03 02:55:29 PDT
All reviewed patches have been landed.  Closing bug.