RE: [PATCH 1/1] perf/core: fix implicitly enable dynamic interrupt throttle

2016-05-12 Thread Liang, Kan
> > > Hmm, would it not be nicer to simply reject the write instead of silently > ignoring it? > Yes, I agree. Thanks, Kan > --- > kernel/events/core.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/kernel/events/core.c b/kernel/events/core.c index >

RE: [PATCH 1/1] perf/core: fix implicitly enable dynamic interrupt throttle

2016-05-12 Thread Liang, Kan
> > > Hmm, would it not be nicer to simply reject the write instead of silently > ignoring it? > Yes, I agree. Thanks, Kan > --- > kernel/events/core.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/kernel/events/core.c b/kernel/events/core.c index >

Re: [PATCH 1/1] perf/core: fix implicitly enable dynamic interrupt throttle

2016-05-12 Thread Peter Zijlstra
On Tue, May 03, 2016 at 12:26:06AM -0700, kan.li...@intel.com wrote: > From: Kan Liang > > This patch fixes an issue which was introduced from 'commit 91a612eea9a3 > ("perf/core: Fix dynamic interrupt throttle")' > The old patch unconditionally sets the

Re: [PATCH 1/1] perf/core: fix implicitly enable dynamic interrupt throttle

2016-05-12 Thread Peter Zijlstra
On Tue, May 03, 2016 at 12:26:06AM -0700, kan.li...@intel.com wrote: > From: Kan Liang > > This patch fixes an issue which was introduced from 'commit 91a612eea9a3 > ("perf/core: Fix dynamic interrupt throttle")' > The old patch unconditionally sets the perf_sample_allowed_ns value to > !0. But