Em Thu, Feb 16, 2012 at 06:03:03PM +0100, Peter Zijlstra escreveu: > On Thu, 2012-02-16 at 13:00 +0100, Jiri Olsa wrote: > > hi, > > here's new version of event parsing patchset. > > > > I saw Arnaldo already sent patches 1-3 to Ingo, but they did not make it > > to the tree yet, so I'm still attaching them here. Right, I'll take the rest (read this just now, keeping the dup comments below :-) ). > > v5 changes: > > 4/8 - patch updated by Peter with actual format definitions > > for x86 processors > > 6/8 - added hardcoded terms for config[12] and sample_period > > the code is ready to include branch_sample_type once > > this field is available in perf_event_attr, just > > uncommenting one line.. ;) > > - allow config option to be used without value in which > > case 1 is assigned as value > > > > Overall 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" > > > > Attached patches: > > 1/8 perf, tool: Remove unused functions from debugfs object > > 2/8 perf, tool: Add sysfs mountpoint interface > > 3/8 perf, tool: Add bitmap_or function into bitmap object > > 4/8 perf: Adding sysfs group format attribute for pmu device > > 5/8 perf, tool: Add parser generator for events parsing > > 6/8 perf, tool: Add config options support for event parsing > > 7/8 perf, tool: Add perf pmu object to access pmu format definition > > 8/8 perf, tool: Add support to specify pmu style event > > I'm fine with this going fwd, and since this is mostly tools/perf I > suggest acme take it, so: > > Acked-by: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx> > > Also, it might make sense to CC everybody who has/maintains a hardware > pmu implementation on patch 4. I'll do it, I already merged patch 1, 2, 3 in my perf/core branch, will look and merge the rest, adding your a-b. - Arnaldo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/