<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:martin.peres@free.fr" title="Martin Peres <martin.peres@free.fr>"> <span class="fn">Martin Peres</span></a>
</span> changed
<a class="bz_bug_link
bz_status_CLOSED bz_closed"
title="CLOSED FIXED - [BAT] igt@kms_psr@* - dmesg-fail/warn - PSR idle timed out 0x3000011[23], atomic update may fail"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107705">bug 107705</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>RESOLVED
</td>
<td>CLOSED
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_CLOSED bz_closed"
title="CLOSED FIXED - [BAT] igt@kms_psr@* - dmesg-fail/warn - PSR idle timed out 0x3000011[23], atomic update may fail"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107705#c2">Comment # 2</a>
on <a class="bz_bug_link
bz_status_CLOSED bz_closed"
title="CLOSED FIXED - [BAT] igt@kms_psr@* - dmesg-fail/warn - PSR idle timed out 0x3000011[23], atomic update may fail"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107705">bug 107705</a>
from <span class="vcard"><a class="email" href="mailto:martin.peres@free.fr" title="Martin Peres <martin.peres@free.fr>"> <span class="fn">Martin Peres</span></a>
</span></b>
<pre>(In reply to Dhinakaran Pandiyan from <a href="show_bug.cgi?id=107705#c1">comment #1</a>)
<span class="quote">> With the below patch, this should now be a test assertion failure instead of
> dmesg-fail
>
> drm/i915/psr: Remove wait_for_idle() for PSR2
> CI runs show PSR2 does not go to IDLE with selective update enabled on
> all PSR exit triggers. Specifically, logs indicate the hardware enters
> "SLEEP Selective Update" and not "IDLE Reset state', like the kernel
> expects, when vblank interrupts are enabled. This check was added for PSR1
> but incorrectly extended to PSR2, remove the check as it breaks tests
> and prints out misleading error messages.
>
> v2: Split out non-code changes (Rodrigo)
>
> Cc: Tarun Vyas <<a href="mailto:tarun.vyas@intel.com">tarun.vyas@intel.com</a>>
> Cc: José Roberto de Souza <<a href="mailto:jose.souza@intel.com">jose.souza@intel.com</a>>
> Cc: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>>
> Fixes: c43dbcbbcc8c ("drm/i915/psr: Lockless version of psr_wait_for_idle")
> Signed-off-by: Dhinakaran Pandiyan <<a href="mailto:dhinakaran.pandiyan@intel.com">dhinakaran.pandiyan@intel.com</a>>
> Reviewed-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>>
> Link:
> <a href="https://patchwork.freedesktop.org/patch/msgid/20180824230844.12428-1">https://patchwork.freedesktop.org/patch/msgid/20180824230844.12428-1</a>-
> <a href="mailto:dhinakaran.pandiyan@intel.com">dhinakaran.pandiyan@intel.com</a></span >
Thanks DK! It indeed did the trick :)</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
<li>You are on the CC list for the bug.</li>
<li>You are the QA Contact for the bug.</li>
</ul>
</body>
</html>