<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:huax.lu@intel.com" title="lu hua <huax.lu@intel.com>"> <span class="fn">lu hua</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [PNV/ILK Regression]X will be no response when run all piglit cases"
href="https://bugs.freedesktop.org/show_bug.cgi?id=76671">bug 76671</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>NEW
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [PNV/ILK Regression]X will be no response when run all piglit cases"
href="https://bugs.freedesktop.org/show_bug.cgi?id=76671#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [PNV/ILK Regression]X will be no response when run all piglit cases"
href="https://bugs.freedesktop.org/show_bug.cgi?id=76671">bug 76671</a>
from <span class="vcard"><a class="email" href="mailto:huax.lu@intel.com" title="lu hua <huax.lu@intel.com>"> <span class="fn">lu hua</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=76671#c4">comment #4</a>)
<span class="quote">> I think:
>
> diff --git a/drivers/gpu/drm/i915/i915_gem_context.c
> b/drivers/gpu/drm/i915/i915_gem_context.c
> index 021e21223c9c..91000ff1b626 100644
> --- a/drivers/gpu/drm/i915/i915_gem_context.c
> +++ b/drivers/gpu/drm/i915/i915_gem_context.c
> @@ -506,9 +506,6 @@ int i915_gem_context_enable(struct drm_i915_private
> *dev_priv)
> struct intel_ring_buffer *ring;
> int ret, i;
>
> - if (!HAS_HW_CONTEXTS(dev_priv->dev))
> - return 0;
> -
> /* This is the only place the aliasing PPGTT gets enabled, which means
> * it has to happen before we bail on reset */
> if (dev_priv->mm.aliasing_ppgtt) {
> @@ -833,7 +830,13 @@ int i915_switch_context(struct intel_ring_buffer *ring,
>
> /* We have the fake context */
> if (!HAS_HW_CONTEXTS(ring->dev)) {
> - ring->last_context = to;
> + if (to != ring->last_context) {
> + if (to)
> + i915_gem_context_reference(to);
> + if (ring->last_context)
> + i915_gem_context_unreference(ring->last_context);
> + ring->last_context = to;
> + }
> return 0;
> }</span >
Test this patch, system boot fail.</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>