[Bug 107792] [CI][DRMTIP] igt at kms_frontbuffer_tracking@ - incomplete - BUG: unable to handle kernel NULL pointer dereference at 0000000000000080 in intel_psr_set_debugfs_mode

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=107792

Martin Peres <martin.peres at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from Martin Peres <martin.peres at free.fr> ---
(In reply to Chris Wilson from comment #3)
> Actually pushed,
> 
> commit 9d3f8d2ff777b94993581bdfe5c595c619429624
> (drm-intel/drm-intel-next-queued)
> Author: Chris Wilson <chris at chris-wilson.co.uk>
> 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.

Thanks!

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-gfx-bugs/attachments/20180907/c022fba3/attachment.html>