<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:hengx.ding@intel.com" title="Ding Heng <hengx.ding@intel.com>"> <span class="fn">Ding Heng</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - [BYT/BDW bisected]igt/pm_rps/reset sporadically causes system hang"
href="https://bugs.freedesktop.org/show_bug.cgi?id=88654">bug 88654</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;">Summary</td>
<td>[BDW]igt/pm_rps/reset sporadically causes system hang
</td>
<td>[BYT/BDW bisected]igt/pm_rps/reset sporadically causes system hang
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [BYT/BDW bisected]igt/pm_rps/reset sporadically causes system hang"
href="https://bugs.freedesktop.org/show_bug.cgi?id=88654#c1">Comment # 1</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [BYT/BDW bisected]igt/pm_rps/reset sporadically causes system hang"
href="https://bugs.freedesktop.org/show_bug.cgi?id=88654">bug 88654</a>
from <span class="vcard"><a class="email" href="mailto:hengx.ding@intel.com" title="Ding Heng <hengx.ding@intel.com>"> <span class="fn">Ding Heng</span></a>
</span></b>
<pre>b8d24a06568368076ebd5a858a011699a97bfa42 is the first bad commit.
commit b8d24a06568368076ebd5a858a011699a97bfa42
Author: Mika Kuoppala <<a href="mailto:mika.kuoppala@linux.intel.com">mika.kuoppala@linux.intel.com</a>>
AuthorDate: Wed Jan 28 17:03:14 2015 +0200
Commit: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>>
CommitDate: Thu Jan 29 18:03:07 2015 +0100
drm/i915: Remove nested work in gpu error handling
Now when we declare gpu errors only through our own dedicated
hangcheck workqueue there is no need to have a separate workqueue
for handling the resetting and waking up the clients as the deadlock
concerns are no more.
The only exception is i915_debugfs::i915_set_wedged, which triggers
error handling through process context. However as this is only used
through
test harness it is responsibility for test harness not to introduce hangs
through both debug interface and through hangcheck mechanism at the same
time.
Remove gpu_error.work and let the hangcheck work do the tasks it used to.
v2: Add a big warning sign into i915_debugfs::i915_set_wedged (Chris)
Cc: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
Signed-off-by: Mika Kuoppala <<a href="mailto:mika.kuoppala@intel.com">mika.kuoppala@intel.com</a>>
Reviewed-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</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>