From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jesse Barnes <jbarnes@virtuousgeek.org>
Subject: Re: [PATCH 1/2] drm/i915: Support pageflipping
 interrupts for all 3-pipes on IVB
Date: Sun, 6 May 2012 09:30:58 -0700
Message-ID: <20120506093058.7147578d@jbarnes-x220>
References: <1335948733-20561-1-git-send-email-chris@chris-wilson.co.uk>
	<20120502124642.2ec29c9b@jbarnes-desktop>
	<20120506153452.GA4756@phenom.ffwll.local>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org>
Received: from oproxy6-pub.bluehost.com (oproxy6-pub.bluehost.com
	[67.222.54.6])
	by gabe.freedesktop.org (Postfix) with SMTP id 247479E75A
	for <intel-gfx@lists.freedesktop.org>;
	Sun,  6 May 2012 09:31:24 -0700 (PDT)
In-Reply-To: <20120506153452.GA4756@phenom.ffwll.local>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
	<mailto:intel-gfx-request@lists.freedesktop.org?subject=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 <daniel@ffwll.ch>
Cc: intel-gfx@lists.freedesktop.org
List-Id: intel-gfx@lists.freedesktop.org

On Sun, 6 May 2012 17:34:53 +0200
Daniel Vetter <daniel@ffwll.ch> wrote:

> On Wed, May 02, 2012 at 12:46:42PM -0700, Jesse Barnes wrote:
> > On Wed,  2 May 2012 09:52:12 +0100
> > Chris Wilson <chris@chris-wilson.co.uk> wrote:
> > 
> > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> > > ---
> > >  drivers/gpu/drm/i915/i915_irq.c |   31
> > > ++++++++++++++++++++++--------- drivers/gpu/drm/i915/i915_reg.h
> > > |    7 +++++-- 2 files changed, 27 insertions(+), 11 deletions(-)
> > > 
> > 
> > Highly embarrassing.
> > 
> > Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>
> 
> Even more embarrassing that I've failed to pick this up until I've
> noticed that lack of a merge conflict that should be there when
> mergin in -fixes. We really need some automated tests to check
> whether pageflipping actually works on all crtcs ...

Yeah definitely.  We have modetest but I'm not sure if that's part of
the QA suite.  I'll add something small and targeted to i-g-t for both
vblank and flipping.  I think Shuang was supposed to do this, but he's
busy with other things...

Jesse