Re: [PATCH 14/14] perf script: show overhead events

From: Jiri Olsa
Date: Wed Nov 23 2016 - 18:34:02 EST


On Wed, Nov 23, 2016 at 04:44:52AM -0500, kan.liang@xxxxxxxxx wrote:

SNIP

> +}
> +
> static void sig_handler(int sig __maybe_unused)
> {
> session_done = 1;
> @@ -1287,6 +1319,8 @@ static int __cmd_script(struct perf_script *script)
> }
> if (script->show_switch_events)
> script->tool.context_switch = process_switch_event;
> + if (script->show_overhead)
> + script->tool.overhead = process_overhead_event;
>
> ret = perf_session__process_events(script->session);
>
> @@ -2172,6 +2206,8 @@ int cmd_script(int argc, const char **argv, const char *prefix __maybe_unused)
> "Show the mmap events"),
> OPT_BOOLEAN('\0', "show-switch-events", &script.show_switch_events,
> "Show context switch events (if recorded)"),
> + OPT_BOOLEAN('\0', "show-overhead", &script.show_overhead,
> + "Show overhead events"),

please add the '-events' suffix as for the mmap and task

thanks,
jirka