lkml.org 
[lkml]   [2011]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: [15/70] perf timechart: Adjust perf timechart to the new power events
Date
Hi,

I should have split out this segfault fix.

On Tuesday 22 February 2011 23:16:54 Greg KH wrote:
> 2.6.37-stable review patch. If anyone has any objections, please let
> us know.
>
> ------------------
>
> From: Thomas Renninger <trenn@suse.de>
>
> commit 20c457b8587bee4644d998331d9e13be82e05b4c upstream.
>
> [This patch is part of mainline git commit 20c457b8587bee4644d9.
> This should fix:
> http://www.mail-archive.com/linux-perf-
> users@vger.kernel.org/msg00057.html
>
> The regression was introduced by git commit:
> 4c21adf26f8fcf86a755b9b9f55c2e9fd241e1fb]
Above link is a nice pointer.
the rest of the changelog (below) does not fit and describes
the mainline patch, the tiny fix was part of.

Don't know whether it's worth to touch it, but if, the changelog (or
title) of the patch should be as short as:
perf: timechart (userspace): Do not segfault on power_end events

Thomas

> builtin-timechart must only pass -e power:xy events if they are
supported by
> the running kernel, otherwise try to fetch the old
power:power{start,end}
> events.
>
> For this I added the tiny helper function:
>
> int is_valid_tracepoint(const char *event_string)
>
> to parse-events.[hc], which could be more generic as an interface and
support
> hardware/software/... events, not only tracepoints, but someone else
could
> extend that if needed...
>
> Signed-off-by: Thomas Renninger <trenn@suse.de>
> Acked-by: Arjan van de Ven <arjan@linux.intel.com>
> Acked-by: Jean Pihet <j-pihet@ti.com>
> LKML-Reference: <1294073445-14812-4-git-send-email-trenn@suse.de>
> Signed-off-by: Ingo Molnar <mingo@elte.hu>
> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
>
> ---
> tools/perf/builtin-timechart.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/tools/perf/builtin-timechart.c
> +++ b/tools/perf/builtin-timechart.c
> @@ -502,7 +502,7 @@ static int process_sample_event(event_t
> c_state_start(pe->cpu_id, data.time, pe->value);
>
> if (strcmp(event_str, "power:power_end") == 0)
> - c_state_end(pe->cpu_id, data.time);
> + c_state_end(data.cpu, data.time);
>
> if (strcmp(event_str, "power:power_frequency") == 0)
> p_state_change(pe->cpu_id, data.time, pe->value);
>
>
>



\
 
 \ /
  Last update: 2011-02-23 01:19    [W:0.363 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site