[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120213131343.GC1626@m.brq.redhat.com>
Date: Mon, 13 Feb 2012 14:13:43 +0100
From: Jiri Olsa <jolsa@...hat.com>
To: acme@...hat.com, a.p.zijlstra@...llo.nl, mingo@...e.hu,
paulus@...ba.org, cjashfor@...ux.vnet.ibm.com
Cc: linux-kernel@...r.kernel.org
Subject: Re: [PATCHv4 0/9] perf tool: parser generator for events parsing
hi,
any feedback? ;)
I saw 1/9 was already taken, thanks
I can send rebased patchset if needed..
thanks,
jirka
On Fri, Jan 27, 2012 at 03:34:19PM +0100, Jiri Olsa wrote:
> hi,
> Here's new version of event parsing patchset. To summarize:
> - adding event parser bison/flex generator
> - the event syntax stays as it was
> - adding event format sysfs group attribute
> for pmu device
> - adding new syntax to specify raw events based
> on the sysfs pmu's format attribute
> eg. "cpu/config=1,config1=2,config2=3/u"
>
> There are helping patches 1-4, which I did not want to
> mangle with the actual change. Patches 5-9 implement
> the feature itself.
>
> v4 changes:
> - 5/9 changed sysfs pmu format definitions to the group attribute,
> each line (in previous patch) is now represented by single
> sysfs attribute under format group
> - 6/9 fix raw events so the number is always considered to be hex number
> - 8/9 move config[12] definitions to be processed by the parser
>
> Attached patches:
> 1/9 perf, tool: Make perf_evlist__splice_list_tail global
> 2/9 perf, tool: Remove unused functions from debugfs object
> 3/9 perf, tool: Add sysfs mountpoint interface
> 4/9 perf, tool: Add bitmap_or function into bitmap object
> 5/9 perf: Adding sysfs group format attribute for pmu device
> 6/9 perf, tool: Add parser generator for events parsing
> 7/9 perf, tool: Add config options support for event parsing
> 8/9 perf, tool: Add perf pmu object to access pmu format definition
> 9/9 perf, tool: Add support to specify pmu style event
>
>
> thanks,
> jirka
> ---
> .../testing/sysfs-bus-event_source-devices-format | 14 +
> include/linux/perf_event.h | 22 +
> kernel/events/core.c | 22 +
> tools/perf/Makefile | 31 +
> tools/perf/builtin-test.c | 52 +-
> tools/perf/util/bitmap.c | 10 +
> tools/perf/util/debugfs.c | 141 --
> tools/perf/util/debugfs.h | 6 -
> tools/perf/util/evlist.c | 6 +-
> tools/perf/util/evlist.h | 5 +
> tools/perf/util/include/linux/bitmap.h | 11 +
> tools/perf/util/parse-events-bison.c | 1813 ++++++++++++++++
> tools/perf/util/parse-events-bison.h | 81 +
> tools/perf/util/parse-events-flex.c | 2225 ++++++++++++++++++++
> tools/perf/util/parse-events-flex.h | 316 +++
> tools/perf/util/parse-events.c | 552 ++---
> tools/perf/util/parse-events.h | 37 +
> tools/perf/util/parse-events.l | 107 +
> tools/perf/util/parse-events.y | 191 ++
> tools/perf/util/pmu-bison.c | 1663 +++++++++++++++
> tools/perf/util/pmu-bison.h | 73 +
> tools/perf/util/pmu-flex.c | 1821 ++++++++++++++++
> tools/perf/util/pmu-flex.h | 316 +++
> tools/perf/util/pmu.c | 462 ++++
> tools/perf/util/pmu.h | 41 +
> tools/perf/util/pmu.l | 43 +
> tools/perf/util/pmu.y | 93 +
> tools/perf/util/sysfs.c | 60 +
> tools/perf/util/sysfs.h | 6 +
> 29 files changed, 9740 insertions(+), 480 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/
Powered by blists - more mailing lists