[Intel-gfx] [PATCH v7 18/63] drm/i915: Populate logical context during first pin.

[Maarten Lankhorst]

This allows us to remove pin_map from state allocation, which saves
us a few retry loops. We won't need this until first pin, anyway.

Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
Reviewed-by: Thomas Hellström <thomas.hellst...@linux.intel.com>
---
 .../drm/i915/gt/intel_execlists_submission.c  | 26 ++++++++++++++++---
 1 file changed, 23 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_execlists_submission.c 
b/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
index e20ab2eab3a8..13e1876a6b6c 100644
--- a/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
+++ b/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
@@ -2456,11 +2456,31 @@ static void execlists_submit_request(struct 
i915_request *request)
        spin_unlock_irqrestore(&engine->active.lock, flags);
 }
 
+static int
+__execlists_context_pre_pin(struct intel_context *ce,
+                           struct intel_engine_cs *engine,
+                           struct i915_gem_ww_ctx *ww, void **vaddr)
+{
+       int err;
+
+       err = lrc_pre_pin(ce, engine, ww, vaddr);
+       if (err)
+               return err;
+
+       if (!__test_and_set_bit(CONTEXT_INIT_BIT, &ce->flags)) {
+               lrc_init_state(ce, engine, *vaddr);
+
+                __i915_gem_object_flush_map(ce->state->obj, 0, 
engine->context_size);
+       }
+
+       return 0;
+}
+
 static int execlists_context_pre_pin(struct intel_context *ce,
                                     struct i915_gem_ww_ctx *ww,
                                     void **vaddr)
 {
-       return lrc_pre_pin(ce, ce->engine, ww, vaddr);
+       return __execlists_context_pre_pin(ce, ce->engine, ww, vaddr);
 }
 
 static int execlists_context_pin(struct intel_context *ce, void *vaddr)
@@ -3366,8 +3386,8 @@ static int virtual_context_pre_pin(struct intel_context 
*ce,
 {
        struct virtual_engine *ve = container_of(ce, typeof(*ve), context);
 
-       /* Note: we must use a real engine class for setting up reg state */
-       return lrc_pre_pin(ce, ve->siblings[0], ww, vaddr);
+        /* Note: we must use a real engine class for setting up reg state */
+       return __execlists_context_pre_pin(ce, ve->siblings[0], ww, vaddr);
 }
 
 static int virtual_context_pin(struct intel_context *ce, void *vaddr)
-- 
2.30.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx