<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:mike@fireburn.co.uk" title="Mike Lothian <mike@fireburn.co.uk>"> <span class="fn">Mike Lothian</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - Continuious warnings from agd5f 5.3-wip branch"
href="https://bugs.freedesktop.org/show_bug.cgi?id=110949">bug 110949</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;">CC</td>
<td>
</td>
<td>mike@fireburn.co.uk
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Continuious warnings from agd5f 5.3-wip branch"
href="https://bugs.freedesktop.org/show_bug.cgi?id=110949#c1">Comment # 1</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Continuious warnings from agd5f 5.3-wip branch"
href="https://bugs.freedesktop.org/show_bug.cgi?id=110949">bug 110949</a>
from <span class="vcard"><a class="email" href="mailto:mike@fireburn.co.uk" title="Mike Lothian <mike@fireburn.co.uk>"> <span class="fn">Mike Lothian</span></a>
</span></b>
<pre>37fb6e8a96fbc9c809c58f9490267ffe7101ac33 is the first bad commit
commit 37fb6e8a96fbc9c809c58f9490267ffe7101ac33
Author: Nicholas Kazlauskas <<a href="mailto:nicholas.kazlauskas@amd.com">nicholas.kazlauskas@amd.com</a>>
Date: Fri Jun 7 11:16:55 2019 -0400
drm/amd/display: Enable fast plane updates when state->allow_modeset = true
[Why]
Whenever the a modeset is allowed (but not neccessarily required) we
currently recreate all the planes in the state. Most IGT tests and
legacy IOCTLs create atomic commits with this flag set, so the pipes
are often unnecessarily reprogrammed.
Poor performance and stuttering can occur when many of these commits
are frequently issued.
This flag was needed when the appropriate conditions for checking
whether the planes needed a reset were not in place, but
should_reset_plane should cover everything needed now.
[How]
Drop the check for state->allow_modeset in should_reset_plane.
All planes on a CRTC should reset in the following conditions:
- The CRTC needs a modeset
- The CRTC degamma changes
- Planes are added or removed to the CRTC
These conditions are all covered in should_reset_plane.
We still can't drop the format change check in should_reset_plane since
fill_dc_plane_info_and_addr isn't called when validating the state, so
we can't tell if a FULL update is needed or not.
Signed-off-by: Nicholas Kazlauskas <<a href="mailto:nicholas.kazlauskas@amd.com">nicholas.kazlauskas@amd.com</a>>
Reviewed-by: David Francis <<a href="mailto:david.francis@amd.com">david.francis@amd.com</a>>
Signed-off-by: Alex Deucher <<a href="mailto:alexander.deucher@amd.com">alexander.deucher@amd.com</a>>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 8 --------
1 file changed, 8 deletions(-)</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>