drm/i915/gvt: Hold a reference on the request
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 20 Oct 2016 09:29:36 +0000 (17:29 +0800)
committerZhenyu Wang <zhenyuw@linux.intel.com>
Thu, 20 Oct 2016 09:29:36 +0000 (17:29 +0800)
The workload took a pointer to the request, and even waited upon,
without holding a reference on the request. Take that reference
explicitly and fix up the error path following request allocation that
missed flushing the request.

v2: [zhenyuw]
- drop request put in error path for dispatch, as main thread
caller will handle it identically to a real request.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
drivers/gpu/drm/i915/gvt/scheduler.c

index 01d23ad03637ff11551e5a085ed947557c711794..3c2d8e9b2d3fae3cb0586d65bd2e8d9efe137483 100644 (file)
@@ -164,6 +164,7 @@ static int dispatch_workload(struct intel_vgpu_workload *workload)
        int ring_id = workload->ring_id;
        struct i915_gem_context *shadow_ctx = workload->vgpu->shadow_ctx;
        struct drm_i915_private *dev_priv = workload->vgpu->gvt->dev_priv;
+       struct drm_i915_gem_request *rq;
        int ret;
 
        gvt_dbg_sched("ring id %d prepare to dispatch workload %p\n",
@@ -172,17 +173,16 @@ static int dispatch_workload(struct intel_vgpu_workload *workload)
        shadow_ctx->desc_template = workload->ctx_desc.addressing_mode <<
                                    GEN8_CTX_ADDRESSING_MODE_SHIFT;
 
-       workload->req = i915_gem_request_alloc(dev_priv->engine[ring_id],
-                                              shadow_ctx);
-       if (IS_ERR_OR_NULL(workload->req)) {
+       rq = i915_gem_request_alloc(dev_priv->engine[ring_id], shadow_ctx);
+       if (IS_ERR(rq)) {
                gvt_err("fail to allocate gem request\n");
-               workload->status = PTR_ERR(workload->req);
-               workload->req = NULL;
+               workload->status = PTR_ERR(rq);
                return workload->status;
        }
 
-       gvt_dbg_sched("ring id %d get i915 gem request %p\n",
-                       ring_id, workload->req);
+       gvt_dbg_sched("ring id %d get i915 gem request %p\n", ring_id, rq);
+
+       workload->req = i915_gem_request_get(rq);
 
        mutex_lock(&gvt->lock);
 
@@ -209,16 +209,15 @@ static int dispatch_workload(struct intel_vgpu_workload *workload)
        gvt_dbg_sched("ring id %d submit workload to i915 %p\n",
                        ring_id, workload->req);
 
-       i915_add_request_no_flush(workload->req);
-
+       i915_add_request_no_flush(rq);
        workload->dispatched = true;
        return 0;
 err:
        workload->status = ret;
-       if (workload->req)
-               workload->req = NULL;
 
        mutex_unlock(&gvt->lock);
+
+       i915_add_request_no_flush(rq);
        return ret;
 }
 
@@ -459,6 +458,8 @@ complete:
 
                complete_current_workload(gvt, ring_id);
 
+               i915_gem_request_put(fetch_and_zero(&workload->req));
+
                if (need_force_wake)
                        intel_uncore_forcewake_put(gvt->dev_priv,
                                        FORCEWAKE_ALL);