[Bug 87180] [SNB+ Bisected]igt/pm_rpm/legacy-planes causes drivers/gpu/drm/i915/intel_uncore.c:47 assert_device_not_suspended+0x45/0x5b

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=87180

lu hua <huax.lu at intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |ASSIGNED
            Summary|[SNB+                       |[SNB+
                   |Regression]igt/pm_rpm/legac |Bisected]igt/pm_rpm/legacy-
                   |y-planes causes             |planes causes
                   |drivers/gpu/drm/i915/intel_ |drivers/gpu/drm/i915/intel_
                   |uncore.c:47                 |uncore.c:47
                   |assert_device_not_suspended |assert_device_not_suspended
                   |+0x45/0x5b                  |+0x45/0x5b

--- Comment #4 from lu hua <huax.lu at intel.com> ---
(In reply to Paulo Zanoni from comment #2)
> Ok, so I got curious why this was not "caught" by PRTS so I decided to
> bisect it myself.
> 
> Apparently, what happened is:
> - The test was PASSing
> - Then some commit made it execute BUG()
> - Then some commit fixed the BUG(), but started giving us WARN()s.
> 
> Apparently the first bad commit is:
> 
> 6beb8c23ebcc3d3287d8a247d11b73d7d0eaa475
> drm/i915: Consolidate plane 'prepare' functions (v2)
> 
> Matt, can you please take a look at this?

I confirmed your bisect result. Thanks.
commit 6beb8c23ebcc3d3287d8a247d11b73d7d0eaa475
Author:     Matt Roper <matthew.d.roper at intel.com>
AuthorDate: Mon Dec 1 15:40:14 2014 -0800
Commit:     Daniel Vetter <daniel.vetter at ffwll.ch>
CommitDate: Sat Dec 6 01:46:24 2014 +0100

    drm/i915: Consolidate plane 'prepare' functions (v2)

    The 'prepare' step for all types of planes are pretty similar;
    consolidate the three 'prepare' functions into a single function.  This
    paves the way for future integration with the atomic plane handlers.

    Note that we pull the 'wait for pending flips' functionality out of the
    primary plane's prepare step and place it directly in the 'setplane'
    code.  When we move to the atomic plane handlers, this code will be in
    the 'atomic begin' step.

    v2: Update GEM fb tracking for physical cursors also (Ander)

    Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
    Reviewed-by: Ander Conselvan de Oliveira
<ander.conselvan.de.oliveira at intel.com>
    Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>

(In reply to Jani Nikula from comment #3)
> Does this fix the issue? http://patchwork.freedesktop.org/patch/38964

Apply this patch, it works well.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20141216/9f0ec5a0/attachment-0001.html>