linux-kernel - Re: [RFC/PATCHSET 0/6] perf tools: Support dynamic sort keys for tracepoints (v1)

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]

Date:	Tue, 15 Dec 2015 10:41:52 +0900
From:	Namhyung Kim <namhyung@...nel.org>
To:	David Ahern <dsahern@...il.com>
Cc:	Arnaldo Carvalho de Melo <arnaldo.melo@...il.com>,
	Ingo Molnar <mingo@...nel.org>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Jiri Olsa <jolsa@...hat.com>,
	LKML <linux-kernel@...r.kernel.org>,
	Steven Rostedt <rostedt@...dmis.org>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Andi Kleen <andi@...stfloor.org>,
	Wang Nan <wangnan0@...wei.com>
Subject: Re: [RFC/PATCHSET 0/6] perf tools: Support dynamic sort keys for
 tracepoints (v1)

Hi David and Arnaldo,

On Mon, Dec 14, 2015 at 03:32:06PM -0700, David Ahern wrote:
> On 12/14/15 10:47 AM, Arnaldo Carvalho de Melo wrote:
> >>With dynamic sort keys, you can use <event.field> as a sort key.  Those
> >>dynamic keys are checked and created on demand.  For instance, below is
> >>to sort by next_pid field on the same data file.
> >>
> >>   $ perf report -s comm,sched:sched_switch.next_pid --stdio
> >>   ...
> >>   # Overhead  Command            next_pid
> >>   # ........  ...............  ..........
> >>   #
> >>       21.23%  transmission-gt           0
> >>       20.86%  swapper               17773
> >>        6.62%  netctl-auto               0
> >>        5.25%  swapper                 109
> >>        5.21%  kworker/0:1H              0
> >>        1.98%  Xephyr                    0
> >>        1.98%  swapper                6524
> >>        1.98%  swapper               27478
> >>        1.37%  swapper               27476
> >>        1.17%  swapper                 233
> >>
> >>Multiple dynamic sort keys are also supported:
> >>
> >>   $ perf report -s comm,sched:sched_switch.next_pid,sched:sched_switch.next_comm --stdio
> >>   ...
> >>   # Overhead  Command            next_pid         next_comm
> >>   # ........  ...............  ..........  ................
> >>   #
> >>       20.86%  swapper               17773   transmission-gt
> >>        9.64%  transmission-gt           0         swapper/0
> >>        9.16%  transmission-gt           0         swapper/2
> >>        5.25%  swapper                 109      kworker/0:1H
> >>        5.21%  kworker/0:1H              0         swapper/0
> >>        2.14%  netctl-auto               0         swapper/2
> >>        1.98%  netctl-auto               0         swapper/0
> >>        1.98%  swapper                6524            Xephyr
> >>        1.98%  swapper               27478       netctl-auto
> >>        1.78%  transmission-gt           0         swapper/3
> >>        1.53%  Xephyr                    0         swapper/0
> >>        1.29%  netctl-auto               0         swapper/1
> >>        1.29%  swapper               27476       netctl-auto
> >>        1.21%  netctl-auto               0         swapper/3
> >>        1.17%  swapper                 233    irq/33-iwlwifi
> >>
> >>Note that pid 0 exists for each cpu so have comm of 'swapper/N'.
> >
> >>This is available on 'perf/dynamic-sort-v1' branch in my tree
> >>
> >>   git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git
> >>
> >>Any comments are welcome, thanks!
> >>Namhyung
> >
> >I'll look at the patches for style, but the idea is so nice and natural
> >I thought about blind merging it :-)

Hehe, thanks!


> 
> yes, that is a cool feature.
> 
> For scheduling tracepoints the analysis could be added to perf-sched to ease
> the burden of the command line syntax.

Yes, the existing tracepoints analysis command might use or wrap it.

For command line syntax, I think it'd be better to provide shorter way
to identify the events - like (non-ambiguous) substring match or
positional parameter (e.g. %1 for first event).

For example, the last example above can be:

  $ perf report -s comm,switch.next_pid,switch.next_comm

or

  $ perf report -s comm,%1.next_pid,%1.next_comm


Thanks,
Namhyung
--
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/