lkml.org 
[lkml]   [2019]   [Feb]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 01/11] perf tools script: Handle missing fields with -F +..
On Sun, Feb 24, 2019 at 07:37:12AM -0800, Andi Kleen wrote:
> From: Andi Kleen <ak@linux.intel.com>
>
> When using -F + syntax to add a field the existing defaults
> are currently all marked user_set. This can cause errors when
> some field is missing in the perf.data
>
> This patch tracks the actually user set fields separately,
> so that we don't error out in this case.
>
> Before:
>
> % perf record true
> % perf script -F +metric
> Samples for 'cycles:ppp' event do not have CPU attribute set. Cannot print 'cpu' field.
> %
>
> After
>
> 5 perf record true
> % perf script -F +metric
> perf 28936 278636.237688: 1 cycles:ppp: ffffffff8117da99 perf_event_exec+0x59 (/lib/modules/4.20.0-odilo/build/vmlinux)
> ...
> %
>
> Signed-off-by: Andi Kleen <ak@linux.intel.com>

nice, that one was annoying

Acked-by: Jiri Olsa <jolsa@kernel.org>

thanks,
jirka

> ---
> tools/perf/builtin-script.c | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
> index 8d5fe092525c..29e95a25c6e6 100644
> --- a/tools/perf/builtin-script.c
> +++ b/tools/perf/builtin-script.c
> @@ -149,6 +149,7 @@ static struct {
> unsigned int print_ip_opts;
> u64 fields;
> u64 invalid_fields;
> + u64 user_set_fields;
> } output[OUTPUT_TYPE_MAX] = {
>
> [PERF_TYPE_HARDWARE] = {
> @@ -345,7 +346,7 @@ static int perf_evsel__do_check_stype(struct perf_evsel *evsel,
> if (attr->sample_type & sample_type)
> return 0;
>
> - if (output[type].user_set) {
> + if (output[type].user_set_fields & field) {
> if (allow_user_set)
> return 0;
> evname = perf_evsel__name(evsel);
> @@ -2628,10 +2629,13 @@ static int parse_output_fields(const struct option *opt __maybe_unused,
> pr_warning("\'%s\' not valid for %s events. Ignoring.\n",
> all_output_options[i].str, event_type(j));
> } else {
> - if (change == REMOVE)
> + if (change == REMOVE) {
> output[j].fields &= ~all_output_options[i].field;
> - else
> + output[j].user_set_fields &= ~all_output_options[i].field;
> + } else {
> output[j].fields |= all_output_options[i].field;
> + output[j].user_set_fields |= all_output_options[i].field;
> + }
> output[j].user_set = true;
> output[j].wildcard_set = true;
> }
> --
> 2.17.2
>

\
 
 \ /
  Last update: 2019-02-25 13:57    [W:0.231 / U:0.388 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site