Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751964AbaAOMLb (ORCPT ); Wed, 15 Jan 2014 07:11:31 -0500 Received: from mx1.redhat.com ([209.132.183.28]:10016 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751672AbaAOML3 (ORCPT ); Wed, 15 Jan 2014 07:11:29 -0500 Date: Wed, 15 Jan 2014 13:10:44 +0100 From: Jiri Olsa To: Namhyung Kim Cc: Arnaldo Carvalho de Melo , Peter Zijlstra , Paul Mackerras , Ingo Molnar , Namhyung Kim , LKML , David Ahern Subject: Re: [PATCH/RFC] perf ui/tui: Show column header in hist browser Message-ID: <20140115121044.GA1669@krava.redhat.com> References: <1389762525-1795-1-git-send-email-namhyung@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1389762525-1795-1-git-send-email-namhyung@kernel.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 15, 2014 at 02:08:45PM +0900, Namhyung Kim wrote: > Add a line for showing column headers like --stdio. https://git.kernel.org/cgit/linux/kernel/git/jolsa/perf.git/commit/?h=perf/lock12&id=1a06d77629cb8870838132688f443b9ca1436026 ;-) I'll check your change jirka > > Signed-off-by: Namhyung Kim > --- > tools/perf/ui/browser.c | 4 +-- > tools/perf/ui/browsers/hists.c | 55 ++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 57 insertions(+), 2 deletions(-) > > diff --git a/tools/perf/ui/browser.c b/tools/perf/ui/browser.c > index d11541d4d7d7..441470ba8d2c 100644 > --- a/tools/perf/ui/browser.c > +++ b/tools/perf/ui/browser.c > @@ -278,8 +278,8 @@ void ui_browser__hide(struct ui_browser *browser) > static void ui_browser__scrollbar_set(struct ui_browser *browser) > { > int height = browser->height, h = 0, pct = 0, > - col = browser->width, > - row = browser->y - 1; > + col = browser->width, row = 0; > + > > if (browser->nr_entries > 1) { > pct = ((browser->index * (browser->height - 1)) / > diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c > index e0ab399d431d..5b94de48f23c 100644 > --- a/tools/perf/ui/browsers/hists.c > +++ b/tools/perf/ui/browsers/hists.c > @@ -25,6 +25,7 @@ struct hist_browser { > struct map_symbol *selection; > int print_seq; > bool show_dso; > + bool show_column_header; > float min_pcnt; > u64 nr_pcnt_entries; > }; > @@ -312,6 +313,57 @@ static void ui_browser__warn_lost_events(struct ui_browser *browser) > > static void hist_browser__update_pcnt_entries(struct hist_browser *hb); > > +static void hist_browser__column_header(struct hist_browser *browser) > +{ > + struct perf_hpp_fmt *fmt; > + struct sort_entry *se; > + struct hists *hists = browser->hists; > + bool show = browser->show_column_header; > + unsigned int width; > + > + pthread_mutex_lock(&ui__lock); > + > + if (!show) > + goto out; > + > + ui_browser__gotorc(&browser->b, 0, 0); > + > + perf_hpp__for_each_format(fmt) { > + char buf[128]; > + struct perf_hpp hpp = { > + .buf = buf, > + .size = sizeof(buf), > + .ptr = hists_to_evsel(hists), > + }; > + > + width = fmt->width(fmt, &hpp); > + fmt->header(fmt, &hpp); > + > + slsmg_printf(" %-*s", width, hpp.buf); > + } > + > + list_for_each_entry(se, &hist_entry__sort_list, list) { > + if (se->elide) > + continue; > + > + width = strlen(se->se_header); > + if (!hists__new_col_len(hists, se->se_width_idx, width)) > + width = hists__col_len(hists, se->se_width_idx); > + > + slsmg_printf(" %-*s", width, se->se_header); > + } > + > + /* > + * We just called ui_browser__refresh_dimensions(). > + * Update it to honor a new column header line. > + */ > + browser->b.y++; > + browser->b.height--; > + > +out: > + pthread_mutex_unlock(&ui__lock); > +} > + > static int hist_browser__run(struct hist_browser *browser, const char *ev_name, > struct hist_browser_timer *hbt) > { > @@ -331,6 +383,8 @@ static int hist_browser__run(struct hist_browser *browser, const char *ev_name, > "Press '?' for help on key bindings") < 0) > return -1; > > + hist_browser__column_header(browser); > + > while (1) { > key = ui_browser__run(&browser->b, delay_secs); > > @@ -1198,6 +1252,7 @@ static struct hist_browser *hist_browser__new(struct hists *hists) > browser->b.refresh = hist_browser__refresh; > browser->b.seek = ui_browser__hists_seek; > browser->b.use_navkeypressed = true; > + browser->show_column_header = true; > } > > return browser; > -- > 1.7.11.7 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/