Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 26 Sep 2018 23:48:37 +0200
From:   Jiri Olsa <jolsa@redhat.com>
To:     Milian Wolff <milian.wolff@kdab.com>
Cc:     Andi Kleen <andi@firstfloor.org>, Jiri Olsa <jolsa@kernel.org>,
        Arnaldo Carvalho de Melo <acme@kernel.org>,
        Peter Zijlstra <a.p.zijlstra@chello.nl>,
        lkml <linux-kernel@vger.kernel.org>,
        Ingo Molnar <mingo@kernel.org>,
        Namhyung Kim <namhyung@kernel.org>,
        David Ahern <dsahern@gmail.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Stephane Eranian <eranian@google.com>,
        Frederic Weisbecker <frederic@kernel.org>
Subject: Re: [RFC 00/10] perf: Add cputime events/metrics
Message-ID: <20180926214837.GB15357@krava>
References: <20180606221513.11302-1-jolsa@kernel.org>
 <20180606231016.o3u3u3nduoq7227e@two.firstfloor.org>
 <6856997.NognPVqETE@milian-kdab2>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <6856997.NognPVqETE@milian-kdab2>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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):
> > >   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 least on intel we already have hardware counters for different idle
> > states. You just would need to add them and convert to the same
> > unit.
> > 
> > But of course it's still useful when this is not available.
> > 
> > > My current plan is now to read those counters in perf top/record/report
> > > to show (at least) the idle percentage for the current profile.
> > 
> > It's useful. Thanks for working on it. I was thinking about doing
> > something similar for some time.
> 
> Hey Jiri,
> 
> what happened to this patch series? I also believe it's super useful, even 
> when it's not yet perfect.

heya,
got sidetracked.. thanks for feedback, will refresh
and repost sane version

thanks,
jirka