Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2013-02-05 Thread Namhyung Kim
On Mon, 4 Feb 2013 13:32:54 +0100, Jiri Olsa wrote: > hi, > adding support to read sample values through the PERF_SAMPLE_READ > sample type. It's now possible to specify 'S' modifier for an event > and get its sample value by PERF_SAMPLE_READ. > > For group the 'S' modifier will enable sampling on

[PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2013-02-04 Thread Jiri Olsa
hi, adding support to read sample values through the PERF_SAMPLE_READ sample type. It's now possible to specify 'S' modifier for an event and get its sample value by PERF_SAMPLE_READ. For group the 'S' modifier will enable sampling only for the leader and read all the group member by PERF_SAMPLE_R

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-26 Thread Arnaldo Carvalho de Melo
Em Sat, Oct 27, 2012 at 12:39:06AM +0900, Namhyung Kim escreveu: > 2012-10-26 (금), 12:23 +0200, Jiri Olsa: > > $ ./perf record -e '{cycles,cache-misses}:S,instructions' ls > > $ ./perf record -e '{cycles:S,cache-misses},instructions' ls > > $ ./perf record -e 'cycles:S,instructions' ls > >

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-26 Thread David Ahern
On 10/26/12 10:25 AM, Namhyung Kim wrote: 2012-10-26 (금), 10:14 -0600, David Ahern: On 10/26/12 9:39 AM, Namhyung Kim wrote: hm, thats the unique sample_type issue again ;) Once we set PERF_SAMPLE_READ for event or group, we need to set it for all other events in session, otherwise the report f

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-26 Thread Namhyung Kim
2012-10-26 (금), 10:14 -0600, David Ahern: > On 10/26/12 9:39 AM, Namhyung Kim wrote: > >> hm, thats the unique sample_type issue again ;) Once we set > >> PERF_SAMPLE_READ for event or group, we need to set it for > >> all other events in session, otherwise the report fails > > > > Sorry, I don't u

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-26 Thread David Ahern
On 10/26/12 9:39 AM, Namhyung Kim wrote: hm, thats the unique sample_type issue again ;) Once we set PERF_SAMPLE_READ for event or group, we need to set it for all other events in session, otherwise the report fails Sorry, I don't understand why we need to set it for all events. Just setting i

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-26 Thread Namhyung Kim
2012-10-26 (금), 12:23 +0200, Jiri Olsa: > On Fri, Oct 26, 2012 at 11:14:45AM +0200, Peter Zijlstra wrote: > > I haven't actually looked at the implementation, but I understood it to > > be a group modifier, not an event modifier. > > we might want to be able to use PERF_SAMPLE_READ for single even

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-26 Thread Jiri Olsa
On Fri, Oct 26, 2012 at 11:14:45AM +0200, Peter Zijlstra wrote: > On Fri, 2012-10-26 at 10:29 +0900, Namhyung Kim wrote: > > Hi Jiri, > > > > On Sat, 20 Oct 2012 16:33:08 +0200, Jiri Olsa wrote: > > > hi, > > > adding support to read sample values through the PERF_SAMPLE_READ > > > sample type. It

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-26 Thread Peter Zijlstra
On Fri, 2012-10-26 at 10:29 +0900, Namhyung Kim wrote: > Hi Jiri, > > On Sat, 20 Oct 2012 16:33:08 +0200, Jiri Olsa wrote: > > hi, > > adding support to read sample values through the PERF_SAMPLE_READ > > sample type. It's now possible to specify 'S' modifier for an event > > and get its sample va

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-25 Thread Namhyung Kim
Hi Jiri, On Sat, 20 Oct 2012 16:33:08 +0200, Jiri Olsa wrote: > hi, > adding support to read sample values through the PERF_SAMPLE_READ > sample type. It's now possible to specify 'S' modifier for an event > and get its sample value by PERF_SAMPLE_READ. I have a question. What's an actual impact

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-22 Thread Jiri Olsa
On Mon, Oct 22, 2012 at 05:51:01PM +0900, Namhyung Kim wrote: > On Mon, 22 Oct 2012 10:09:31 +0200, Jiri Olsa wrote: > > On Sun, Oct 21, 2012 at 06:38:49PM +0200, Ingo Molnar wrote: > >> > # > >> > # Samples: 86 of event 'anon group { cycles, cache-misses }' > >> > # Event count (approx.): 3

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-22 Thread Jiri Olsa
On Mon, Oct 22, 2012 at 05:53:32PM +0900, Namhyung Kim wrote: > On Mon, 22 Oct 2012 09:53:05 +0200, Jiri Olsa wrote: > > On Mon, Oct 22, 2012 at 04:32:11PM +0900, Namhyung Kim wrote: > >> It'd be better if you provide a git branch for this series. > > > > aaah right, I forgot to mention that.. ;) >

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-22 Thread Namhyung Kim
On Mon, 22 Oct 2012 09:53:05 +0200, Jiri Olsa wrote: > On Mon, Oct 22, 2012 at 04:32:11PM +0900, Namhyung Kim wrote: >> It'd be better if you provide a git branch for this series. > > aaah right, I forgot to mention that.. ;) > > git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/linux.git > perf/

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-22 Thread Namhyung Kim
On Mon, 22 Oct 2012 10:09:31 +0200, Jiri Olsa wrote: > On Sun, Oct 21, 2012 at 06:38:49PM +0200, Ingo Molnar wrote: >> > # >> > # Samples: 86 of event 'anon group { cycles, cache-misses }' >> > # Event count (approx.): 34863674 >> > # >> > # OverheadPeriod Co

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-22 Thread Jiri Olsa
On Sun, Oct 21, 2012 at 06:38:49PM +0200, Ingo Molnar wrote: > > * Jiri Olsa wrote: > > > hi, > > adding support to read sample values through the PERF_SAMPLE_READ > > sample type. It's now possible to specify 'S' modifier for an event > > and get its sample value by PERF_SAMPLE_READ. > > > > F

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-22 Thread Jiri Olsa
On Mon, Oct 22, 2012 at 04:32:11PM +0900, Namhyung Kim wrote: > Hi Jiri, > > It'd be better if you provide a git branch for this series. aaah right, I forgot to mention that.. ;) git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/linux.git perf/group2 thanks, jirka -- To unsubscribe from this

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-22 Thread Namhyung Kim
Hi Jiri, It'd be better if you provide a git branch for this series. Thanks, Namhyung -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the

Re: [PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-21 Thread Ingo Molnar
* Jiri Olsa wrote: > hi, > adding support to read sample values through the PERF_SAMPLE_READ > sample type. It's now possible to specify 'S' modifier for an event > and get its sample value by PERF_SAMPLE_READ. > > For group the 'S' modifier will enable sampling only for the leader > and read a

[PATCH 00/11] perf tool: Add PERF_SAMPLE_READ sample read support

2012-10-20 Thread Jiri Olsa
hi, adding support to read sample values through the PERF_SAMPLE_READ sample type. It's now possible to specify 'S' modifier for an event and get its sample value by PERF_SAMPLE_READ. For group the 'S' modifier will enable sampling only for the leader and read all the group member by PERF_SAMPLE_R