From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Widawsky Subject: Re: [PATCH v2 3/3] tests: rc6 residency test Date: Mon, 26 Mar 2012 14:54:35 -0700 Message-ID: <20120326145435.7a296690@bwidawsk.net> References: <1332722012-4853-1-git-send-email-ben@bwidawsk.net> <1332722012-4853-6-git-send-email-ben@bwidawsk.net> <20120326093713.GK4014@phenom.ffwll.local> <20120326094055.GL4014@phenom.ffwll.local> 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 832419ED63 for ; Mon, 26 Mar 2012 14:54:48 -0700 (PDT) In-Reply-To: <20120326094055.GL4014@phenom.ffwll.local> 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: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Mon, 26 Mar 2012 11:40:55 +0200 Daniel Vetter wrote: > On Mon, Mar 26, 2012 at 11:37:13AM +0200, Daniel Vetter wrote: > > On Sun, Mar 25, 2012 at 05:33:32PM -0700, Ben Widawsky wrote: > > > This is meant to test the sysfs entry for showing rc6 residency in > > > milliseconds. Remember, sysfs is a permanent interface. > > > > > > v2: use new get_card interface to try "all" devices > > > check rc6p and rc6pp in addition to rc6 > > > > > > Signed-off-by: Ben Widawsky > > > > You're still missing the drm_quiescent_gpu(fd) call before doing the > > residency measurements. Which makes this test fail reliably in the > > i-g-t testsuite because the gpu is still busy from previous tests. > > Please run new tests also in the context of the full testuite to > > catch such issues. > > Actually the quiescent seems to be there, so I don't know what's > going on. -Daniel I ran this on IVB, both individual and with make test (Ken's machine), and it passed. Ben