On Sun, 6 May 2012 17:34:53 +0200 Daniel Vetter <daniel at 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 at chris-wilson.co.uk> wrote: > > > > > Signed-off-by: Chris Wilson <chris at 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 at 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
