Re: [PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2016-02-19 Thread Steven Rostedt
On Fri, 19 Feb 2016 17:31:11 -0600 Tom Zanussi wrote: > > Do you want me to respin adding this and the fixes for the previous > comments before continuing? Yeah, respin, but start with patch 9 of this series. I've already looked at and even pulled into a local

Re: [PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2016-02-19 Thread Steven Rostedt
On Fri, 19 Feb 2016 17:31:11 -0600 Tom Zanussi wrote: > > Do you want me to respin adding this and the fixes for the previous > comments before continuing? Yeah, respin, but start with patch 9 of this series. I've already looked at and even pulled into a local tree the first 8 patches. --

Re: [PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2016-02-19 Thread Tom Zanussi
On Fri, 2016-02-19 at 17:35 -0500, Steven Rostedt wrote: > On Fri, 19 Feb 2016 14:30:13 -0500 > Steven Rostedt wrote: > > > > @@ -1333,6 +1339,7 @@ struct event_command { > > > char*name; > > > enum event_trigger_type trigger_type; > > > bool

Re: [PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2016-02-19 Thread Tom Zanussi
On Fri, 2016-02-19 at 17:35 -0500, Steven Rostedt wrote: > On Fri, 19 Feb 2016 14:30:13 -0500 > Steven Rostedt wrote: > > > > @@ -1333,6 +1339,7 @@ struct event_command { > > > char*name; > > > enum event_trigger_type trigger_type; > > > bool

Re: [PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2016-02-19 Thread Steven Rostedt
On Fri, 19 Feb 2016 14:30:13 -0500 Steven Rostedt wrote: > > @@ -1333,6 +1339,7 @@ struct event_command { > > char*name; > > enum event_trigger_type trigger_type; > > boolpost_trigger; > > + bool

Re: [PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2016-02-19 Thread Steven Rostedt
On Fri, 19 Feb 2016 14:30:13 -0500 Steven Rostedt wrote: > > @@ -1333,6 +1339,7 @@ struct event_command { > > char*name; > > enum event_trigger_type trigger_type; > > boolpost_trigger; > > + boolneeds_rec; > > Note,

Re: [PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2016-02-19 Thread Steven Rostedt
On Thu, 10 Dec 2015 12:50:48 -0600 Tom Zanussi wrote: > Add a new needs_rec flag for triggers that require unconditional > access to trace records in order to function. > > Normally a trigger requires access to the contents of a trace record > only if it has a

Re: [PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2016-02-19 Thread Steven Rostedt
On Thu, 10 Dec 2015 12:50:48 -0600 Tom Zanussi wrote: > Add a new needs_rec flag for triggers that require unconditional > access to trace records in order to function. > > Normally a trigger requires access to the contents of a trace record > only if it has a filter associated with it (since

[PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2015-12-10 Thread Tom Zanussi
Add a new needs_rec flag for triggers that require unconditional access to trace records in order to function. Normally a trigger requires access to the contents of a trace record only if it has a filter associated with it (since filters need the contents of a record in order to make a filtering

[PATCH v13 06/29] tracing: Add needs_rec flag to event triggers

2015-12-10 Thread Tom Zanussi
Add a new needs_rec flag for triggers that require unconditional access to trace records in order to function. Normally a trigger requires access to the contents of a trace record only if it has a filter associated with it (since filters need the contents of a record in order to make a filtering