Re: [PATCH 0/5] perf tool: Fix enable/disable events logic for record command

From: Namhyung Kim
Date: Mon Nov 12 2012 - 21:25:59 EST


On Mon, 12 Nov 2012 18:33:59 +0100, Jiri Olsa wrote:
> hi,
> this patchset fixes the event enable/disable logic for record command
> (patches 1 - 4) plus small fix for event parsing (patch 5).
>
> Attached patches:
> 1/5 perf tool: Fix attributes for '{}' defined event groups
> 2/5 perf tool: Fix 'disabled' attribute config for record command
> 3/5 perf tool: Ensure single disable call per event in record comand
> 4/5 perf tool: Omit group members from perf_evlist__disable/enable
> 5/5 perf tool: Add basic event modifier sanity check
>
> Also available in:
> git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/linux.git
> perf/enable5
>
> thanks,
> jirka
>
> Cc: Corey Ashford <cjashfor@xxxxxxxxxxxxxxxxxx>
> Cc: Frederic Weisbecker <fweisbec@xxxxxxxxx>
> Cc: Ingo Molnar <mingo@xxxxxxx>
> Cc: Namhyung Kim <namhyung@xxxxxxxxxx>
> Cc: Paul Mackerras <paulus@xxxxxxxxx>
> Cc: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>
> Cc: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>

I commented on the patch 5/5, anyway looks good to me:

Acked-by: Namhyung Kim <namhyung@xxxxxxxxxx>

Thanks,
Namhyung
--
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/