Commit d8cb8875 authored by Mika Kuoppala's avatar Mika Kuoppala Committed by Daniel Vetter

drm/i915: Convert execlist_submit_contexts() for requests

Pass around requests to carry context deeper in callchain.
Signed-off-by: default avatarMika Kuoppala <mika.kuoppala@intel.com>
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent a647828a
...@@ -358,29 +358,29 @@ static int execlists_update_context(struct drm_i915_gem_object *ctx_obj, ...@@ -358,29 +358,29 @@ static int execlists_update_context(struct drm_i915_gem_object *ctx_obj,
return 0; return 0;
} }
static void execlists_submit_contexts(struct intel_engine_cs *ring, static void execlists_submit_requests(struct drm_i915_gem_request *rq0,
struct intel_context *to0, u32 tail0, struct drm_i915_gem_request *rq1)
struct intel_context *to1, u32 tail1)
{ {
struct drm_i915_gem_object *ctx_obj0 = to0->engine[ring->id].state; struct intel_engine_cs *ring = rq0->ring;
struct intel_ringbuffer *ringbuf0 = to0->engine[ring->id].ringbuf; struct drm_i915_gem_object *ctx_obj0 = rq0->ctx->engine[ring->id].state;
struct drm_i915_gem_object *ctx_obj1 = NULL; struct drm_i915_gem_object *ctx_obj1 = NULL;
struct intel_ringbuffer *ringbuf1 = NULL;
BUG_ON(!ctx_obj0); BUG_ON(!ctx_obj0);
WARN_ON(!i915_gem_obj_is_pinned(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); BUG_ON(!ctx_obj1);
WARN_ON(!i915_gem_obj_is_pinned(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); execlists_elsp_write(ring, ctx_obj0, ctx_obj1);
...@@ -443,9 +443,7 @@ static void execlists_context_unqueue(struct intel_engine_cs *ring) ...@@ -443,9 +443,7 @@ static void execlists_context_unqueue(struct intel_engine_cs *ring)
WARN_ON(req1 && req1->elsp_submitted); WARN_ON(req1 && req1->elsp_submitted);
execlists_submit_contexts(ring, req0->ctx, req0->tail, execlists_submit_requests(req0, req1);
req1 ? req1->ctx : NULL,
req1 ? req1->tail : 0);
req0->elsp_submitted++; req0->elsp_submitted++;
if (req1) if (req1)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment