Re: [PATCH v2 06/10] perf, tools, script: Implement - for typed fields
On Fri, Aug 31, 2018 at 03:02:02PM -0700, Andi Kleen wrote: > From: Andi Kleen > > For perf script -F the - removal of fields didn't work for typed > fields, only for untyped. Add the REMOVE case for this variant too. missing changelog example jirka > > v2: Use correct index > Signed-off-by: Andi Kleen > --- > tools/perf/builtin-script.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c > index d7d454d10074..129b11404a9c 100644 > --- a/tools/perf/builtin-script.c > +++ b/tools/perf/builtin-script.c > @@ -2502,6 +2502,10 @@ static int parse_output_fields(const struct option > *opt __maybe_unused, > rc = -EINVAL; > goto out; > } > + if (change == REMOVE) > + output[type].fields &= > ~all_output_options[i].field; > + else > + output[type].fields |= > all_output_options[i].field; > output[type].user_set = true; > output[type].wildcard_set = true; > } > -- > 2.17.1 >
Re: [PATCH v2 06/10] perf, tools, script: Implement - for typed fields
On Fri, Aug 31, 2018 at 03:02:02PM -0700, Andi Kleen wrote: > From: Andi Kleen > > For perf script -F the - removal of fields didn't work for typed > fields, only for untyped. Add the REMOVE case for this variant too. missing changelog example jirka > > v2: Use correct index > Signed-off-by: Andi Kleen > --- > tools/perf/builtin-script.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c > index d7d454d10074..129b11404a9c 100644 > --- a/tools/perf/builtin-script.c > +++ b/tools/perf/builtin-script.c > @@ -2502,6 +2502,10 @@ static int parse_output_fields(const struct option > *opt __maybe_unused, > rc = -EINVAL; > goto out; > } > + if (change == REMOVE) > + output[type].fields &= > ~all_output_options[i].field; > + else > + output[type].fields |= > all_output_options[i].field; > output[type].user_set = true; > output[type].wildcard_set = true; > } > -- > 2.17.1 >
[PATCH v2 06/10] perf, tools, script: Implement - for typed fields
From: Andi Kleen For perf script -F the - removal of fields didn't work for typed fields, only for untyped. Add the REMOVE case for this variant too. v2: Use correct index Signed-off-by: Andi Kleen --- tools/perf/builtin-script.c | 4 1 file changed, 4 insertions(+) diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c index d7d454d10074..129b11404a9c 100644 --- a/tools/perf/builtin-script.c +++ b/tools/perf/builtin-script.c @@ -2502,6 +2502,10 @@ static int parse_output_fields(const struct option *opt __maybe_unused, rc = -EINVAL; goto out; } + if (change == REMOVE) + output[type].fields &= ~all_output_options[i].field; + else + output[type].fields |= all_output_options[i].field; output[type].user_set = true; output[type].wildcard_set = true; } -- 2.17.1
[PATCH v2 06/10] perf, tools, script: Implement - for typed fields
From: Andi Kleen For perf script -F the - removal of fields didn't work for typed fields, only for untyped. Add the REMOVE case for this variant too. v2: Use correct index Signed-off-by: Andi Kleen --- tools/perf/builtin-script.c | 4 1 file changed, 4 insertions(+) diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c index d7d454d10074..129b11404a9c 100644 --- a/tools/perf/builtin-script.c +++ b/tools/perf/builtin-script.c @@ -2502,6 +2502,10 @@ static int parse_output_fields(const struct option *opt __maybe_unused, rc = -EINVAL; goto out; } + if (change == REMOVE) + output[type].fields &= ~all_output_options[i].field; + else + output[type].fields |= all_output_options[i].field; output[type].user_set = true; output[type].wildcard_set = true; } -- 2.17.1