Re: [GIT PULL||RFC 00/11] perf library and regression testing improvements

Stephane Eranian
2011-01-04T15:30:59Z

On Tue, Jan 4, 2011 at 4:30 PM, Stephane Eranian <eranian@google.com> wrote:
> Just submitted a patch to fix this.
>
That also means that on your build machine, you have a matching perf_event.h.
You cannot expect this from most distros.

>
> On Tue, Jan 4, 2011 at 4:24 PM, Arnaldo Carvalho de Melo
> <acme@infradead.org> wrote:
>> Em Tue, Jan 04, 2011 at 04:10:17PM +0100, Stephane Eranian escreveu:
>>> Ok, The problem comes both files picking up a different verison for
>>> perf_event.h.
>>>
>>> In my case, util/evsel.c was using /usr/include/linux/perf_event.h whereas
>>> builtin-stat.c was using ../../include/linux/perf_event.h. Both have a different
>>> struct perf_event_attr.
>>>
>>> When I remove the /usr/include/linux/perf_event.h file, then I cannot compile
>>> perf anymore:
>>> In file included from perf.c:15:
>>> util/parse-events.h:7:30: error: linux/perf_event.h: No such file or directory
>>>
>>> Looks like something changed in the Makefile and the util modules don't know
>>> where to pickup perf_event.h
>>
>> Ouch, all need to use ../../include/linux/perf_event.h, I think, my bad,
>> checking that.
>>
>> Thanks for figuring out the issue!
>>
>> - Arnaldo
>>
>