summaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/i915/intel_lrc.c
diff options
context:
space:
mode:
authorMika Kuoppala2015-07-03 16:09:32 +0200
committerDaniel Vetter2015-07-06 16:44:34 +0200
commitd8cb8875ac60e2832614a2c48af236b5899cf209 (patch)
tree890c2d26597899284337d0c624e03bcbff057ef1 /drivers/gpu/drm/i915/intel_lrc.c
parentdrm/i915: Also perform gpu reset under execlist mode. (diff)
downloadkernel-qcow2-linux-d8cb8875ac60e2832614a2c48af236b5899cf209.tar.gz
kernel-qcow2-linux-d8cb8875ac60e2832614a2c48af236b5899cf209.tar.xz
kernel-qcow2-linux-d8cb8875ac60e2832614a2c48af236b5899cf209.zip
drm/i915: Convert execlist_submit_contexts() for requests
Pass around requests to carry context deeper in callchain. Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_lrc.c')
-rw-r--r--drivers/gpu/drm/i915/intel_lrc.c30
1 files changed, 14 insertions, 16 deletions
diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
index a499f16db194..7282466ee567 100644
--- a/drivers/gpu/drm/i915/intel_lrc.c
+++ b/drivers/gpu/drm/i915/intel_lrc.c
@@ -358,29 +358,29 @@ static int execlists_update_context(struct drm_i915_gem_object *ctx_obj,
return 0;
}
-static void execlists_submit_contexts(struct intel_engine_cs *ring,
- struct intel_context *to0, u32 tail0,
- struct intel_context *to1, u32 tail1)
+static void execlists_submit_requests(struct drm_i915_gem_request *rq0,
+ struct drm_i915_gem_request *rq1)
{
- struct drm_i915_gem_object *ctx_obj0 = to0->engine[ring->id].state;
- struct intel_ringbuffer *ringbuf0 = to0->engine[ring->id].ringbuf;
+ struct intel_engine_cs *ring = rq0->ring;
+ struct drm_i915_gem_object *ctx_obj0 = rq0->ctx->engine[ring->id].state;
struct drm_i915_gem_object *ctx_obj1 = NULL;
- struct intel_ringbuffer *ringbuf1 = NULL;
BUG_ON(!ctx_obj0);
WARN_ON(!i915_gem_obj_is_pinned(ctx_obj0));
- WARN_ON(!i915_gem_obj_is_pinned(ringbuf0->obj));
+ WARN_ON(!i915_gem_obj_is_pinned(rq0->ringbuf->obj));
- execlists_update_context(ctx_obj0, ringbuf0->obj, to0->ppgtt, tail0);
+ execlists_update_context(ctx_obj1, rq0->ringbuf->obj,
+ rq0->ctx->ppgtt, rq0->tail);
+
+ if (rq1) {
+ ctx_obj1 = rq1->ctx->engine[ring->id].state;
- if (to1) {
- ringbuf1 = to1->engine[ring->id].ringbuf;
- ctx_obj1 = to1->engine[ring->id].state;
BUG_ON(!ctx_obj1);
WARN_ON(!i915_gem_obj_is_pinned(ctx_obj1));
- WARN_ON(!i915_gem_obj_is_pinned(ringbuf1->obj));
+ WARN_ON(!i915_gem_obj_is_pinned(rq1->ringbuf->obj));
- execlists_update_context(ctx_obj1, ringbuf1->obj, to1->ppgtt, tail1);
+ execlists_update_context(ctx_obj1, rq1->ringbuf->obj,
+ rq1->ctx->ppgtt, rq1->tail);
}
execlists_elsp_write(ring, ctx_obj0, ctx_obj1);
@@ -443,9 +443,7 @@ static void execlists_context_unqueue(struct intel_engine_cs *ring)
WARN_ON(req1 && req1->elsp_submitted);
- execlists_submit_contexts(ring, req0->ctx, req0->tail,
- req1 ? req1->ctx : NULL,
- req1 ? req1->tail : 0);
+ execlists_submit_requests(req0, req1);
req0->elsp_submitted++;
if (req1)