On Wed, Mar 27, 2013 at 4:45 PM, Ilya Tumaykin <itumaykin@xxxxxxxxx> wrote: > 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. Nack. This thing blew up all over the place and caused about 5 different regressions. It works all correctly in 3.9 due to the different context (completely revamped init sequence around irq handling). I realize that this means that ilk rc6 is broken in 3.8 (and we can't really fix it without either going back to the ilk rc6 code in 3.6 or copying the entire 3.9 irq init sequence, both a bit too intrusive for stable). But: - ilk rc6 is known to hang machines - ilk rc6 is (opposed to rc6 on snb+) not know to do anything good like safe power And so disabled by default. So I hope that regression is bearable, feel free to beat the drm/i915 guys as usual for their failings ;-) -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch -- 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