Re: Linux 2.6.32.34

[Teck Choon Giam]

On Thu, Mar 24, 2011 at 8:18 AM, Jiri Slaby <jslaby@xxxxxxx> wrote:
> On 03/23/2011 09:34 PM, Greg KH wrote:
>> --- a/kernel/perf_event.c
>> +++ b/kernel/perf_event.c
>> @@ -4167,6 +4167,8 @@ static void tp_perf_event_destroy(struct perf_event *event)
>>
>> Âstatic const struct pmu *tp_perf_event_init(struct perf_event *event)
>> Â{
>> + Â Â if (event->hw.state & PERF_HES_STOPPED)
>> + Â Â Â Â Â Â return 0;
>> Â Â Â /*
>> Â Â Â Â* Raw tracepoint data is a severe data leak, only allow root to
>> Â Â Â Â* have these.
>
> This causes build to fail:
> /usr/src/packages/BUILD/kernel-vanilla-2.6.32.34/linux-2.6.32/kernel/perf_event.c:
> Inâfunctionâ'tp_perf_event_init':
> /usr/src/packages/BUILD/kernel-vanilla-2.6.32.34/linux-2.6.32/kernel/perf_event.c:4170:
> error:â'structâhw_perf_event'âhasânoâmemberânamedâ'state'
> /usr/src/packages/BUILD/kernel-vanilla-2.6.32.34/linux-2.6.32/kernel/perf_event.c:4170:
> error:â'PERF_HES_STOPPED'âundeclaredâ(firstâuseâinâthisâfunction)
> /usr/src/packages/BUILD/kernel-vanilla-2.6.32.34/linux-2.6.32/kernel/perf_event.c:4170:
> error:â(Eachâundeclaredâidentifierâisâreportedâonlyâonce
> /usr/src/packages/BUILD/kernel-vanilla-2.6.32.34/linux-2.6.32/kernel/perf_event.c:4170:
> error:âforâeachâfunctionâitâappearsâin.)
>
>
> The source:
> commit 6f197b73304b3bd3d5a43b931383a5331d6b2987
> Author: Frederic Weisbecker <fweisbec@xxxxxxxxx>
> Date: Â Mon Mar 7 21:27:09 2011 +0100
>
> Â Âperf: Handle stopped state with tracepoints
>
> Â Âcommit a0f7d0f7fc02465bb9758501f611f63381792996 upstream.
>
> Â ÂWe toggle the state from start and stop callbacks but actually
> Â Âdon't check it when the event triggers. Do it so that
> Â Âthese callbacks actually work.

It builds fine for me with gcc version 4.1.2 for both x86_32 and x86_64.

Thanks.

Kindest regards,
Giam Teck Choon
--
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/