Re: [RFC 00/10] perf: Add cputime events/metrics

2018-09-26 Thread Jiri Olsa
On Wed, Sep 26, 2018 at 04:44:08PM +0200, Milian Wolff wrote: > On Thursday, June 7, 2018 1:10:18 AM CEST Andi Kleen wrote: > > > I had some issues with IDLE counter being miscounted due to stopping > > > of the idle tick. I tried to solve it in this patch (it's part of the > > > > > > patchset):

Re: [RFC 00/10] perf: Add cputime events/metrics

2018-09-26 Thread Jiri Olsa
On Wed, Sep 26, 2018 at 04:44:08PM +0200, Milian Wolff wrote: > On Thursday, June 7, 2018 1:10:18 AM CEST Andi Kleen wrote: > > > I had some issues with IDLE counter being miscounted due to stopping > > > of the idle tick. I tried to solve it in this patch (it's part of the > > > > > > patchset):

Re: [RFC 00/10] perf: Add cputime events/metrics

2018-09-26 Thread Milian Wolff
On Thursday, June 7, 2018 1:10:18 AM CEST Andi Kleen wrote: > > I had some issues with IDLE counter being miscounted due to stopping > > of the idle tick. I tried to solve it in this patch (it's part of the > > > > patchset): > > perf/cputime: Don't stop idle tick if there's live cputime event

Re: [RFC 00/10] perf: Add cputime events/metrics

2018-09-26 Thread Milian Wolff
On Thursday, June 7, 2018 1:10:18 AM CEST Andi Kleen wrote: > > I had some issues with IDLE counter being miscounted due to stopping > > of the idle tick. I tried to solve it in this patch (it's part of the > > > > patchset): > > perf/cputime: Don't stop idle tick if there's live cputime event

Re: [RFC 00/10] perf: Add cputime events/metrics

2018-06-06 Thread Andi Kleen
> I had some issues with IDLE counter being miscounted due to stopping > of the idle tick. I tried to solve it in this patch (it's part of the > patchset): > perf/cputime: Don't stop idle tick if there's live cputime event > > but I'm pretty sure it's wrong and there's better solution. At

Re: [RFC 00/10] perf: Add cputime events/metrics

2018-06-06 Thread Andi Kleen
> I had some issues with IDLE counter being miscounted due to stopping > of the idle tick. I tried to solve it in this patch (it's part of the > patchset): > perf/cputime: Don't stop idle tick if there's live cputime event > > but I'm pretty sure it's wrong and there's better solution. At

[RFC 00/10] perf: Add cputime events/metrics

2018-06-06 Thread Jiri Olsa
hi, so.. I failed to make work reliably the exclude_idle bit for cpu-clock event using the idle's process sum_exec_runtime as Peter outlined in his patch [1]. The time jumped up and down and I couldn't make it stable. But I noticed we actually have IDLE stats (and many more) available for each

[RFC 00/10] perf: Add cputime events/metrics

2018-06-06 Thread Jiri Olsa
hi, so.. I failed to make work reliably the exclude_idle bit for cpu-clock event using the idle's process sum_exec_runtime as Peter outlined in his patch [1]. The time jumped up and down and I couldn't make it stable. But I noticed we actually have IDLE stats (and many more) available for each