Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-09 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Tue, Dec 08, 2015 at 09:57:07AM +0100, Peter Zijlstra wrote: > > On Tue, Dec 08, 2015 at 09:50:26AM +0100, Peter Zijlstra wrote: > > > On Tue, Dec 08, 2015 at 06:36:04AM +0100, Ingo Molnar wrote: > > > > > > > So I checked my NHM box with your latest queue and it no

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-08 Thread Peter Zijlstra
On Tue, Dec 08, 2015 at 09:57:07AM +0100, Peter Zijlstra wrote: > On Tue, Dec 08, 2015 at 09:50:26AM +0100, Peter Zijlstra wrote: > > On Tue, Dec 08, 2015 at 06:36:04AM +0100, Ingo Molnar wrote: > > > > > So I checked my NHM box with your latest queue and it now works > > > correctly. Do you > >

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-08 Thread Peter Zijlstra
On Tue, Dec 08, 2015 at 09:50:26AM +0100, Peter Zijlstra wrote: > On Tue, Dec 08, 2015 at 06:36:04AM +0100, Ingo Molnar wrote: > > > So I checked my NHM box with your latest queue and it now works correctly. > > Do you > > have any idea what the difference is/was? > > Sadly, no clue :/ > > I w

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-08 Thread Peter Zijlstra
On Tue, Dec 08, 2015 at 06:36:04AM +0100, Ingo Molnar wrote: > So I checked my NHM box with your latest queue and it now works correctly. Do > you > have any idea what the difference is/was? Sadly, no clue :/ I went over those patches and cannot find anything that should affect NHM (or <=SNB i

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-07 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Sun, Dec 06, 2015 at 02:11:02PM +0100, Ingo Molnar wrote: > > > the symptom is that latest 'perf top' and 'perf record' produces no samples. > > > > So I've removed this commit and the related Skylake commit from -tip: > > > > 4576ceaa56a8 perf/x86: Use INST_RETI

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-07 Thread Peter Zijlstra
On Mon, Dec 07, 2015 at 11:02:58AM -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, Dec 07, 2015 at 07:48:41AM +0100, Ingo Molnar escreveu: > > * Peter Zijlstra wrote: > > > On Sun, Dec 06, 2015 at 02:11:02PM +0100, Ingo Molnar wrote: > > > > Also, I'm not convinced we need a new 'ppp' qualifier f

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-07 Thread Arnaldo Carvalho de Melo
Em Mon, Dec 07, 2015 at 07:48:41AM +0100, Ingo Molnar escreveu: > * Peter Zijlstra wrote: > > On Sun, Dec 06, 2015 at 02:11:02PM +0100, Ingo Molnar wrote: > > > Also, I'm not convinced we need a new 'ppp' qualifier for any of this, > > > why not > > > just replace 'pp' with this event - 'pp' is

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-07 Thread Peter Zijlstra
On Sun, Dec 06, 2015 at 02:11:02PM +0100, Ingo Molnar wrote: > the symptom is that latest 'perf top' and 'perf record' produces no samples. > > So I've removed this commit and the related Skylake commit from -tip: > > 4576ceaa56a8 perf/x86: Use INST_RETIRED.PREC_DIST for cycles:ppp > ac1e1d30c

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-07 Thread Peter Zijlstra
On Mon, Dec 07, 2015 at 07:48:41AM +0100, Ingo Molnar wrote: > > * Peter Zijlstra wrote: > > > On Sun, Dec 06, 2015 at 02:11:02PM +0100, Ingo Molnar wrote: > > > > > Also, I'm not convinced we need a new 'ppp' qualifier for any of this, > > > why not > > > just replace 'pp' with this event - '

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-06 Thread Ingo Molnar
* Peter Zijlstra wrote: > On Sun, Dec 06, 2015 at 02:11:02PM +0100, Ingo Molnar wrote: > > > Also, I'm not convinced we need a new 'ppp' qualifier for any of this, why > > not > > just replace 'pp' with this event - 'pp' is meant to be our most precise > > event. > > I requested this because

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-06 Thread Peter Zijlstra
On Sun, Dec 06, 2015 at 02:11:02PM +0100, Ingo Molnar wrote: > Also, I'm not convinced we need a new 'ppp' qualifier for any of this, why > not > just replace 'pp' with this event - 'pp' is meant to be our most precise > event. I requested this because the PREC_DIST events can only be scheduled

Re: [tip:perf/core] perf/x86: Use INST_RETIRED.PREC_DIST for cycles: ppp

2015-12-06 Thread Ingo Molnar
* tip-bot for Andi Kleen wrote: > Commit-ID: 4576ceaa56a86bd0c041c204d51c3f3ca582a49c > Gitweb: http://git.kernel.org/tip/4576ceaa56a86bd0c041c204d51c3f3ca582a49c > Author: Andi Kleen > AuthorDate: Mon, 30 Nov 2015 16:28:09 -0800 > Committer: Ingo Molnar > CommitDate: Fri, 4 Dec 2015