Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-03-01 Thread Ravi Bangoria
Thank you Brendan for reviewing, On Wednesday 01 March 2017 10:34 AM, Brendan Gregg wrote: > On Tue, Feb 28, 2017 at 2:31 PM, Brendan Gregg > wrote: >> G'Day Ravi, >> > [...] >> Now retrying perf: >> >> # ./perf record -e sdt_node:http__server__request -a >> ^C[ perf

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-03-01 Thread Ravi Bangoria
Thank you Brendan for reviewing, On Wednesday 01 March 2017 10:34 AM, Brendan Gregg wrote: > On Tue, Feb 28, 2017 at 2:31 PM, Brendan Gregg > wrote: >> G'Day Ravi, >> > [...] >> Now retrying perf: >> >> # ./perf record -e sdt_node:http__server__request -a >> ^C[ perf record: Woken up 1 times to

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-28 Thread Brendan Gregg
On Tue, Feb 28, 2017 at 2:31 PM, Brendan Gregg wrote: > G'Day Ravi, > [...] > Now retrying perf: > > # ./perf record -e sdt_node:http__server__request -a > ^C[ perf record: Woken up 1 times to write data ] > [ perf record: Captured and wrote 0.446 MB perf.data (3

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-28 Thread Brendan Gregg
On Tue, Feb 28, 2017 at 2:31 PM, Brendan Gregg wrote: > G'Day Ravi, > [...] > Now retrying perf: > > # ./perf record -e sdt_node:http__server__request -a > ^C[ perf record: Woken up 1 times to write data ] > [ perf record: Captured and wrote 0.446 MB perf.data (3 samples) ] > # ./perf script >

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-28 Thread Masami Hiramatsu
On Tue, 28 Feb 2017 16:26:19 +0530 Ravi Bangoria wrote: > Thanks Masami for the review. > Please find my replies to your comments. > > On Tuesday 28 February 2017 11:15 AM, Masami Hiramatsu wrote: > > On Fri, 24 Feb 2017 13:13:25 +0530 > > Ravi Bangoria

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-28 Thread Masami Hiramatsu
On Tue, 28 Feb 2017 16:26:19 +0530 Ravi Bangoria wrote: > Thanks Masami for the review. > Please find my replies to your comments. > > On Tuesday 28 February 2017 11:15 AM, Masami Hiramatsu wrote: > > On Fri, 24 Feb 2017 13:13:25 +0530 > > Ravi Bangoria wrote: > > > >> After invoking 'perf

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-28 Thread Brendan Gregg
G'Day Ravi, On Thu, Feb 23, 2017 at 11:43 PM, Ravi Bangoria wrote: > > From: Hemant Kumar > > Add support for directly recording SDT events which are present in > the probe cache. Without this patch, we could probe into SDT events >

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-28 Thread Brendan Gregg
G'Day Ravi, On Thu, Feb 23, 2017 at 11:43 PM, Ravi Bangoria wrote: > > From: Hemant Kumar > > Add support for directly recording SDT events which are present in > the probe cache. Without this patch, we could probe into SDT events > using 'perf probe' and 'perf record'. With this patch, we can

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-28 Thread Ravi Bangoria
Thanks Masami for the review. Please find my replies to your comments. On Tuesday 28 February 2017 11:15 AM, Masami Hiramatsu wrote: > On Fri, 24 Feb 2017 13:13:25 +0530 > Ravi Bangoria wrote: > >> After invoking 'perf record', behind the scenes, it checks

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-28 Thread Ravi Bangoria
Thanks Masami for the review. Please find my replies to your comments. On Tuesday 28 February 2017 11:15 AM, Masami Hiramatsu wrote: > On Fri, 24 Feb 2017 13:13:25 +0530 > Ravi Bangoria wrote: > >> After invoking 'perf record', behind the scenes, it checks whether >> the event specified is an

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-27 Thread Masami Hiramatsu
On Fri, 24 Feb 2017 13:13:25 +0530 Ravi Bangoria wrote: > From: Hemant Kumar > > Add support for directly recording SDT events which are present in > the probe cache. Without this patch, we could probe into SDT events > using 'perf

Re: [PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-27 Thread Masami Hiramatsu
On Fri, 24 Feb 2017 13:13:25 +0530 Ravi Bangoria wrote: > From: Hemant Kumar > > Add support for directly recording SDT events which are present in > the probe cache. Without this patch, we could probe into SDT events > using 'perf probe' and 'perf record'. With this patch, we can probe > the

[PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-23 Thread Ravi Bangoria
From: Hemant Kumar Add support for directly recording SDT events which are present in the probe cache. Without this patch, we could probe into SDT events using 'perf probe' and 'perf record'. With this patch, we can probe the SDT events directly using 'perf record'.

[PATCH v3 2/2] perf/sdt: Directly record SDT events with 'perf record'

2017-02-23 Thread Ravi Bangoria
From: Hemant Kumar Add support for directly recording SDT events which are present in the probe cache. Without this patch, we could probe into SDT events using 'perf probe' and 'perf record'. With this patch, we can probe the SDT events directly using 'perf record'. For example : $ perf list