<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 - [CI][DRMTIP] igt@kms_frontbuffer_tracking@ - incomplete - BUG: unable to handle kernel NULL pointer dereference at 0000000000000080 in intel_psr_set_debugfs_mode"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107792">bug 107792</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 - [CI][DRMTIP] igt@kms_frontbuffer_tracking@ - incomplete - BUG: unable to handle kernel NULL pointer dereference at 0000000000000080 in intel_psr_set_debugfs_mode"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107792#c4">Comment # 4</a>
on <a class="bz_bug_link
bz_status_CLOSED bz_closed"
title="CLOSED FIXED - [CI][DRMTIP] igt@kms_frontbuffer_tracking@ - incomplete - BUG: unable to handle kernel NULL pointer dereference at 0000000000000080 in intel_psr_set_debugfs_mode"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107792">bug 107792</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 Chris Wilson from <a href="show_bug.cgi?id=107792#c3">comment #3</a>)
<span class="quote">> Actually pushed,
>
> commit 9d3f8d2ff777b94993581bdfe5c595c619429624
> (drm-intel/drm-intel-next-queued)
> Author: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
> Date: Tue Sep 4 17:29:02 2018 +0100
>
> drm/i915: Be defensive and don't assume PSR has any commit to sync
> against
>
> If the previous modeset commit has completed and is no longer part of
> the crtc state, skip waiting for it.
>
> Ville pointed out that, in fact, the commit is never removed after a
> modeset so the only way we could see a NULL here should be if there was
> never a commit attached. Nevertheless, we have the evidence it can be
> NULL and it has been defended against elsewhere, for example commit
> 93313538c153 ("drm/i915: Pass idle crtc_state to intel_dp_sink_crc").
>
> That we also check for commit being NULL elsewhere was the clincher.</span >
Thanks!</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>