From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755537AbaCRNYH (ORCPT ); Tue, 18 Mar 2014 09:24:07 -0400 Received: from mx1.redhat.com ([209.132.183.28]:4523 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754050AbaCRNYD (ORCPT ); Tue, 18 Mar 2014 09:24:03 -0400 Date: Tue, 18 Mar 2014 14:23:55 +0100 From: Jiri Olsa To: Namhyung Kim Cc: Ramkumar Ramachandra , LKML , Steven Rostedt , Arnaldo Carvalho de Melo Subject: Re: [PATCH 5/5] perf test: squelch warnings about undefined sizeof Message-ID: <20140318132355.GA7845@krava.brq.redhat.com> References: <1395095198-20034-1-git-send-email-artagnon@gmail.com> <1395095198-20034-6-git-send-email-artagnon@gmail.com> <878us8rkqq.fsf@sejong.aot.lge.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <878us8rkqq.fsf@sejong.aot.lge.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 18, 2014 at 05:09:33PM +0900, Namhyung Kim wrote: > On Mon, 17 Mar 2014 18:26:38 -0400, Ramkumar Ramachandra wrote: > > perf test emits the following warnings on the parse events test: > > > > $ perf test > > 5: parse events tests > > Warning: function sizeof not defined > > Warning: function sizeof not defined > > Warning: function sizeof not defined > > Warning: function sizeof not defined > > Warning: function sizeof not defined > > Warning: function sizeof not defined > > Warning: function sizeof not defined > > Warning: function sizeof not defined > > ... > > > > Squelch the warnings by explicitly ignoring the sizeof function. > > It just hides the warning leaving the real problem untouched. If you > really don't want to see those, I guess installing proper plugin for the > failing events will help you (in case you didn't). it wont' help, sizeof is special.. AFAIK we did not figure out a way to handle that so far but I think it was used only in one trace subsystem.. maybe we could replace it and forbid to use it in future ;-) jirka