<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:jinxianx.guo@intel.com" title="Guo Jinxian <jinxianx.guo@intel.com>"> <span class="fn">Guo Jinxian</span></a>
</span> changed
<a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED --- - [BDW Bisected]igt/kms_universal_plane/universal-plane-pipe-A-functional fails"
href="https://bugs.freedesktop.org/show_bug.cgi?id=81410">bug 81410</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;">Summary</td>
<td>[BDW Regression]igt/kms_universal_plane/universal-plane-pipe-A-functional fails
</td>
<td>[BDW Bisected]igt/kms_universal_plane/universal-plane-pipe-A-functional fails
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED --- - [BDW Bisected]igt/kms_universal_plane/universal-plane-pipe-A-functional fails"
href="https://bugs.freedesktop.org/show_bug.cgi?id=81410#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_REOPENED "
title="REOPENED --- - [BDW Bisected]igt/kms_universal_plane/universal-plane-pipe-A-functional fails"
href="https://bugs.freedesktop.org/show_bug.cgi?id=81410">bug 81410</a>
from <span class="vcard"><a class="email" href="mailto:jinxianx.guo@intel.com" title="Guo Jinxian <jinxianx.guo@intel.com>"> <span class="fn">Guo Jinxian</span></a>
</span></b>
<pre>ab60c84d87edffe6555fc9f1703e8f9074d06416 is the first bad commit.
commit ab60c84d87edffe6555fc9f1703e8f9074d06416
Author: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a>>
AuthorDate: Mon Sep 1 18:08:25 2014 +0300
Commit: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>>
CommitDate: Tue Sep 2 17:25:47 2014 +0200
drm/i915: Don't call intel_plane_restore() when the prop value didn't
change
No point in calling intel_plane_restore() in .set_property() if the
value didn't change.
More importantly this papers over a bug where the current primary plane
code forgets to update the user coordinates we store under intel_plane
unless the primary plane .update_plane() hook is actually called. This
means we have 0 in the coordinates straight after boot and any call
to intel_restore_plane() (such as from restore_fbdev_mode()) will
actually turn off the primary plane. This mess needs to be fixed properly
but that's a bigger task and the first step there is killing off
intel_pipe_set_base() and just calling the primary plane
.update_plane() hook. For the immediate problem of black screen after
boot this small patch is enough to hide it.
The problem originates from these two commits:
commit 3a5f87c286515c54ff5c52c3e64d0c522b7570c0
Author: Thomas Wood <<a href="mailto:thomas.wood@intel.com">thomas.wood@intel.com</a>>
Date: Wed Aug 20 14:45:00 2014 +0100
drm: fix plane rotation when restoring fbdev configuration
commit d91a2cb8e5104233c02bbde539bd4ee455ec12ac
Author: Sonika Jindal <<a href="mailto:sonika.jindal@intel.com">sonika.jindal@intel.com</a>>
Date: Fri Aug 22 14:06:04 2014 +0530
drm/i915: Add 180 degree primary plane rotation support
Cc: Thomas Wood <<a href="mailto:thomas.wood@intel.com">thomas.wood@intel.com</a>>
Cc: Sonika Jindal <<a href="mailto:sonika.jindal@intel.com">sonika.jindal@intel.com</a>>
Tested-by: Mika Kuoppala <<a href="mailto:mika.kuoppala@linux.intel.com">mika.kuoppala@linux.intel.com</a>>
Signed-off-by: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a>>
Reviewed-by: Damien Lespiau <<a href="mailto:damien.lespiau@intel.com">damien.lespiau@intel.com</a>>
Tested-by: Alan Stern <<a href="mailto:stern@rowland.harvard.edu">stern@rowland.harvard.edu</a>>
Signed-off-by: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>></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>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>