Re: [PATCH v9 1/1] Tracepoint: register/unregister struct tracepoint

2014-04-04 Thread Mathieu Desnoyers
nel@vger.kernel.org, "Ingo Molnar" , > > "Frederic Weisbecker" , > > "Andrew Morton" , "Frank Ch. Eigler" > > , "Johannes Berg" > > > > Sent: Thursday, April 3, 2014 2:54:41 PM > > Subject: Re: [PATCH v9 1/1] Trace

Re: [PATCH v9 1/1] Tracepoint: register/unregister struct tracepoint

2014-04-03 Thread Mathieu Desnoyers
es Berg" > > Sent: Thursday, April 3, 2014 2:54:41 PM > Subject: Re: [PATCH v9 1/1] Tracepoint: register/unregister struct tracepoint > > On Thu, 3 Apr 2014 17:49:54 + (UTC) > Mathieu Desnoyers wrote: > > > > So my current thinking is that the pre-existing

Re: [PATCH v9 1/1] Tracepoint: register/unregister struct tracepoint

2014-04-03 Thread Steven Rostedt
On Thu, 3 Apr 2014 17:49:54 + (UTC) Mathieu Desnoyers wrote: > So my current thinking is that the pre-existing code was erroneously > enabling tracepoints with the name of every event enabled (including > e.g. function tracer, kprobes, etc). It was not failing because > tracepoint.c silently

Re: [PATCH v9 1/1] Tracepoint: register/unregister struct tracepoint

2014-04-03 Thread Mathieu Desnoyers
es Berg" > > Sent: Thursday, April 3, 2014 1:15:59 PM > Subject: Re: [PATCH v9 1/1] Tracepoint: register/unregister struct tracepoint > > On Tue, 1 Apr 2014 18:31:48 -0400 > Mathieu Desnoyers wrote: > > > Register/unregister tracepoint probes with struct trace

Re: [PATCH v9 1/1] Tracepoint: register/unregister struct tracepoint

2014-04-03 Thread Steven Rostedt
On Tue, 1 Apr 2014 18:31:48 -0400 Mathieu Desnoyers wrote: > Register/unregister tracepoint probes with struct tracepoint pointer > rather than tracepoint name. > > This change, which vastly simplifies tracepoint.c, has been proposed by > Steven Rostedt. It also removes 8.8kB (mostly of text) t

[PATCH v9 1/1] Tracepoint: register/unregister struct tracepoint

2014-04-01 Thread Mathieu Desnoyers
Register/unregister tracepoint probes with struct tracepoint pointer rather than tracepoint name. This change, which vastly simplifies tracepoint.c, has been proposed by Steven Rostedt. It also removes 8.8kB (mostly of text) to the vmlinux size. >From this point on, the tracers need to pass a str