linux-kernel - Re: [GIT PULL] Re: [PATCH 04/27] perf list: Add support for listing only json events

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] [day] [month] [year] [list]

Message-ID: <20161017144959.GA24104@gmail.com>
Date:   Mon, 17 Oct 2016 16:49:59 +0200
From:   Ingo Molnar <mingo@...nel.org>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:     Andi Kleen <ak@...ux.intel.com>, linux-kernel@...r.kernel.org,
        Linux Weekly News <lwn@....net>,
        Jiri Olsa <jolsa@...nel.org>,
        Arnaldo Carvalho de Melo <acme@...hat.com>,
        Peter Zijlstra <a.p.zijlstra@...llo.nl>,
        Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com>
Subject: Re: [GIT PULL] Re: [PATCH 04/27] perf list: Add support for listing
 only json events


* Arnaldo Carvalho de Melo <acme@...nel.org> wrote:

> Em Mon, Oct 17, 2016 at 06:57:56AM -0700, Andi Kleen escreveu:
> > > What we do know about the CPU vendor event names is that they are vendor specific, 
> > > and that's a pretty stable property. So my suggestion would be to simply make it:
> > > 
> > > 	perf list vendor
> > > 
> > > with perhaps add aliases as well:
> > > 
> > > 	perf list model
> > > 	perf list cpu-model
> > > 
> > > ... and 'perf list hardware' should probably also list all the vendor specific 
> > > hardware events as well.
> > 
> > Sure can change it.
> > 
> > > 
> > > Could we please work a bit more on making vendor specific hardware event handling 
> > > more usable to people who don't care about internals?
> > 
> > Do you prefer fully new patches, or are incremential patches ok? 
> 
> I would like for you to base of my perf/urgent branch as of now, that
> has those two patches yanked out so that the others can go thru.
> 
> Ingo, please consider pulling from the perf-urgent-for-mingo-20161017
> tag, just signed and pushed, it has:
> 
> tag perf-urgent-for-mingo-20161017
> Tagger: Arnaldo Carvalho de Melo <acme@...hat.com>
> Date:   Mon Oct 17 11:26:10 2016 -0300
> 
> perf/urgent fixes:
> 
> - Fix handling of numa nodes in perf.data files (Jiri Olsa)
> 
> - Fix scrolling when refreshing 'perf top --tui --hierarchy' entries
>   (Namhyung Kim)
> 
> - Fix building of JIT support on Ubuntu 16.04 (Anton Blanchard)
> 
> - Fix handling of events including .c and .o, that were being treated as
>   BPF scripts instead of vendor ones (Wang Nan)
> 
> Signed-off-by: Arnaldo Carvalho de Melo <acme@...hat.com>

Pulled, thanks a lot Arnaldo!

	Ingo