linux-kernel - Re: [PATCH v2 1/2] perf hists browser: Support horizontal scrolling with '<' and '>' key

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150810224603.GC27431@krava.redhat.com>
Date:	Tue, 11 Aug 2015 00:46:03 +0200
From:	Jiri Olsa <jolsa@...hat.com>
To:	Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:	Namhyung Kim <namhyung@...nel.org>, Ingo Molnar <mingo@...nel.org>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	LKML <linux-kernel@...r.kernel.org>,
	David Ahern <dsahern@...il.com>,
	Andi Kleen <andi@...stfloor.org>
Subject: Re: [PATCH v2 1/2] perf hists browser: Support horizontal scrolling
 with '<' and '>' key

On Mon, Aug 10, 2015 at 12:50:04PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Sun, Aug 09, 2015 at 01:21:39PM +0200, Jiri Olsa escreveu:
> > On Sun, Aug 09, 2015 at 07:35:42PM +0900, Namhyung Kim wrote:
> > > Hi Jiri,
> > > 
> > > On Sun, Aug 09, 2015 at 11:30:24AM +0200, Jiri Olsa wrote:
> > > > On Sun, Aug 09, 2015 at 05:21:01PM +0900, Namhyung Kim wrote:
> > > > > Currently perf TUI report browser doesn't support horizontal scrolling.
> > > > > So if terminal width is smaller than the actual contents, there's no way
> > > > > to see them.  This patch adds support horizontal movement by '<' and '>'
> > > > > keys.
> > > > 
> > > > nice, I wonder we could also have some way to scroll
> > > > by the column width.. it might be more eye friendly?
> > > > would need to try first ;-)
> > > 
> > > Good suggesion.  Please see below..
> > > 
> > > > 
> > > > I also tried it with SKIP_COLS_STEP=1, and it wasn't bad
> > > 
> > > OK.
> > > 
> > > > 
> > > > how about having several scroll step options? like:
> > > >      , . - SKIP_COLS_STEP=1
> > > >      < > - SKIP_COLS_STEP=10
> > > > CTRL-< > - SKIP_COLS_STEP=columns width
> > > 
> > > I tried to use CTRL but it seems not working.
> > > 
> > > 
> > > > 
> > > > we could also bind some of this to regular arrows
> > > > with SHIFT or CTRL, bacause it's probably the most
> > > > convenient binding for this
> > > 
> > > Yes, I agree with you.  But I don't know how to bind the arrow keys
> > > with SHIFT or CTRL to do the thing.  So I just changed that < > to
> > > make SKIP_COLS_STEP = column width.
> 
> > > From 8e1f0a8be36895f9f37df133dcc8020e123b76e9 Mon Sep 17 00:00:00 2001
> > > From: Namhyung Kim <namhyung@...nel.org>
> > > Date: Sun, 9 Aug 2015 19:25:32 +0900
> > > Subject: [PATCH] perf hists browser: Move to prev/next column start by '<' or
> > >  '>' keys
> > > 
> > > Jiri said that it'd be more eye-friendly if it can move by column
> > > widths.  So change the keys to do it rather than jumping 10 characters.
> > > Also add ',' and '.' keys which reside same position in the keyboard to
> > > move by 1 characters.
> > 
> > I like it ;-)
> 
> Humm, I don't, its not natural, I doubt anyone will think about using ,
> and . to move by one character right/left, ditto for <>, I think we
> should just recover <- and -> (left and right arrows) for navigation,
> leave ENTER to be what -> was doing and esc for what <- does, in fact it
> is already like that.

I would ;-) anyway I also think arrows are the best, but I thought we
dont want to break existing interface much

> 
> Probably using shift or control or alt + <- -> for moving one column at
> at time is enough, but humm, is there value in that?

it'd really help for perf mem output, which is quite wide.. and we probably
have more wide outputs, or users with narrow terminals ;-)

jirka
--
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/