@@ -186,6 +186,8 @@ intel_write_status_page(struct intel_engine_cs *engine, int reg, u32 value)
#define I915_GEM_HWS_PREEMPT_ADDR (I915_GEM_HWS_PREEMPT * sizeof(u32))
#define I915_GEM_HWS_SEQNO 0x40
#define I915_GEM_HWS_SEQNO_ADDR (I915_GEM_HWS_SEQNO * sizeof(u32))
+#define I915_GEM_HWS_BLITTER 0x42
+#define I915_GEM_HWS_BLITTER_ADDR (I915_GEM_HWS_BLITTER * sizeof(u32))
#define I915_GEM_HWS_SCRATCH 0x80
#define I915_HWS_CSB_BUF0_INDEX 0x10
@@ -819,6 +819,7 @@ create_pinned_context(struct intel_engine_cs *engine,
int err;
ce = intel_context_create(engine);
+
if (IS_ERR(ce))
return ce;
@@ -851,6 +852,20 @@ create_kernel_context(struct intel_engine_cs *engine)
&kernel, "kernel_context");
}
+static struct intel_context *
+create_blitter_context(struct intel_engine_cs *engine)
+{
+ static struct lock_class_key blitter;
+ struct intel_context *ce;
+
+ ce = create_pinned_context(engine, I915_GEM_HWS_BLITTER_ADDR, &blitter,
+ "blitter_context");
+ if (IS_ERR(ce))
+ return ce;
+
+ return ce;
+}
+
/**
* intel_engines_init_common - initialize cengine state which might require hw access
* @engine: Engine to initialize.
@@ -881,17 +896,33 @@ static int engine_init_common(struct intel_engine_cs *engine)
if (IS_ERR(ce))
return PTR_ERR(ce);
+ engine->kernel_context = ce;
ret = measure_breadcrumb_dw(ce);
if (ret < 0)
goto err_context;
engine->emit_fini_breadcrumb_dw = ret;
- engine->kernel_context = ce;
+
+ /*
+ * The blitter context is used to quickly memset or migrate objects
+ * in local memory, so it has to always be available.
+ */
+ if (engine->class == COPY_ENGINE_CLASS) {
+ ce = create_blitter_context(engine);
+ if (IS_ERR(ce)) {
+ ret = PTR_ERR(ce);
+ goto err_unpin;
+ }
+
+ engine->blitter_context = ce;
+ }
return 0;
+err_unpin:
+ intel_context_unpin(engine->kernel_context);
err_context:
- intel_context_put(ce);
+ intel_context_put(engine->kernel_context);
return ret;
}
@@ -947,6 +978,11 @@ void intel_engine_cleanup_common(struct intel_engine_cs *engine)
if (engine->default_state)
fput(engine->default_state);
+ if (engine->blitter_context) {
+ intel_context_unpin(engine->blitter_context);
+ intel_context_put(engine->blitter_context);
+ }
+
if (engine->kernel_context) {
intel_context_unpin(engine->kernel_context);
intel_context_put(engine->kernel_context);
@@ -347,6 +347,7 @@ struct intel_engine_cs {
struct llist_head barrier_tasks;
struct intel_context *kernel_context; /* pinned */
+ struct intel_context *blitter_context; /* pinned; exists for BCS only */
intel_engine_mask_t saturated; /* submitting semaphores too late? */
We may be without a context to perform various internal blitter operations, for example when performing object migration. Piggybacking off the kernel_context is probably a bad idea, since it has other uses. Signed-off-by: Matthew Auld <matthew.auld@intel.com> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: Abdiel Janulgue <abdiel.janulgue@linux.intel.com> --- drivers/gpu/drm/i915/gt/intel_engine.h | 2 + drivers/gpu/drm/i915/gt/intel_engine_cs.c | 40 +++++++++++++++++++- drivers/gpu/drm/i915/gt/intel_engine_types.h | 1 + 3 files changed, 41 insertions(+), 2 deletions(-)