Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-14 Thread Tom Zanussi
On Mon, 2017-02-13 at 10:04 +0900, Namhyung Kim wrote: > On Fri, Feb 10, 2017 at 12:58:30PM -0600, Tom Zanussi wrote: > > Hi Masami, > > > > On Fri, 2017-02-10 at 18:34 +0900, Masami Hiramatsu wrote: > > > On Fri, 10 Feb 2017 13:16:17 +0900 > > > Namhyung Kim wrote: > > > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-14 Thread Tom Zanussi
On Mon, 2017-02-13 at 10:04 +0900, Namhyung Kim wrote: > On Fri, Feb 10, 2017 at 12:58:30PM -0600, Tom Zanussi wrote: > > Hi Masami, > > > > On Fri, 2017-02-10 at 18:34 +0900, Masami Hiramatsu wrote: > > > On Fri, 10 Feb 2017 13:16:17 +0900 > > > Namhyung Kim wrote: > > > > > > > > > > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-14 Thread Masami Hiramatsu
On Mon, 13 Feb 2017 10:04:16 +0900 Namhyung Kim wrote: > On Fri, Feb 10, 2017 at 12:58:30PM -0600, Tom Zanussi wrote: > > Hi Masami, > > > > On Fri, 2017-02-10 at 18:34 +0900, Masami Hiramatsu wrote: > > > On Fri, 10 Feb 2017 13:16:17 +0900 > > > Namhyung Kim

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-14 Thread Masami Hiramatsu
On Mon, 13 Feb 2017 10:04:16 +0900 Namhyung Kim wrote: > On Fri, Feb 10, 2017 at 12:58:30PM -0600, Tom Zanussi wrote: > > Hi Masami, > > > > On Fri, 2017-02-10 at 18:34 +0900, Masami Hiramatsu wrote: > > > On Fri, 10 Feb 2017 13:16:17 +0900 > > > Namhyung Kim wrote: > > > > > > > > > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-12 Thread Namhyung Kim
On Fri, Feb 10, 2017 at 12:58:30PM -0600, Tom Zanussi wrote: > Hi Masami, > > On Fri, 2017-02-10 at 18:34 +0900, Masami Hiramatsu wrote: > > On Fri, 10 Feb 2017 13:16:17 +0900 > > Namhyung Kim wrote: > > > > > > > > > > Example - wakeup latency > > > > > > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-12 Thread Namhyung Kim
On Fri, Feb 10, 2017 at 12:58:30PM -0600, Tom Zanussi wrote: > Hi Masami, > > On Fri, 2017-02-10 at 18:34 +0900, Masami Hiramatsu wrote: > > On Fri, 10 Feb 2017 13:16:17 +0900 > > Namhyung Kim wrote: > > > > > > > > > > Example - wakeup latency > > > > > > > > > > > > This

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-10 Thread Tom Zanussi
Hi Masami, On Fri, 2017-02-10 at 18:34 +0900, Masami Hiramatsu wrote: > On Fri, 10 Feb 2017 13:16:17 +0900 > Namhyung Kim wrote: > > > > > > > Example - wakeup latency > > > > > > > > > This basically implements the -RT latency_hist 'wakeup_latency' > > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-10 Thread Tom Zanussi
Hi Masami, On Fri, 2017-02-10 at 18:34 +0900, Masami Hiramatsu wrote: > On Fri, 10 Feb 2017 13:16:17 +0900 > Namhyung Kim wrote: > > > > > > > Example - wakeup latency > > > > > > > > > This basically implements the -RT latency_hist 'wakeup_latency' > > > histogram using

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-10 Thread Tom Zanussi
Hi Namhyung, On Fri, 2017-02-10 at 13:16 +0900, Namhyung Kim wrote: > Hi Tom, > > On Wed, Feb 08, 2017 at 11:24:56AM -0600, Tom Zanussi wrote: > > This patchset adds support for 'inter-event' quantities to the trace > > event subsystem. The most important example of inter-event quantities > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-10 Thread Tom Zanussi
Hi Namhyung, On Fri, 2017-02-10 at 13:16 +0900, Namhyung Kim wrote: > Hi Tom, > > On Wed, Feb 08, 2017 at 11:24:56AM -0600, Tom Zanussi wrote: > > This patchset adds support for 'inter-event' quantities to the trace > > event subsystem. The most important example of inter-event quantities > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-10 Thread Masami Hiramatsu
On Fri, 10 Feb 2017 13:16:17 +0900 Namhyung Kim wrote: > > > > Example - wakeup latency > > > > > > This basically implements the -RT latency_hist 'wakeup_latency' > > histogram using the synthetic events, variables, and actions > > described. The

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-10 Thread Masami Hiramatsu
On Fri, 10 Feb 2017 13:16:17 +0900 Namhyung Kim wrote: > > > > Example - wakeup latency > > > > > > This basically implements the -RT latency_hist 'wakeup_latency' > > histogram using the synthetic events, variables, and actions > > described. The output below is from a

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Namhyung Kim
Hi Tom, On Wed, Feb 08, 2017 at 11:24:56AM -0600, Tom Zanussi wrote: > This patchset adds support for 'inter-event' quantities to the trace > event subsystem. The most important example of inter-event quantities > are latencies, or the time differences between two events. > > One of the main

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Namhyung Kim
Hi Tom, On Wed, Feb 08, 2017 at 11:24:56AM -0600, Tom Zanussi wrote: > This patchset adds support for 'inter-event' quantities to the trace > event subsystem. The most important example of inter-event quantities > are latencies, or the time differences between two events. > > One of the main

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Steven Rostedt
On Thu, 09 Feb 2017 11:18:32 -0600 Tom Zanussi wrote: > I think it might make a lot of sense at this point to actually create a > system-wide list of active triggers e.g. tracing/events/triggers or > something like that. It's something I've kind of wanted anyway,

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Steven Rostedt
On Thu, 09 Feb 2017 11:18:32 -0600 Tom Zanussi wrote: > I think it might make a lot of sense at this point to actually create a > system-wide list of active triggers e.g. tracing/events/triggers or > something like that. It's something I've kind of wanted anyway, and > would be really useful if

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Tom Zanussi
Hi Frank, On Thu, 2017-02-09 at 09:46 -0500, Frank Ch. Eigler wrote: > Hi, Tom - > > > tom.zanussi wrote: > > > [...] > >> Hmm, this looks a bit hard to understand, I guess that onmatch() means > >> "if there is an event which has ts0 variable and the event's key matches > >> this key, take

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Tom Zanussi
Hi Frank, On Thu, 2017-02-09 at 09:46 -0500, Frank Ch. Eigler wrote: > Hi, Tom - > > > tom.zanussi wrote: > > > [...] > >> Hmm, this looks a bit hard to understand, I guess that onmatch() means > >> "if there is an event which has ts0 variable and the event's key matches > >> this key, take

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Tom Zanussi
On Thu, 2017-02-09 at 23:18 +0900, Masami Hiramatsu wrote: > Hi Tom, > > On Wed, 08 Feb 2017 19:14:22 -0600 > Tom Zanussi wrote: > > > > > I'm submitting the patchset (based on tracing/for-next) as an RFC not > > > > only to get comments, but because there are still

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Tom Zanussi
On Thu, 2017-02-09 at 23:18 +0900, Masami Hiramatsu wrote: > Hi Tom, > > On Wed, 08 Feb 2017 19:14:22 -0600 > Tom Zanussi wrote: > > > > > I'm submitting the patchset (based on tracing/for-next) as an RFC not > > > > only to get comments, but because there are still some problems I > > > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Frank Ch. Eigler
Hi, Tom - tom.zanussi wrote: > [...] >> Hmm, this looks a bit hard to understand, I guess that onmatch() means >> "if there is an event which has ts0 variable and the event's key matches >> this key, take some action". > > Yes, that's pretty much it. It's essentially shorthand for this kind of

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Frank Ch. Eigler
Hi, Tom - tom.zanussi wrote: > [...] >> Hmm, this looks a bit hard to understand, I guess that onmatch() means >> "if there is an event which has ts0 variable and the event's key matches >> this key, take some action". > > Yes, that's pretty much it. It's essentially shorthand for this kind of

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Masami Hiramatsu
Hi Tom, On Wed, 08 Feb 2017 19:14:22 -0600 Tom Zanussi wrote: > > > I'm submitting the patchset (based on tracing/for-next) as an RFC not > > > only to get comments, but because there are still some problems I > > > haven't fixed yet... > > > > > > Here are some

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-09 Thread Masami Hiramatsu
Hi Tom, On Wed, 08 Feb 2017 19:14:22 -0600 Tom Zanussi wrote: > > > I'm submitting the patchset (based on tracing/for-next) as an RFC not > > > only to get comments, but because there are still some problems I > > > haven't fixed yet... > > > > > > Here are some examples that should make

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Steven Rostedt
On Wed, 08 Feb 2017 17:28:50 -0600 Tom Zanussi wrote: > On Wed, 2017-02-08 at 15:01 -0500, Steven Rostedt wrote: > > On Wed, 8 Feb 2017 11:24:56 -0600 > > Tom Zanussi wrote: > > > > > # echo 'wakeup_latency

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Steven Rostedt
On Wed, 08 Feb 2017 17:28:50 -0600 Tom Zanussi wrote: > On Wed, 2017-02-08 at 15:01 -0500, Steven Rostedt wrote: > > On Wed, 8 Feb 2017 11:24:56 -0600 > > Tom Zanussi wrote: > > > > > # echo 'wakeup_latency lat=sched_switch:wakeup_lat \ > > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Tom Zanussi
Hi Masami, On Thu, 2017-02-09 at 08:13 +0900, Masami Hiramatsu wrote: > Hi Tom, > > On Wed, 8 Feb 2017 11:24:56 -0600 > Tom Zanussi wrote: > > > This patchset adds support for 'inter-event' quantities to the trace > > event subsystem. The most important example

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Tom Zanussi
Hi Masami, On Thu, 2017-02-09 at 08:13 +0900, Masami Hiramatsu wrote: > Hi Tom, > > On Wed, 8 Feb 2017 11:24:56 -0600 > Tom Zanussi wrote: > > > This patchset adds support for 'inter-event' quantities to the trace > > event subsystem. The most important example of inter-event quantities > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Tom Zanussi
Hi Steve, On Wed, 2017-02-08 at 15:01 -0500, Steven Rostedt wrote: > On Wed, 8 Feb 2017 11:24:56 -0600 > Tom Zanussi wrote: > > > # echo 'wakeup_latency lat=sched_switch:wakeup_lat \ > >pid=sched_switch:woken_pid \ > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Tom Zanussi
Hi Steve, On Wed, 2017-02-08 at 15:01 -0500, Steven Rostedt wrote: > On Wed, 8 Feb 2017 11:24:56 -0600 > Tom Zanussi wrote: > > > # echo 'wakeup_latency lat=sched_switch:wakeup_lat \ > >pid=sched_switch:woken_pid \ > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Tom Zanussi
On Wed, 2017-02-08 at 15:01 -0500, Steven Rostedt wrote: > On Wed, 8 Feb 2017 11:24:56 -0600 > Tom Zanussi wrote: > > > # echo 'wakeup_latency lat=sched_switch:wakeup_lat \ > >pid=sched_switch:woken_pid \ > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Tom Zanussi
On Wed, 2017-02-08 at 15:01 -0500, Steven Rostedt wrote: > On Wed, 8 Feb 2017 11:24:56 -0600 > Tom Zanussi wrote: > > > # echo 'wakeup_latency lat=sched_switch:wakeup_lat \ > >pid=sched_switch:woken_pid \ > >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Masami Hiramatsu
Hi Tom, On Wed, 8 Feb 2017 11:24:56 -0600 Tom Zanussi wrote: > This patchset adds support for 'inter-event' quantities to the trace > event subsystem. The most important example of inter-event quantities > are latencies, or the time differences between two events.

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Masami Hiramatsu
Hi Tom, On Wed, 8 Feb 2017 11:24:56 -0600 Tom Zanussi wrote: > This patchset adds support for 'inter-event' quantities to the trace > event subsystem. The most important example of inter-event quantities > are latencies, or the time differences between two events. Great! This is what I

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Steven Rostedt
On Wed, 8 Feb 2017 11:24:56 -0600 Tom Zanussi wrote: > # echo 'wakeup_latency lat=sched_switch:wakeup_lat \ >pid=sched_switch:woken_pid \ >prio=sched_switch:woken_prio' >> \ >

Re: [RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Steven Rostedt
On Wed, 8 Feb 2017 11:24:56 -0600 Tom Zanussi wrote: > # echo 'wakeup_latency lat=sched_switch:wakeup_lat \ >pid=sched_switch:woken_pid \ >prio=sched_switch:woken_prio' >> \ > /sys/kernel/debug/tracing/synthetic_events I

[RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Tom Zanussi
This patchset adds support for 'inter-event' quantities to the trace event subsystem. The most important example of inter-event quantities are latencies, or the time differences between two events. One of the main motivations for adding this capability is to provide a general-purpose base that

[RFC][PATCH 00/21] tracing: Inter-event (e.g. latency) support

2017-02-08 Thread Tom Zanussi
This patchset adds support for 'inter-event' quantities to the trace event subsystem. The most important example of inter-event quantities are latencies, or the time differences between two events. One of the main motivations for adding this capability is to provide a general-purpose base that