Commit 8753181e authored by John Harrison's avatar John Harrison Committed by Daniel Vetter

drm/i915: Update init_context() to take a request structure

Now that everything above has been converted to use requests, it is possible to
update init_context() to take a request pointer instead of a ring/context pair.

For: VIZ-5115
Signed-off-by: default avatarJohn Harrison <John.C.Harrison@Intel.com>
Reviewed-by: default avatarTomas Elf <tomas.elf@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 76c39168
...@@ -418,7 +418,7 @@ int i915_gem_context_enable(struct drm_i915_gem_request *req) ...@@ -418,7 +418,7 @@ int i915_gem_context_enable(struct drm_i915_gem_request *req)
if (ring->init_context == NULL) if (ring->init_context == NULL)
return 0; return 0;
ret = ring->init_context(ring, ring->default_context); ret = ring->init_context(req);
} else } else
ret = i915_switch_context(req); ret = i915_switch_context(req);
...@@ -760,7 +760,7 @@ static int do_switch(struct drm_i915_gem_request *req) ...@@ -760,7 +760,7 @@ static int do_switch(struct drm_i915_gem_request *req)
if (uninitialized) { if (uninitialized) {
if (ring->init_context) { if (ring->init_context) {
ret = ring->init_context(ring, to); ret = ring->init_context(req);
if (ret) if (ret)
DRM_ERROR("ring init context: %d\n", ret); DRM_ERROR("ring init context: %d\n", ret);
} }
......
...@@ -1602,16 +1602,15 @@ static int intel_lr_context_render_state_init(struct intel_engine_cs *ring, ...@@ -1602,16 +1602,15 @@ static int intel_lr_context_render_state_init(struct intel_engine_cs *ring,
return ret; return ret;
} }
static int gen8_init_rcs_context(struct intel_engine_cs *ring, static int gen8_init_rcs_context(struct drm_i915_gem_request *req)
struct intel_context *ctx)
{ {
int ret; int ret;
ret = intel_logical_ring_workarounds_emit(ring, ctx); ret = intel_logical_ring_workarounds_emit(req->ring, req->ctx);
if (ret) if (ret)
return ret; return ret;
return intel_lr_context_render_state_init(ring, ctx); return intel_lr_context_render_state_init(req->ring, req->ctx);
} }
/** /**
...@@ -2236,7 +2235,7 @@ int intel_lr_context_deferred_create(struct intel_context *ctx, ...@@ -2236,7 +2235,7 @@ int intel_lr_context_deferred_create(struct intel_context *ctx,
if (ret) if (ret)
return ret; return ret;
ret = ring->init_context(req->ring, ctx); ret = ring->init_context(req);
if (ret) { if (ret) {
DRM_ERROR("ring init context: %d\n", ret); DRM_ERROR("ring init context: %d\n", ret);
i915_gem_request_cancel(req); i915_gem_request_cancel(req);
......
...@@ -742,16 +742,15 @@ static int intel_ring_workarounds_emit(struct intel_engine_cs *ring, ...@@ -742,16 +742,15 @@ static int intel_ring_workarounds_emit(struct intel_engine_cs *ring,
return 0; return 0;
} }
static int intel_rcs_ctx_init(struct intel_engine_cs *ring, static int intel_rcs_ctx_init(struct drm_i915_gem_request *req)
struct intel_context *ctx)
{ {
int ret; int ret;
ret = intel_ring_workarounds_emit(ring, ctx); ret = intel_ring_workarounds_emit(req->ring, req->ctx);
if (ret != 0) if (ret != 0)
return ret; return ret;
ret = i915_gem_render_state_init(ring); ret = i915_gem_render_state_init(req->ring);
if (ret) if (ret)
DRM_ERROR("init render state: %d\n", ret); DRM_ERROR("init render state: %d\n", ret);
......
...@@ -176,8 +176,7 @@ struct intel_engine_cs { ...@@ -176,8 +176,7 @@ struct intel_engine_cs {
int (*init_hw)(struct intel_engine_cs *ring); int (*init_hw)(struct intel_engine_cs *ring);
int (*init_context)(struct intel_engine_cs *ring, int (*init_context)(struct drm_i915_gem_request *req);
struct intel_context *ctx);
void (*write_tail)(struct intel_engine_cs *ring, void (*write_tail)(struct intel_engine_cs *ring,
u32 value); u32 value);
......
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