From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915: add rc6 residency times to debugfs Date: Sun, 25 Mar 2012 01:01:10 +0100 Message-ID: <20120325000110.GC18786@phenom.ffwll.local> References: <1332466967-4584-1-git-send-email-ben@bwidawsk.net> <20120324145801.GB7563@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f169.google.com (mail-wi0-f169.google.com [209.85.212.169]) by gabe.freedesktop.org (Postfix) with ESMTP id 968259E771 for ; Sat, 24 Mar 2012 17:00:25 -0700 (PDT) Received: by wibhm17 with SMTP id hm17so2732029wib.0 for ; Sat, 24 Mar 2012 17:00:24 -0700 (PDT) In-Reply-To: <20120324145801.GB7563@phenom.ffwll.local> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Eugeni Dodonov Cc: Ben Widawsky , Arjan van de Ven , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Sat, Mar 24, 2012 at 03:58:01PM +0100, Daniel Vetter wrote: > On Fri, Mar 23, 2012 at 01:43:22PM -0300, Eugeni Dodonov wrote: > > On Thu, Mar 22, 2012 at 22:42, Ben Widawsky wrote: > > > > > RC6 residency should be in intervals of 1.28us, and the counter wraps. > > > Here is an example using awk to get the RC6 and RC6+ residency in > > > seconds, since boot. > > > > > > cat /sys/kernel/debug/dri/0/i915_drpc_info | grep residency | awk -F':' > > > -F' ' '{print $5 * 1.28 / 1000000}' > > > > > > This is primarily for QA, but has other applications as well. > > > > > > > Note that we already have this in intel_reg_dumper as well, since > > http://cgit.freedesktop.org/xorg/app/intel-gpu-tools/commit/?id=47a5bc505cb972cc94f1fbd9882d4e5c2e72ac43 > > . > > > > But if it makes sense to include it into kernel as well, why not? > > > > Reviewed-by: Eugeni Dodonov > > Queued for -next, thanks for the patch and review. Unqueued after some irc discussion with Ben, looks like we don't yet fully understand how this works (and at least an rc6pp reg seems to be missing). -Daniel -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48