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 > 050a290c72c7..0a51a568d

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 t

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

2016-05-03 Thread kan . liang
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 that could trigger an issue in the following corner case. The user can disable the