Messages in this thread | | | Date | Thu, 22 Dec 2011 14:32:28 -0500 (EST) | From | Vince Weaver <> | Subject | Re: [PATCH 1/3] perf, tool: Add parser generator for events parsing |
| |
On Tue, 20 Dec 2011, Peter Zijlstra wrote:
> > This would allow us to write things like: > > perf record -e cpu/event=0x34,umask=0x01,inv,cmask=2
I know this is beating a dead horse, but are you guys *really* that against just linking against libpfm4?
Instead of (to use your example): perf record -e cpu/event=0x34,umask=0x01,inv,cmask=2 Insn't perf record -e nhm::UNC_CBO_CACHE_LOOKUP:M:i=1:cmask=2 much nicer?
And the code that translates that to bits for you already exists in a nice library form. And it supports Pentium 4 already.
In PAPI we're finally moving to all string-based event names simply because trying to create cross-platform ways of specifying bits like you are just didn't work very well.
Vince
| |