Messages in this thread | | | Date | Mon, 4 Feb 2013 08:30:27 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH 1/8] perf tools: Add '.' as part of the event 'name' token |
| |
On Sun, Feb 03, 2013 at 09:37:16PM +0100, Stephane Eranian wrote: > On Tue, Jan 29, 2013 at 9:03 AM, Ingo Molnar <mingo@kernel.org> wrote: > > > > * Stephane Eranian <eranian@google.com> wrote: > > > >> On Mon, Jan 28, 2013 at 9:52 PM, Stephane Eranian <eranian@google.com> wrote:
SNIP
> Adding the option for users to link with libpfm4 was no big > deal in my mind. You are opposed to that. That's your > choice. I don't buy your arguments for this patch series > and will continue to provide a patch to link perf and libpfm4 > for users who are interested. > > I would have appreciated some discussion on this instead, once > again, work was done behind closed door and thrown at my face.
Not sure why you think it was 'closed door'.. I needed some way to access those events and sent RFC for it with you being on CC list.
> The worst being that as far I can see in the patches, no proper > credits to libpfm4 contributors is even given for providing the raw > information that allowed you to build your event tables without too
Right, sorry for that.. I'll add it.
> much sweat! Building electronic event tables from vendor specs is > a lot harder than it seems, but you may not know this.
jirka
| |