Re: [PATCHv3] perf powerpc: Don't call perf_event_disable from atomic context

2016-10-26 Thread Jiri Olsa
On Wed, Oct 26, 2016 at 05:12:49PM +0200, Peter Zijlstra wrote: > On Wed, Oct 26, 2016 at 11:48:24AM +0200, Jiri Olsa wrote: > > > diff --git a/kernel/events/core.c b/kernel/events/core.c > > index c6e47e97b33f..04477983945e 100644 > > --- a/kernel/events/core.c > > +++ b/kernel/events/core.c > >

Re: [PATCHv3] perf powerpc: Don't call perf_event_disable from atomic context

2016-10-26 Thread Peter Zijlstra
On Wed, Oct 26, 2016 at 11:48:24AM +0200, Jiri Olsa wrote: > diff --git a/kernel/events/core.c b/kernel/events/core.c > index c6e47e97b33f..04477983945e 100644 > --- a/kernel/events/core.c > +++ b/kernel/events/core.c > @@ -1960,6 +1960,13 @@ void perf_event_disable(struct perf_event *event) > }

[PATCHv3] perf powerpc: Don't call perf_event_disable from atomic context

2016-10-26 Thread Jiri Olsa
On Wed, Oct 26, 2016 at 10:09:23AM +0800, Huang, Ying wrote: SNIP > > ARGH... so what is the normal metric for this test and did that change? > > And why can't I still find that? These reports suck! > > There is observable changes between the benchmark (will-it-scale) > scores. That is said in