[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20151013023050.GQ14409@kernel.org>
Date: Mon, 12 Oct 2015 23:30:50 -0300
From: Arnaldo Carvalho de Melo <acme@...nel.org>
To: Namhyung Kim <namhyung@...nel.org>
Cc: Ingo Molnar <mingo@...nel.org>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Jiri Olsa <jolsa@...hat.com>,
LKML <linux-kernel@...r.kernel.org>,
David Ahern <dsahern@...il.com>
Subject: Re: [PATCH] perf hists browser: Add 'm' key for context menu display
Em Tue, Oct 13, 2015 at 09:02:01AM +0900, Namhyung Kim escreveu:
> With horizontal scrolling, the left/right arrow keys are used to
> scroll columns and ENTER/ESC keys are used to enter/exit menu.
> However if callchain is recorded, the ENTER key is used to toggle
> callchain expansion so there's no way to display menu. Use 'm' key to
> display the menu for this case.
Thanks, applied.
- Arnaldo
> Signed-off-by: Namhyung Kim <namhyung@...nel.org>
> ---
> tools/perf/ui/browsers/hists.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c
> index 9b7346a881cf..c8ec9bdb9c04 100644
> --- a/tools/perf/ui/browsers/hists.c
> +++ b/tools/perf/ui/browsers/hists.c
> @@ -1769,6 +1769,7 @@ static int perf_evsel__hists_browse(struct perf_evsel *evsel, int nr_events,
> "E Expand all callchains\n" \
> "F Toggle percentage of filtered entries\n" \
> "H Display column headers\n" \
> + "m Display context menu\n" \
> "S Zoom into current Processor Socket\n" \
>
> /* help messages are sorted by lexical order of the hotkey */
> @@ -1934,6 +1935,7 @@ static int perf_evsel__hists_browse(struct perf_evsel *evsel, int nr_events,
> continue;
> case K_ENTER:
> case K_RIGHT:
> + case 'm':
> /* menu */
> break;
> case K_ESC:
> --
> 2.6.1
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists