lkml.org 
[lkml]   [2013]   [Mar]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [ 74/75] Revert "drm/i915: enable irqs earlier when resuming"
    From
    On Wed, Mar 27, 2013 at 4:45 PM, Ilya Tumaykin <itumaykin@gmail.com> 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


    \
     
     \ /
      Last update: 2013-03-27 18:41    [W:4.244 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site