<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:daniel@ffwll.ch" title="Daniel Vetter <daniel@ffwll.ch>"> <span class="fn">Daniel Vetter</span></a>
</span> changed
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [SNB+ Regression]igt/gem_reset_stats some subcases causes system hang sporadically"
href="https://bugs.freedesktop.org/show_bug.cgi?id=86959">bug 86959</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>NEEDINFO
</td>
<td>RESOLVED
</td>
</tr>
<tr>
<td style="text-align:right;">Resolution</td>
<td>---
</td>
<td>FIXED
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [SNB+ Regression]igt/gem_reset_stats some subcases causes system hang sporadically"
href="https://bugs.freedesktop.org/show_bug.cgi?id=86959#c3">Comment # 3</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [SNB+ Regression]igt/gem_reset_stats some subcases causes system hang sporadically"
href="https://bugs.freedesktop.org/show_bug.cgi?id=86959">bug 86959</a>
from <span class="vcard"><a class="email" href="mailto:daniel@ffwll.ch" title="Daniel Vetter <daniel@ffwll.ch>"> <span class="fn">Daniel Vetter</span></a>
</span></b>
<pre>commit f4cbb3a5f707ae4155beaf103adf50351f6509a0
Author: John Harrison <<a href="mailto:John.C.Harrison@Intel.com">John.C.Harrison@Intel.com</a>>
Date: Fri Dec 5 13:49:34 2014 +0000
drm/i915: Zero fill the request structure
There is a general theory that kzmalloc is better/safer than kmalloc,
especially
for interesting data structures. This change updates the request structure
allocation to be zero filled.
This also fixes crashes in the reset code. Quoting Mika's patch:
"Clean the request structure on alloc. Otherwise we might end up
referencing uninitialized fields. This is apparent when we try to
cleanup the preallocated request on ring reset, before any request has
been submitted to the ring. The request->ctx is foobar and we end up
freeing the foobarness."
Note that this fixes a regression introduced in
commit 9eba5d4a1d79d5094321469479b4dbe418f60110
Author: John Harrison <<a href="mailto:John.C.Harrison@Intel.com">John.C.Harrison@Intel.com</a>>
Date: Mon Nov 24 18:49:23 2014 +0000
drm/i915: Ensure OLS & PLR are always in sync
References: <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [SNB+ Regression]igt/gem_reset_stats some subcases causes system hang sporadically"
href="show_bug.cgi?id=86959">https://bugs.freedesktop.org/show_bug.cgi?id=86959</a>
References: <a class="bz_bug_link
bz_status_NEW "
title="NEW - [HSW/BDW Regression]igt/kms_flip/vblank-vs-hang causes system hang"
href="show_bug.cgi?id=86962">https://bugs.freedesktop.org/show_bug.cgi?id=86962</a>
References: <a class="bz_bug_link
bz_status_NEW "
title="NEW - [all Regression]igt/gem_ctx_exec/reset-pin-leak causes system hang"
href="show_bug.cgi?id=86992">https://bugs.freedesktop.org/show_bug.cgi?id=86992</a>
Change-Id: I68715ef758025fab8db763941ef63bf60d7031e2
For: VIZ-4377
Signed-off-by: John Harrison <<a href="mailto:John.C.Harrison@Intel.com">John.C.Harrison@Intel.com</a>>
Reviewed-by: Thomas Daniel <<a href="mailto:Thomas.Daniel@intel.com">Thomas.Daniel@intel.com</a>>
Cc: Mika Kuoppala <<a href="mailto:mika.kuoppala@intel.com">mika.kuoppala@intel.com</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>