Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1507659

Re: [PATCH] perf hist browser: Fix hierarchy column counts

From Markus Trippelsdorf <markus@trippelsdorf.de>
Newsgroups linux.kernel
Subject Re: [PATCH] perf hist browser: Fix hierarchy column counts
Date 2016-10-24 21:40 +0200
Message-ID <svTge-3ya-13@gated-at.bofh.it> (permalink)
References <svQim-1Di-59@gated-at.bofh.it> <svQV4-26N-15@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 2016.10.24 at 19:00 +0200, Markus Trippelsdorf wrote:
> On 2016.10.25 at 01:21 +0900, Namhyung Kim wrote:
> > The perf report/top on TUI supports horizontal scrolling using LEFT and
> > RIGHT keys.  But it calculate the number of columns incorrectly when
> > hierarchy mode is enabled so that keep pressing RIGHT key can make the
> > output disappeared.  In the hierarchy mode, all sort keys are collapsed
> > into a single column, so it needs to be applied when calculating column
> > numbers.
> 
> Thanks, this is much better now. 
> 
> But it doesn't scroll wide enough. In "perf top" when I press the RIGHT
> key the symbols are shown on the hard left side. In "perf top --hierarchy",
> when I press the RIGHT key, there are 5 empty spaces on the left hand
> side that should be deleted.

It might be a good idea to keep the + and - signs on the left hand side
when scrolling horizontally like "perf report -g" does.

-- 
Markus

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread


Thread

[PATCH] perf hist browser: Fix hierarchy column counts Namhyung Kim <namhyung@kernel.org> - 2016-10-24 18:30 +0200
  Re: [PATCH] perf hist browser: Fix hierarchy column counts Markus Trippelsdorf <markus@trippelsdorf.de> - 2016-10-24 19:10 +0200
    Re: [PATCH] perf hist browser: Fix hierarchy column counts Markus Trippelsdorf <markus@trippelsdorf.de> - 2016-10-24 21:40 +0200

csiph-web