linux-kernel - Re: [PATCHv2 0/6] perf, tool: event parsing enhancements/fixes

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: <20120426101030.GA1659@m.brq.redhat.com>
Date:	Thu, 26 Apr 2012 12:10:30 +0200
From:	Jiri Olsa <jolsa@...hat.com>
To:	Namhyung Kim <namhyung.kim@....com>
Cc:	acme@...hat.com, a.p.zijlstra@...llo.nl, mingo@...e.hu,
	paulus@...ba.org, cjashfor@...ux.vnet.ibm.com, fweisbec@...il.com,
	linux-kernel@...r.kernel.org, eranian@...gle.com,
	robert.richter@....com
Subject: Re: [PATCHv2 0/6] perf, tool: event parsing enhancements/fixes

On Thu, Apr 26, 2012 at 03:10:21PM +0900, Namhyung Kim wrote:
> Hi,
> 
> On Wed, 25 Apr 2012 18:24:56 +0200, Jiri Olsa wrote:
> > Hi,
> > adding some enhancements/fixies for perf event parsing.
> >
> > v2 changes:
> >    - rebased to current tip tree
> >    - 2/6 - ommitting pmu tests in case there's no pmu sysfs record
> 
> Nice! It bothered me somewhat so I was thinking about fixing it.
> I guess we have a same problem on x86 rdpmc test too?

right.. I haven't seen that code, so not sure how hard the fix would be

jirka

> 
> Thanks,
> Namhyung
> 
> 
> >    - 6/6 - default the pmu event name to raw 0x<perf_event_attr::config>
> >
> > Attached patches:
> >   1/6 perf, tool: Split term type into value type and term type
> >   2/6 perf, tool: Move parse event automated tests to separated object
> >   3/6 perf, tool: Add support for displaying event parser debug info
> >   4/6 perf, tool: Use allocated list for each parsed event
> >   5/6 perf, tool: Separate 'mem:' event scanner bits
> >   6/6 perf, tool: Add hardcoded name term for pmu events
> >
> > thanks,
> > jirka
> > ---
> >  tools/perf/Makefile                 |   13 +-
> >  tools/perf/builtin-test.c           |  552 +------------------------------
> >  tools/perf/util/parse-events-test.c |  625 +++++++++++++++++++++++++++++++++++
> >  tools/perf/util/parse-events.c      |  114 +++++--
> >  tools/perf/util/parse-events.h      |   42 ++-
> >  tools/perf/util/parse-events.l      |   26 ++-
> >  tools/perf/util/parse-events.y      |   93 ++++--
> >  tools/perf/util/pmu.c               |   74 +++--
> >  8 files changed, 883 insertions(+), 656 deletions(-)
--
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/