Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-18 Thread Namhyung Kim
Hi Arnaldo, Thanks for your work! I'm gonna be offline this week as it's one of the biggest holyday in Korea (and other Asian countries). I'll catch up next week. -- Thanks, Namhyung -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-18 Thread Arnaldo Carvalho de Melo
Em Wed, Sep 18, 2013 at 12:12:53PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Sep 18, 2013 at 11:20:38AM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Tue, Sep 17, 2013 at 02:54:22PM +0900, Namhyung Kim escreveu: > > > On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: > > > >

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-18 Thread Arnaldo Carvalho de Melo
Em Wed, Sep 18, 2013 at 11:20:38AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Tue, Sep 17, 2013 at 02:54:22PM +0900, Namhyung Kim escreveu: > > On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: > > > * Frederic Weisbecker wrote: > > >> My patches and Namhyung's should improve the comm

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-18 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 17, 2013 at 02:54:22PM +0900, Namhyung Kim escreveu: > Hi Ingo, > > On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: > > * Frederic Weisbecker wrote: > >> My patches and Namhyung's should improve the comm situation a lot but we > >> can't do much miracle. The only way would be

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-18 Thread Arnaldo Carvalho de Melo
Em Tue, Sep 17, 2013 at 02:54:22PM +0900, Namhyung Kim escreveu: Hi Ingo, On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: * Frederic Weisbecker fweis...@gmail.com wrote: My patches and Namhyung's should improve the comm situation a lot but we can't do much miracle. The only way

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-18 Thread Arnaldo Carvalho de Melo
Em Wed, Sep 18, 2013 at 11:20:38AM -0300, Arnaldo Carvalho de Melo escreveu: Em Tue, Sep 17, 2013 at 02:54:22PM +0900, Namhyung Kim escreveu: On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: * Frederic Weisbecker fweis...@gmail.com wrote: My patches and Namhyung's should improve the

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-18 Thread Arnaldo Carvalho de Melo
Em Wed, Sep 18, 2013 at 12:12:53PM -0300, Arnaldo Carvalho de Melo escreveu: Em Wed, Sep 18, 2013 at 11:20:38AM -0300, Arnaldo Carvalho de Melo escreveu: Em Tue, Sep 17, 2013 at 02:54:22PM +0900, Namhyung Kim escreveu: On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: Well, it was

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-18 Thread Namhyung Kim
Hi Arnaldo, Thanks for your work! I'm gonna be offline this week as it's one of the biggest holyday in Korea (and other Asian countries). I'll catch up next week. -- Thanks, Namhyung -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-17 Thread Ingo Molnar
* Namhyung Kim wrote: > Hi Ingo, > > On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: > > * Frederic Weisbecker wrote: > >> My patches and Namhyung's should improve the comm situation a lot but we > >> can't do much miracle. The only way would be perhaps to be able to limit > >> the

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-17 Thread Ingo Molnar
* Namhyung Kim namhy...@kernel.org wrote: Hi Ingo, On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: * Frederic Weisbecker fweis...@gmail.com wrote: My patches and Namhyung's should improve the comm situation a lot but we can't do much miracle. The only way would be perhaps to be

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-16 Thread Namhyung Kim
Hi Ingo, On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: > * Frederic Weisbecker wrote: >> My patches and Namhyung's should improve the comm situation a lot but we >> can't do much miracle. The only way would be perhaps to be able to limit >> the deepness of the callchain branches. >>

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-16 Thread Namhyung Kim
Hi Ingo, On Sat, 14 Sep 2013 08:11:49 +0200, Ingo Molnar wrote: * Frederic Weisbecker fweis...@gmail.com wrote: My patches and Namhyung's should improve the comm situation a lot but we can't do much miracle. The only way would be perhaps to be able to limit the deepness of the callchain

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-14 Thread David Ahern
On 9/13/13 5:43 AM, Frederic Weisbecker wrote: The only way would be perhaps to be able to limit the deepness of the callchain branches. I was thinking about such a feature two days ago. Max callchain depth is set to 255 at compile time which can generate huge event sizes. perf needs to

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-14 Thread Ingo Molnar
* Frederic Weisbecker wrote: > On Thu, Sep 12, 2013 at 10:36:58PM +0200, Ingo Molnar wrote: > > > > * Frederic Weisbecker wrote: > > > > > The way we handle hists sorted by comm is to first gather them by tid > > > then in the end merge/collapse hists that end up with the same comm. > > >

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-14 Thread Ingo Molnar
* Frederic Weisbecker fweis...@gmail.com wrote: On Thu, Sep 12, 2013 at 10:36:58PM +0200, Ingo Molnar wrote: * Frederic Weisbecker fweis...@gmail.com wrote: The way we handle hists sorted by comm is to first gather them by tid then in the end merge/collapse hists that end up with

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-14 Thread David Ahern
On 9/13/13 5:43 AM, Frederic Weisbecker wrote: The only way would be perhaps to be able to limit the deepness of the callchain branches. I was thinking about such a feature two days ago. Max callchain depth is set to 255 at compile time which can generate huge event sizes. perf needs to

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-13 Thread Frederic Weisbecker
On Fri, Sep 13, 2013 at 03:32:34PM +0900, Namhyung Kim wrote: > Hi Frederic, > > On Thu, 12 Sep 2013 22:29:39 +0200, Frederic Weisbecker wrote: > > The way we handle hists sorted by comm is to first gather them by tid then > > in the end merge/collapse hists that end up with the same comm. > > >

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-13 Thread Frederic Weisbecker
On Thu, Sep 12, 2013 at 10:36:58PM +0200, Ingo Molnar wrote: > > * Frederic Weisbecker wrote: > > > The way we handle hists sorted by comm is to first gather them by tid > > then in the end merge/collapse hists that end up with the same comm. > > > > But merging hists has shown some

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-13 Thread Namhyung Kim
Hi Frederic, On Thu, 12 Sep 2013 22:29:39 +0200, Frederic Weisbecker wrote: > The way we handle hists sorted by comm is to first gather them by tid then > in the end merge/collapse hists that end up with the same comm. > > But merging hists has shown some performances issues, especially with >

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-13 Thread Namhyung Kim
Hi Frederic, On Thu, 12 Sep 2013 22:29:39 +0200, Frederic Weisbecker wrote: The way we handle hists sorted by comm is to first gather them by tid then in the end merge/collapse hists that end up with the same comm. But merging hists has shown some performances issues, especially with

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-13 Thread Frederic Weisbecker
On Thu, Sep 12, 2013 at 10:36:58PM +0200, Ingo Molnar wrote: * Frederic Weisbecker fweis...@gmail.com wrote: The way we handle hists sorted by comm is to first gather them by tid then in the end merge/collapse hists that end up with the same comm. But merging hists has shown some

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-13 Thread Frederic Weisbecker
On Fri, Sep 13, 2013 at 03:32:34PM +0900, Namhyung Kim wrote: Hi Frederic, On Thu, 12 Sep 2013 22:29:39 +0200, Frederic Weisbecker wrote: The way we handle hists sorted by comm is to first gather them by tid then in the end merge/collapse hists that end up with the same comm. But

[PATCH 0/4] perf tools: New comm infrastructure

2013-09-12 Thread Frederic Weisbecker
The way we handle hists sorted by comm is to first gather them by tid then in the end merge/collapse hists that end up with the same comm. But merging hists has shown some performances issues, especially with callchain where the operation can be very heavy. So this new comm infrastructure aims

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-12 Thread Ingo Molnar
* Frederic Weisbecker wrote: > The way we handle hists sorted by comm is to first gather them by tid > then in the end merge/collapse hists that end up with the same comm. > > But merging hists has shown some performances issues, especially with > callchain where the operation can be very

[PATCH 0/4] perf tools: New comm infrastructure

2013-09-12 Thread Frederic Weisbecker
The way we handle hists sorted by comm is to first gather them by tid then in the end merge/collapse hists that end up with the same comm. But merging hists has shown some performances issues, especially with callchain where the operation can be very heavy. So this new comm infrastructure aims

Re: [PATCH 0/4] perf tools: New comm infrastructure

2013-09-12 Thread Ingo Molnar
* Frederic Weisbecker fweis...@gmail.com wrote: The way we handle hists sorted by comm is to first gather them by tid then in the end merge/collapse hists that end up with the same comm. But merging hists has shown some performances issues, especially with callchain where the operation