Re: [Powertop] [PATCH 1/2] perf: Pass 0 as a third argument to ioctl on perf_fd

[Chris Ferron <chris.e.ferron at linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1213 bytes --]

On 09/16/2012 07:17 AM, Namhyung Kim wrote:
> The third argument of PERF_EVENT_IOC_{ENABLE,DISABLE} ioctl is a flag
> to determine whether the command should be applied to group members.
> Since the powertop doesn't use the event grouping feature, it'd better
> passing 0 as a third argument explicitly.
>
> Signed-off-by: Namhyung Kim <namhyung(a)gmail.com>
> ---
>   src/perf/perf.cpp |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/perf/perf.cpp b/src/perf/perf.cpp
> index 69f1da3..c8ba7a8 100644
> --- a/src/perf/perf.cpp
> +++ b/src/perf/perf.cpp
> @@ -128,7 +128,7 @@ void perf_event::create_perf_event(char *eventname, int _cpu)
>   		return;
>   	}
>   
> -	ret = ioctl(perf_fd, PERF_EVENT_IOC_ENABLE);
> +	ret = ioctl(perf_fd, PERF_EVENT_IOC_ENABLE, 0);
>   
>   	if (ret < 0) {
>   		fprintf(stderr, "failed to enable perf \n");
> @@ -210,7 +210,7 @@ void perf_event::start(void)
>   void perf_event::stop(void)
>   {
>   	int ret;
> -	ret = ioctl(perf_fd, PERF_EVENT_IOC_DISABLE);
> +	ret = ioctl(perf_fd, PERF_EVENT_IOC_DISABLE, 0);
>   	if (ret)
>   		cout << "stop failing\n";
>   }
Your patch as been merged.
Thank You
-C