From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Widawsky Subject: Re: [PATCH] drm/i915: add rc6 residency times to debugfs Date: Fri, 23 Mar 2012 11:44:10 -0700 Message-ID: <20120323114410.4259f450@bwidawsk.net> References: <1332466967-4584-1-git-send-email-ben@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from cloud01.chad-versace.us (184-106-247-128.static.cloud-ips.com [184.106.247.128]) by gabe.freedesktop.org (Postfix) with ESMTP id 097B39E775 for ; Fri, 23 Mar 2012 11:44:24 -0700 (PDT) In-Reply-To: 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: intel-gfx@lists.freedesktop.org, Arjan van de Ven List-Id: intel-gfx@lists.freedesktop.org On Fri, 23 Mar 2012 13:43:22 -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? > Yeah, reg dumper does a whole lot of other stuff in addition to this. I'd agree it wouldn't be worth adding a new debugfs entry though since reg dumper does it (obviously intel_reg_read does it too :)). > Reviewed-by: Eugeni Dodonov >