drm/i915: enable irqs earlier when resuming
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 5 Mar 2013 08:50:58 +0000 (09:50 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Tue, 5 Mar 2013 09:21:36 +0000 (10:21 +0100)
commit17dbd0c24d3295c3c4287311f2615a3bd61a01e1
treec509c1cf3b205796816c69d53bd1feb4c3e7d822
parent927ebeabff19982a53d6b41ce41f5967cd72f567
drm/i915: enable irqs earlier when resuming

We need it to restore the ilk rc6 context, since the gpu wait no
requires interrupts. But in general having interrupts around should
help in code sanity, since more and more stuff is interrupt driven.

This regression has been introduced in

commit 8efe3e7f75824af43dd058c613dfc80495d06739
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date:   Tue Nov 27 16:22:54 2012 +0000

    drm/i915: Rearrange code to only have a single method for waiting upon the ring

Like in the driver load code we need to make sure that hotplug
interrupts don't cause havoc with our modeset state, hence block them
with the existing infrastructure. Again we ignore races where we might
loose hotplug interrupts ...

Note that the driver load part of the regression has already been
fixed in

commit b594adb6d5b4b11061377c6428d5649bc6e47af2
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Sat Dec 1 21:03:22 2012 +0100

    drm/i915: reorder setup sequence to have irqs for output setup

v2: Add a note to the commit message about which patch fixed the
driver load part of the regression. Stable kernels need to backport
both patches.

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=54691
Cc: stable@vger.kernel.org (for 3.8 only, plese backport
    b594adb6d5b4b1106 first)
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@intel.com>
Reported-and-Tested-by: Ilya Tumaykin <itumaykin@gmail.com>
Reviewed-by: Chris wilson <chris@chris-wilson.co.uk> (v1)
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_drv.c