lkml.org 
[lkml]   [2013]   [Jan]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[tip:perf/core] perf script: Don' t display trace info when invoking scripts
    Commit-ID:  bdb71db29d50b61f45459b8facfb876768e8cacc
    Gitweb: http://git.kernel.org/tip/bdb71db29d50b61f45459b8facfb876768e8cacc
    Author: Tom Zanussi <tom.zanussi@linux.intel.com>
    AuthorDate: Fri, 18 Jan 2013 13:51:26 -0600
    Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
    CommitDate: Thu, 24 Jan 2013 16:40:52 -0300

    perf script: Don't display trace info when invoking scripts

    Only display the trace info if using the default event display. When
    invoking scripts we assume they have complete control of what's
    displayed so we shouldn't unconditionally display the trace info, and
    when generating scripts we don't expect to see trace info obscuring the
    output message.

    Signed-off-by: Tom Zanussi <tom.zanussi@linux.intel.com>
    Link: http://lkml.kernel.org/r/12ec084ef2870178915c907d16cd1dfa19fbb39e.1358527965.git.tom.zanussi@linux.intel.com
    Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
    ---
    tools/perf/builtin-script.c | 3 ++-
    1 file changed, 2 insertions(+), 1 deletion(-)

    diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
    index fee4c9a..92d4658 100644
    --- a/tools/perf/builtin-script.c
    +++ b/tools/perf/builtin-script.c
    @@ -1475,7 +1475,8 @@ int cmd_script(int argc, const char **argv, const char *prefix __maybe_unused)
    return -1;
    }

    - perf_session__fprintf_info(session, stdout, show_full_info);
    + if (!script_name && !generate_script_lang)
    + perf_session__fprintf_info(session, stdout, show_full_info);

    if (!no_callchain)
    symbol_conf.use_callchain = true;

    \
     
     \ /
      Last update: 2013-01-25 13:45    [W:4.391 / U:0.220 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site