On Thu, Mar 14, 2013 at 05:52:04PM +0200, Mika Kuoppala wrote: > @@ -2101,6 +2107,10 @@ static void i915_gem_reset_ring_lists(struct drm_i915_private *dev_priv, > > list_del(&request->list); > i915_gem_request_remove_from_client(request); > + > + if (request->ctx) > + kref_put(&request->ctx->ref, i915_gem_context_free); > + > kfree(request); Methinks it is time to kill this code duplication. -Chris -- Chris Wilson, Intel Open Source Technology Centre