linux-kernel - Re: [PATCHSET 00/10] perf tools: Support dynamic sort keys for tracepoints (v3)

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: <20160105002238.GA667@sejong>
Date:	Tue, 5 Jan 2016 09:22:38 +0900
From:	Namhyung Kim <namhyung@...nel.org>
To:	Arnaldo Carvalho de Melo <acme@...nel.org>
CC:	Ingo Molnar <mingo@...nel.org>,
	Peter Zijlstra <peterz@...radead.org>,
	Jiri Olsa <jolsa@...hat.com>,
	LKML <linux-kernel@...r.kernel.org>,
	David Ahern <dsahern@...il.com>,
	Steven Rostedt <rostedt@...dmis.org>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Andi Kleen <andi@...stfloor.org>,
	Wang Nan <wangnan0@...wei.com>
Subject: Re: [PATCHSET 00/10] perf tools: Support dynamic sort keys for
 tracepoints (v3)

Hi Arnaldo,

On Mon, Jan 04, 2016 at 06:17:46PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Mon, Dec 21, 2015 at 11:26:43PM +0900, Namhyung Kim escreveu:
> > Hello,
> > 
> > This is an attempt to improve perf to deal with tracepoint events
> > better.  The perf tools can handle tracepoint events but perf report
> > on them is less useful since they're always sampled in a fixed
> > location and not provide event specific info.  We can use perf script
> > but I always wishes there's more convenient way to see the result.
> > 
> >  * changes in v3)
> >   - save trace_output for dynamic entries  (Jiri)
> >   - update field length for each entry
> 
> Thanks, applied, I made a few changes, related to libtraceevent function
> names, split a patch in two, please take a look at my perf/core branch,
> should all be ok.

Looks good to me, thanks!

> 
> Ah, please consider looking for field names directly, i.e. make this
> work for a kmem:kmalloc event:
> 
>     perf report -s comm,gfp_flags
> 
> :-)

This is supported already if the perf.data only has the kmem:kmalloc
event.

> 
> I think this can get mapped even when multiple events are available,
> i.e. use that for the events that have such a field.

Agreed.  Related events can have same field name and it'd be nice to
add all of them if possible.  I'll cook a patch for it.


> 
> It could even get more relaxed, like:
> 
>     perf report -s comm,flags
> 
> No?

Not sure.  I'd like to keep same behavior as other sort keys.  IOW it
currently finds sort keys using prefix match not substring.  For
example, 'sym' will be match to 'symbol' but 'bol' won't.  So matching
'flags' to 'gfp_flags' seems inconsistent.

In addition, it uses pevent_find_any_field() which uses strcmp()
internally.  That means the field name should be matched exactly.

However, from an usability perspective, I agree that it'd be nice to
support that kind of shortcuts.  What about adding a simple wildcard
style like:

  perf report -s comm,*_flags

or

  perf report -s comm,gfp*

?

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/