Re: [PATCH] building libtraceevent with clang

2017-02-17 Thread Arnaldo Carvalho de Melo
Em Thu, Feb 16, 2017 at 09:00:23PM -0500, Steven Rostedt escreveu: > On Mon, 13 Feb 2017 14:20:20 -0300 > Arnaldo Carvalho de Melo wrote: > > +++ b/tools/lib/traceevent/plugin_function.c > > @@ -130,7 +130,7 @@ static int function_handler(struct trace_seq *s, struct > > pevent_record *record, > >

Re: [PATCH] building libtraceevent with clang

2017-02-16 Thread Steven Rostedt
On Mon, 13 Feb 2017 14:20:20 -0300 Arnaldo Carvalho de Melo wrote: > Cc: Adrian Hunter > Cc: David Ahern > Cc: Jiri Olsa > Cc: Namhyung Kim > Cc: Wang Nan > Link: http://lkml.kernel.org/n/tip-b5wyjocel55gorl2jq2cb...@git.kernel.org > Signed-off-by: Arnaldo Carv

Re: [PATCH] building libtraceevent with clang

2017-02-13 Thread Steven Rostedt
On Mon, 13 Feb 2017 14:20:20 -0300 Arnaldo Carvalho de Melo wrote: > Sure, now take a look at this another one: > > commit 6401e4361df183bd9953dce56f7c51d8ef28b11e > Author: Arnaldo Carvalho de Melo > Date: Mon Feb 13 13:33:57 2017 -0300 > > tools lib traceevent plugin function: Initial

Re: [PATCH] building libtraceevent with clang

2017-02-13 Thread Arnaldo Carvalho de Melo
Em Mon, Feb 13, 2017 at 12:14:18PM -0500, Steven Rostedt escreveu: > On Mon, 13 Feb 2017 13:26:22 -0300 > Arnaldo Carvalho de Melo wrote: > > > > > diff --git a/tools/lib/traceevent/kbuffer-parse.c > > > b/tools/lib/traceevent/kbuffer-parse.c > > > index 65984f1c2974..2009cb7d9675 100644 > > >

Re: [PATCH] building libtraceevent with clang

2017-02-13 Thread Steven Rostedt
On Mon, 13 Feb 2017 13:26:22 -0300 Arnaldo Carvalho de Melo wrote: > > diff --git a/tools/lib/traceevent/kbuffer-parse.c > > b/tools/lib/traceevent/kbuffer-parse.c > > index 65984f1c2974..2009cb7d9675 100644 > > --- a/tools/lib/traceevent/kbuffer-parse.c > > +++ b/tools/lib/traceevent/kbuffer-p

Re: [PATCH] building libtraceevent with clang

2017-02-13 Thread Arnaldo Carvalho de Melo
Em Mon, Feb 13, 2017 at 01:24:55PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Fri, Feb 10, 2017 at 02:14:08PM -0500, Steven Rostedt escreveu: > > On Fri, 10 Feb 2017 14:03:17 -0300 > > Arnaldo Carvalho de Melo wrote: > > > + case OLD_RINGBUF_TYPE_TIME_EXTEND: > > > + extend = read_4(kb

Re: [PATCH] building libtraceevent with clang

2017-02-13 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 10, 2017 at 02:14:08PM -0500, Steven Rostedt escreveu: > On Fri, 10 Feb 2017 14:03:17 -0300 > Arnaldo Carvalho de Melo wrote: > > + case OLD_RINGBUF_TYPE_TIME_EXTEND: > > + extend = read_4(kbuf, ptr); > > + extend <<= TS_SHIFT; > > + extend += delta; > >

Re: [PATCH] building libtraceevent with clang

2017-02-10 Thread Steven Rostedt
On Fri, 10 Feb 2017 14:03:17 -0300 Arnaldo Carvalho de Melo wrote: > Hi Steven, > > I tried building perf (and thus libtraceevent) with clang and > got this one: > > kbuffer-parse.c:312:7: warning: variable 'length' is used uninitialized > whenever switch case is taken [-Wsometimes-unini

[PATCH] building libtraceevent with clang

2017-02-10 Thread Arnaldo Carvalho de Melo
Hi Steven, I tried building perf (and thus libtraceevent) with clang and got this one: kbuffer-parse.c:312:7: warning: variable 'length' is used uninitialized whenever switch case is taken [-Wsometimes-uninitialized] case OLD_RINGBUF_TYPE_TIME_EXTEND: ^~~