[Intel-gfx] [PATCH v9 05/21] drm/i915: Add support for per engine reset recovery

Michel Thierry michel.thierry at intel.com
Mon Jun 19 18:42:43 UTC 2017


On 19/06/17 05:46, Chris Wilson wrote:
> Quoting Michel Thierry (2017-06-15 21:18:12)
>> @@ -2992,10 +3014,8 @@ void i915_gem_reset_finish(struct drm_i915_private *dev_priv)
>>
>>         lockdep_assert_held(&dev_priv->drm.struct_mutex);
>>
>> -       for_each_engine(engine, dev_priv, id) {
>> -               tasklet_enable(&engine->irq_tasklet);
>> -               kthread_unpark(engine->breadcrumbs.signaler);
>> -       }
>> +       for_each_engine(engine, dev_priv, id)
>> +               i915_gem_reset_finish_engine(engine);
>
> And we shouldn't forget to clear the active_request here!

The hangcheck.active_request?

And thanks for spotting the break/continue error.

-Michel


More information about the Intel-gfx mailing list