On 18.03.13 14:07:37 Greg Kroah-Hartman wrote: > 3.8-stable review patch. If anyone has any objections, please let me know. Hello, Greg. The original "drm/i915: enable irqs earlier when resuming" commit was introduced in order to fix bug appeared in 3.8 kernels. Reverting this commit you also bring back that bug. Bugzilla link: https://bugzilla.kernel.org/show_bug.cgi?id=54691#c17 Sorry for late reply, I wasn't sure if this is simple revert or another mechanism was introduced by devs to avoid mentioned bug and that fix became unnecessary. Please undo this revert. > ------------------ > > This reverts commit 31f14f4219d2a74b7a6d86c7798f49141b5eccbe which was > commit 15239099d7a7a9ecdc1ccb5b187ae4cda5488ff9 upstream. > > It caused problems in the 3.8-stable series, but 3.9-rc is just fine. > > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Mika Kuoppala <mika.kuoppala@xxxxxxxxx> > Cc: Ilya Tumaykin <itumaykin@xxxxxxxxx> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> > > --- > drivers/gpu/drm/i915/i915_drv.c | 13 +------------ > 1 file changed, 1 insertion(+), 12 deletions(-) > > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -486,7 +486,6 @@ static int i915_drm_freeze(struct drm_de > intel_modeset_disable(dev); > > drm_irq_uninstall(dev); > - dev_priv->enable_hotplug_processing = false; > } > > i915_save_state(dev); > @@ -563,19 +562,9 @@ static int __i915_drm_thaw(struct drm_de > error = i915_gem_init_hw(dev); > mutex_unlock(&dev->struct_mutex); > > - /* We need working interrupts for modeset enabling ... */ > - drm_irq_install(dev); > - > intel_modeset_init_hw(dev); > intel_modeset_setup_hw_state(dev, false); > - > - /* > - * ... but also need to make sure that hotplug processing > - * doesn't cause havoc. Like in the driver load code we don't > - * bother with the tiny race here where we might loose hotplug > - * notifications. > - * */ > - dev_priv->enable_hotplug_processing = true; > + drm_irq_install(dev); > } > > intel_opregion_init(dev); -- Best regards. Tumaykin Ilya. -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html