<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:huax.lu@intel.com" title="lu hua <huax.lu@intel.com>"> <span class="fn">lu hua</span></a>
</span> changed
<a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - [SNB+ Bisected]igt/pm_rpm/legacy-planes causes drivers/gpu/drm/i915/intel_uncore.c:47 assert_device_not_suspended+0x45/0x5b"
href="https://bugs.freedesktop.org/show_bug.cgi?id=87180">bug 87180</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>NEEDINFO
</td>
<td>ASSIGNED
</td>
</tr>
<tr>
<td style="text-align:right;">Summary</td>
<td>[SNB+ Regression]igt/pm_rpm/legacy-planes causes drivers/gpu/drm/i915/intel_uncore.c:47 assert_device_not_suspended+0x45/0x5b
</td>
<td>[SNB+ Bisected]igt/pm_rpm/legacy-planes causes drivers/gpu/drm/i915/intel_uncore.c:47 assert_device_not_suspended+0x45/0x5b
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - [SNB+ Bisected]igt/pm_rpm/legacy-planes causes drivers/gpu/drm/i915/intel_uncore.c:47 assert_device_not_suspended+0x45/0x5b"
href="https://bugs.freedesktop.org/show_bug.cgi?id=87180#c4">Comment # 4</a>
on <a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - [SNB+ Bisected]igt/pm_rpm/legacy-planes causes drivers/gpu/drm/i915/intel_uncore.c:47 assert_device_not_suspended+0x45/0x5b"
href="https://bugs.freedesktop.org/show_bug.cgi?id=87180">bug 87180</a>
from <span class="vcard"><a class="email" href="mailto:huax.lu@intel.com" title="lu hua <huax.lu@intel.com>"> <span class="fn">lu hua</span></a>
</span></b>
<pre>(In reply to Paulo Zanoni from <a href="show_bug.cgi?id=87180#c2">comment #2</a>)
<span class="quote">> 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?</span >
I confirmed your bisect result. Thanks.
commit 6beb8c23ebcc3d3287d8a247d11b73d7d0eaa475
Author: Matt Roper <<a href="mailto:matthew.d.roper@intel.com">matthew.d.roper@intel.com</a>>
AuthorDate: Mon Dec 1 15:40:14 2014 -0800
Commit: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>>
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 <<a href="mailto:matthew.d.roper@intel.com">matthew.d.roper@intel.com</a>>
Reviewed-by: Ander Conselvan de Oliveira
<<a href="mailto:ander.conselvan.de.oliveira@intel.com">ander.conselvan.de.oliveira@intel.com</a>>
Signed-off-by: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>>
(In reply to Jani Nikula from <a href="show_bug.cgi?id=87180#c3">comment #3</a>)
<span class="quote">> Does this fix the issue? <a href="http://patchwork.freedesktop.org/patch/38964">http://patchwork.freedesktop.org/patch/38964</a></span >
Apply this patch, it works well.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>