Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1323499 > unrolled thread

Re: [PATCH v2 1/3] perf kvm: Introduce evsel as argument to perf_event__preprocess_sample

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v2 1/3] perf kvm: Introduce evsel as argument to  perf_event__preprocess_sample Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-02-01 22:00 +0100
    Re: [PATCH v2 1/3] perf kvm: Introduce evsel as argument to  perf_event__preprocess_sample Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> - 2016-02-02 10:10 +0100

#1323499 — Re: [PATCH v2 1/3] perf kvm: Introduce evsel as argument to perf_event__preprocess_sample

Em Fri, Jan 22, 2016 at 11:28:10AM +0530, Ravi Bangoria escreveu:
> This patch changes prototype of perf_event__preprocess_sample() with
> additional argument evsel added at last.
> 
> This change is required because perf_event__preprocess_sample()
> function will use evsel to determine cpumode of samples for powerpc
> architecture.
> 
> Signed-off-by: Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com>

Fixing these problems:



  CC       /tmp/build/perf/ui/gtk/util.o
util/event.c: In function ‘perf_event__preprocess_sample’:
util/event.c:1302:26: error: unused parameter ‘evsel’ [-Werror=unused-parameter]
       struct perf_evsel *evsel)
                          ^
cc1: all warnings being treated as errors
mv: cannot stat ‘/tmp/build/perf/util/.event.o.tmp’: No such file or directory
/home/acme/git/linux/tools/build/Makefile.build:77: recipe for target '/tmp/build/perf/util/event.o' failed
make[3]: *** [/tmp/build/perf/util/event.o] Error 1
make[3]: *** Waiting for unfinished jobs....
  CC       /tmp/build/perf/ui/gtk/helpline.o
  CC       /tmp/build/perf/arch/common.o
/home/acme/git/linux/tools/build/Makefile.build:116: recipe for target 'util' failed
make[2]: *** [util] Error 2
make[2]: *** Waiting for unfinished jobs....

[toc] | [next] | [standalone]

#1323899

HI acme,

Thanks for reviewing the patch.

On Tuesday 02 February 2016 02:23 AM, Arnaldo Carvalho de Melo wrote:
> Em Fri, Jan 22, 2016 at 11:28:10AM +0530, Ravi Bangoria escreveu:
>> This patch changes prototype of perf_event__preprocess_sample() with
>> additional argument evsel added at last.
>>
>> This change is required because perf_event__preprocess_sample()
>> function will use evsel to determine cpumode of samples for powerpc
>> architecture.
>>
>> Signed-off-by: Ravi Bangoria<ravi.bangoria@linux.vnet.ibm.com>
> Fixing these problems:
>
>
>
>    CC       /tmp/build/perf/ui/gtk/util.o
> util/event.c: In function ‘perf_event__preprocess_sample’:
> util/event.c:1302:26: error: unused parameter ‘evsel’ [-Werror=unused-parameter]
>         struct perf_evsel *evsel)
>                            ^
> cc1: all warnings being treated as errors
> mv: cannot stat ‘/tmp/build/perf/util/.event.o.tmp’: No such file or directory
> /home/acme/git/linux/tools/build/Makefile.build:77: recipe for target '/tmp/build/perf/util/event.o' failed
> make[3]: *** [/tmp/build/perf/util/event.o] Error 1
> make[3]: *** Waiting for unfinished jobs....
>    CC       /tmp/build/perf/ui/gtk/helpline.o
>    CC       /tmp/build/perf/arch/common.o
> /home/acme/git/linux/tools/build/Makefile.build:116: recipe for target 'util' failed
> make[2]: *** [util] Error 2
> make[2]: *** Waiting for unfinished jobs....

Thanks for pointing this out. Actually I was not aware about this.

Will take care next time onwards.

Regards,
Ravi

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web