[PATCH 38/42] drm/i915: Pass engine to engine->emit_bb_start()
Chris Wilson
chris at chris-wilson.co.uk
Wed Feb 10 11:39:41 UTC 2021
Pass the physical engine to the vfuncs as in the future that may not be
directly available from i915_request.
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
.../gpu/drm/i915/gem/i915_gem_client_blt.c | 2 +-
.../gpu/drm/i915/gem/i915_gem_execbuffer.c | 6 +++---
.../gpu/drm/i915/gem/i915_gem_object_blt.c | 4 ++--
.../i915/gem/selftests/i915_gem_client_blt.c | 9 ++++----
.../drm/i915/gem/selftests/i915_gem_context.c | 10 ++++++---
.../drm/i915/gem/selftests/i915_gem_mman.c | 2 +-
.../drm/i915/gem/selftests/igt_gem_utils.c | 9 ++++----
drivers/gpu/drm/i915/gt/gen2_engine_cs.c | 13 +++++++-----
drivers/gpu/drm/i915/gt/gen2_engine_cs.h | 9 +++++---
drivers/gpu/drm/i915/gt/gen6_engine_cs.c | 6 ++++--
drivers/gpu/drm/i915/gt/gen6_engine_cs.h | 6 ++++--
drivers/gpu/drm/i915/gt/gen8_engine_cs.c | 8 ++++---
drivers/gpu/drm/i915/gt/gen8_engine_cs.h | 6 ++++--
drivers/gpu/drm/i915/gt/intel_engine_types.h | 3 ++-
drivers/gpu/drm/i915/gt/intel_renderstate.c | 4 ++--
.../gpu/drm/i915/gt/intel_ring_submission.c | 2 +-
drivers/gpu/drm/i915/gt/selftest_engine_cs.c | 20 +++++++++---------
drivers/gpu/drm/i915/gt/selftest_execlists.c | 21 ++++++++++---------
drivers/gpu/drm/i915/gt/selftest_hangcheck.c | 4 +++-
drivers/gpu/drm/i915/gt/selftest_rps.c | 12 +++++------
.../gpu/drm/i915/gt/selftest_workarounds.c | 4 ++--
drivers/gpu/drm/i915/i915_perf.c | 11 +++++-----
drivers/gpu/drm/i915/selftests/i915_gem_gtt.c | 2 +-
drivers/gpu/drm/i915/selftests/i915_perf.c | 8 +++----
drivers/gpu/drm/i915/selftests/i915_request.c | 8 ++++---
drivers/gpu/drm/i915/selftests/igt_spinner.c | 4 +++-
26 files changed, 111 insertions(+), 82 deletions(-)
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_client_blt.c b/drivers/gpu/drm/i915/gem/i915_gem_client_blt.c
index fb457f854ba0..7837ac881d18 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_client_blt.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_client_blt.c
@@ -218,7 +218,7 @@ static void clear_pages_worker(struct work_struct *work)
goto out_request;
}
- err = engine->emit_bb_start(rq,
+ err = engine->emit_bb_start(engine, rq,
batch->node.start, batch->node.size,
0);
out_request:
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c
index aaf4cfa200fe..efbe1e92a3ab 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_execbuffer.c
@@ -1337,7 +1337,7 @@ static int __reloc_gpu_alloc(struct i915_execbuffer *eb,
if (err)
goto err_request;
- err = eb->engine->emit_bb_start(rq,
+ err = eb->engine->emit_bb_start(eb->engine, rq,
batch->node.start, PAGE_SIZE,
cache->gen > 5 ? 0 : I915_DISPATCH_SECURE);
if (err)
@@ -2553,7 +2553,7 @@ static int eb_submit(struct i915_execbuffer *eb, struct i915_vma *batch)
return err;
}
- err = eb->engine->emit_bb_start(eb->request,
+ err = eb->engine->emit_bb_start(eb->engine, eb->request,
batch->node.start +
eb->batch_start_offset,
eb->batch_len,
@@ -2563,7 +2563,7 @@ static int eb_submit(struct i915_execbuffer *eb, struct i915_vma *batch)
if (eb->trampoline) {
GEM_BUG_ON(eb->batch_start_offset);
- err = eb->engine->emit_bb_start(eb->request,
+ err = eb->engine->emit_bb_start(eb->engine, eb->request,
eb->trampoline->node.start +
eb->batch_len,
0, 0);
diff --git a/drivers/gpu/drm/i915/gem/i915_gem_object_blt.c b/drivers/gpu/drm/i915/gem/i915_gem_object_blt.c
index a0a68549580e..817956c58cf4 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_object_blt.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_object_blt.c
@@ -199,7 +199,7 @@ int i915_gem_object_fill_blt(struct drm_i915_gem_object *obj,
err = engine->emit_init_breadcrumb(engine, rq);
if (likely(!err))
- err = engine->emit_bb_start(rq,
+ err = engine->emit_bb_start(engine, rq,
batch->node.start,
batch->node.size,
0);
@@ -424,7 +424,7 @@ int i915_gem_object_copy_blt(struct drm_i915_gem_object *src,
goto out_request;
}
- err = engine->emit_bb_start(rq,
+ err = engine->emit_bb_start(engine, rq,
batch->node.start, batch->node.size,
0);
diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_client_blt.c b/drivers/gpu/drm/i915/gem/selftests/i915_gem_client_blt.c
index 175581724d44..6b435f722512 100644
--- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_client_blt.c
+++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_client_blt.c
@@ -453,6 +453,7 @@ tiled_blit(struct tiled_blits *t,
struct blit_buffer *dst, u64 dst_addr,
struct blit_buffer *src, u64 src_addr)
{
+ struct intel_engine_cs *engine = t->ce->engine;
struct i915_request *rq;
int err;
@@ -490,10 +491,10 @@ tiled_blit(struct tiled_blits *t,
if (!err)
err = move_to_active(dst->vma, rq, 0);
if (!err)
- err = rq->engine->emit_bb_start(rq,
- t->batch->node.start,
- t->batch->node.size,
- 0);
+ err = engine->emit_bb_start(engine, rq,
+ t->batch->node.start,
+ t->batch->node.size,
+ 0);
i915_request_get(rq);
i915_request_add(rq);
if (i915_request_wait(rq, 0, HZ / 2) < 0)
diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c b/drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c
index defea10c7347..03c6cab447d1 100644
--- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c
+++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_context.c
@@ -992,7 +992,7 @@ emit_rpcs_query(struct drm_i915_gem_object *obj,
goto skip_request;
}
- err = engine->emit_bb_start(rq,
+ err = engine->emit_bb_start(engine, rq,
batch->node.start, batch->node.size,
0);
if (err)
@@ -1562,7 +1562,9 @@ static int write_to_scratch(struct i915_gem_context *ctx,
goto skip_request;
}
- err = engine->emit_bb_start(rq, vma->node.start, vma->node.size, 0);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start, vma->node.size,
+ 0);
if (err)
goto skip_request;
@@ -1700,7 +1702,9 @@ static int read_from_scratch(struct i915_gem_context *ctx,
goto skip_request;
}
- err = engine->emit_bb_start(rq, vma->node.start, vma->node.size, flags);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start, vma->node.size,
+ flags);
if (err)
goto skip_request;
diff --git a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
index 39293d98f34d..cb4d84c9179a 100644
--- a/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
+++ b/drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c
@@ -1145,7 +1145,7 @@ static int __igt_mmap_gpu(struct drm_i915_private *i915,
if (err == 0)
err = i915_vma_move_to_active(vma, rq, 0);
- err = engine->emit_bb_start(rq, vma->node.start, 0, 0);
+ err = engine->emit_bb_start(engine, rq, vma->node.start, 0, 0);
i915_request_get(rq);
i915_request_add(rq);
diff --git a/drivers/gpu/drm/i915/gem/selftests/igt_gem_utils.c b/drivers/gpu/drm/i915/gem/selftests/igt_gem_utils.c
index b7e064667d39..f351ff63defb 100644
--- a/drivers/gpu/drm/i915/gem/selftests/igt_gem_utils.c
+++ b/drivers/gpu/drm/i915/gem/selftests/igt_gem_utils.c
@@ -112,12 +112,13 @@ int igt_gpu_fill_dw(struct intel_context *ce,
struct i915_vma *vma, u64 offset,
unsigned long count, u32 val)
{
+ struct intel_engine_cs *engine = ce->engine;
struct i915_request *rq;
struct i915_vma *batch;
unsigned int flags;
int err;
- GEM_BUG_ON(!intel_engine_can_store_dword(ce->engine));
+ GEM_BUG_ON(!intel_engine_can_store_dword(engine));
GEM_BUG_ON(!i915_vma_is_pinned(vma));
batch = igt_emit_store_dw(vma, offset, count, val);
@@ -152,9 +153,9 @@ int igt_gpu_fill_dw(struct intel_context *ce,
if (INTEL_GEN(ce->vm->i915) <= 5)
flags |= I915_DISPATCH_SECURE;
- err = rq->engine->emit_bb_start(rq,
- batch->node.start, batch->node.size,
- flags);
+ err = engine->emit_bb_start(engine, rq,
+ batch->node.start, batch->node.size,
+ flags);
skip_request:
if (err)
diff --git a/drivers/gpu/drm/i915/gt/gen2_engine_cs.c b/drivers/gpu/drm/i915/gt/gen2_engine_cs.c
index 7829aff8bfb8..4da0a63c231d 100644
--- a/drivers/gpu/drm/i915/gt/gen2_engine_cs.c
+++ b/drivers/gpu/drm/i915/gt/gen2_engine_cs.c
@@ -261,15 +261,16 @@ int gen4_emit_init_breadcrumb_xcs(const struct intel_engine_cs *engine,
#define I830_BATCH_LIMIT SZ_256K
#define I830_TLB_ENTRIES (2)
#define I830_WA_SIZE max(I830_TLB_ENTRIES * SZ_4K, I830_BATCH_LIMIT)
-int i830_emit_bb_start(struct i915_request *rq,
+int i830_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
unsigned int dispatch_flags)
{
u32 *cs, cs_offset =
- intel_gt_scratch_offset(rq->engine->gt,
+ intel_gt_scratch_offset(engine->gt,
INTEL_GT_SCRATCH_FIELD_DEFAULT);
- GEM_BUG_ON(rq->engine->gt->scratch->size < I830_WA_SIZE);
+ GEM_BUG_ON(engine->gt->scratch->size < I830_WA_SIZE);
cs = intel_ring_begin(rq, 6);
if (IS_ERR(cs))
@@ -326,7 +327,8 @@ int i830_emit_bb_start(struct i915_request *rq,
return 0;
}
-int gen3_emit_bb_start(struct i915_request *rq,
+int gen3_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
unsigned int dispatch_flags)
{
@@ -346,7 +348,8 @@ int gen3_emit_bb_start(struct i915_request *rq,
return 0;
}
-int gen4_emit_bb_start(struct i915_request *rq,
+int gen4_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 length,
unsigned int dispatch_flags)
{
diff --git a/drivers/gpu/drm/i915/gt/gen2_engine_cs.h b/drivers/gpu/drm/i915/gt/gen2_engine_cs.h
index a07b163608e2..f22e912ffbe6 100644
--- a/drivers/gpu/drm/i915/gt/gen2_engine_cs.h
+++ b/drivers/gpu/drm/i915/gt/gen2_engine_cs.h
@@ -34,13 +34,16 @@ u32 *gen4_emit_breadcrumb_xcs(const struct intel_engine_cs *engine,
int gen4_emit_init_breadcrumb_xcs(const struct intel_engine_cs *engine,
struct i915_request *rq);
-int i830_emit_bb_start(struct i915_request *rq,
+int i830_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
unsigned int dispatch_flags);
-int gen3_emit_bb_start(struct i915_request *rq,
+int gen3_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
unsigned int dispatch_flags);
-int gen4_emit_bb_start(struct i915_request *rq,
+int gen4_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 length,
unsigned int dispatch_flags);
diff --git a/drivers/gpu/drm/i915/gt/gen6_engine_cs.c b/drivers/gpu/drm/i915/gt/gen6_engine_cs.c
index 6d9279af2762..fe06554729f9 100644
--- a/drivers/gpu/drm/i915/gt/gen6_engine_cs.c
+++ b/drivers/gpu/drm/i915/gt/gen6_engine_cs.c
@@ -247,7 +247,8 @@ int gen6_emit_flush_vcs(const struct intel_engine_cs *engine,
return gen6_flush_dw(rq, mode, MI_INVALIDATE_TLB | MI_INVALIDATE_BSD);
}
-int gen6_emit_bb_start(struct i915_request *rq,
+int gen6_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
unsigned int dispatch_flags)
{
@@ -269,7 +270,8 @@ int gen6_emit_bb_start(struct i915_request *rq,
}
int
-hsw_emit_bb_start(struct i915_request *rq,
+hsw_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
unsigned int dispatch_flags)
{
diff --git a/drivers/gpu/drm/i915/gt/gen6_engine_cs.h b/drivers/gpu/drm/i915/gt/gen6_engine_cs.h
index 9d752f9b7873..b1812bb4ecd8 100644
--- a/drivers/gpu/drm/i915/gt/gen6_engine_cs.h
+++ b/drivers/gpu/drm/i915/gt/gen6_engine_cs.h
@@ -41,10 +41,12 @@ u32 *gen7_emit_breadcrumb_xcs(const struct intel_engine_cs *engine,
struct i915_request *rq,
u32 *cs);
-int gen6_emit_bb_start(struct i915_request *rq,
+int gen6_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
unsigned int dispatch_flags);
-int hsw_emit_bb_start(struct i915_request *rq,
+int hsw_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
unsigned int dispatch_flags);
diff --git a/drivers/gpu/drm/i915/gt/gen8_engine_cs.c b/drivers/gpu/drm/i915/gt/gen8_engine_cs.c
index 85cdd777f54f..4e8dddc817a5 100644
--- a/drivers/gpu/drm/i915/gt/gen8_engine_cs.c
+++ b/drivers/gpu/drm/i915/gt/gen8_engine_cs.c
@@ -393,7 +393,8 @@ int gen8_emit_init_breadcrumb(const struct intel_engine_cs *engine,
return 0;
}
-int gen8_emit_bb_start_noarb(struct i915_request *rq,
+int gen8_emit_bb_start_noarb(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
const unsigned int flags)
{
@@ -429,14 +430,15 @@ int gen8_emit_bb_start_noarb(struct i915_request *rq,
return 0;
}
-int gen8_emit_bb_start(struct i915_request *rq,
+int gen8_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
const unsigned int flags)
{
u32 *cs;
if (unlikely(i915_request_has_nopreempt(rq)))
- return gen8_emit_bb_start_noarb(rq, offset, len, flags);
+ return gen8_emit_bb_start_noarb(engine, rq, offset, len, flags);
cs = intel_ring_begin(rq, 6);
if (IS_ERR(cs))
diff --git a/drivers/gpu/drm/i915/gt/gen8_engine_cs.h b/drivers/gpu/drm/i915/gt/gen8_engine_cs.h
index 3225c186e363..c4b57fb0d801 100644
--- a/drivers/gpu/drm/i915/gt/gen8_engine_cs.h
+++ b/drivers/gpu/drm/i915/gt/gen8_engine_cs.h
@@ -36,10 +36,12 @@ int gen12_emit_flush_xcs(const struct intel_engine_cs *engine,
int gen8_emit_init_breadcrumb(const struct intel_engine_cs *engine,
struct i915_request *rq);
-int gen8_emit_bb_start_noarb(struct i915_request *rq,
+int gen8_emit_bb_start_noarb(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
const unsigned int flags);
-int gen8_emit_bb_start(struct i915_request *rq,
+int gen8_emit_bb_start(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 len,
const unsigned int flags);
diff --git a/drivers/gpu/drm/i915/gt/intel_engine_types.h b/drivers/gpu/drm/i915/gt/intel_engine_types.h
index 8bc0881989f8..b2865d4cd76c 100644
--- a/drivers/gpu/drm/i915/gt/intel_engine_types.h
+++ b/drivers/gpu/drm/i915/gt/intel_engine_types.h
@@ -394,7 +394,8 @@ struct intel_engine_cs {
#define EMIT_FLUSH BIT(1)
#define EMIT_BARRIER (EMIT_INVALIDATE | EMIT_FLUSH)
- int (*emit_bb_start)(struct i915_request *rq,
+ int (*emit_bb_start)(const struct intel_engine_cs *engine,
+ struct i915_request *rq,
u64 offset, u32 length,
unsigned int dispatch_flags);
#define I915_DISPATCH_SECURE BIT(0)
diff --git a/drivers/gpu/drm/i915/gt/intel_renderstate.c b/drivers/gpu/drm/i915/gt/intel_renderstate.c
index 0f7c0a148b80..6b98f73fa48e 100644
--- a/drivers/gpu/drm/i915/gt/intel_renderstate.c
+++ b/drivers/gpu/drm/i915/gt/intel_renderstate.c
@@ -219,14 +219,14 @@ int intel_renderstate_emit(struct intel_renderstate *so,
if (err)
return err;
- err = engine->emit_bb_start(rq,
+ err = engine->emit_bb_start(engine, rq,
so->batch_offset, so->batch_size,
I915_DISPATCH_SECURE);
if (err)
return err;
if (so->aux_size > 8) {
- err = engine->emit_bb_start(rq,
+ err = engine->emit_bb_start(engine, rq,
so->aux_offset, so->aux_size,
I915_DISPATCH_SECURE);
if (err)
diff --git a/drivers/gpu/drm/i915/gt/intel_ring_submission.c b/drivers/gpu/drm/i915/gt/intel_ring_submission.c
index acd76e475f4f..d30ae2139483 100644
--- a/drivers/gpu/drm/i915/gt/intel_ring_submission.c
+++ b/drivers/gpu/drm/i915/gt/intel_ring_submission.c
@@ -806,7 +806,7 @@ clear_residuals(struct intel_engine_cs *engine, struct i915_request *rq)
return ret;
}
- ret = engine->emit_bb_start(rq,
+ ret = engine->emit_bb_start(engine, rq,
engine->wa_ctx.vma->node.start, 0,
0);
if (ret)
diff --git a/drivers/gpu/drm/i915/gt/selftest_engine_cs.c b/drivers/gpu/drm/i915/gt/selftest_engine_cs.c
index e33ec4e3b35d..c7f912140f2f 100644
--- a/drivers/gpu/drm/i915/gt/selftest_engine_cs.c
+++ b/drivers/gpu/drm/i915/gt/selftest_engine_cs.c
@@ -161,9 +161,9 @@ static int perf_mi_bb_start(void *arg)
if (err)
goto out;
- err = rq->engine->emit_bb_start(rq,
- batch->node.start, 8,
- 0);
+ err = engine->emit_bb_start(engine, rq,
+ batch->node.start, 8,
+ 0);
if (err)
goto out;
@@ -301,9 +301,9 @@ static int perf_mi_noop(void *arg)
if (err)
goto out;
- err = rq->engine->emit_bb_start(rq,
- base->node.start, 8,
- 0);
+ err = engine->emit_bb_start(engine, rq,
+ base->node.start, 8,
+ 0);
if (err)
goto out;
@@ -311,10 +311,10 @@ static int perf_mi_noop(void *arg)
if (err)
goto out;
- err = rq->engine->emit_bb_start(rq,
- nop->node.start,
- nop->node.size,
- 0);
+ err = engine->emit_bb_start(engine, rq,
+ nop->node.start,
+ nop->node.size,
+ 0);
if (err)
goto out;
diff --git a/drivers/gpu/drm/i915/gt/selftest_execlists.c b/drivers/gpu/drm/i915/gt/selftest_execlists.c
index e90d0f4fef15..5a8e97e74982 100644
--- a/drivers/gpu/drm/i915/gt/selftest_execlists.c
+++ b/drivers/gpu/drm/i915/gt/selftest_execlists.c
@@ -2744,9 +2744,9 @@ static int create_gang(struct intel_engine_cs *engine,
if (!err)
err = i915_vma_move_to_active(vma, rq, 0);
if (!err)
- err = rq->engine->emit_bb_start(rq,
- vma->node.start,
- PAGE_SIZE, 0);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start,
+ PAGE_SIZE, 0);
i915_vma_unlock(vma);
i915_request_add(rq);
if (err)
@@ -3168,9 +3168,9 @@ create_gpr_client(struct intel_engine_cs *engine,
if (!err)
err = i915_vma_move_to_active(batch, rq, 0);
if (!err)
- err = rq->engine->emit_bb_start(rq,
- batch->node.start,
- PAGE_SIZE, 0);
+ err = engine->emit_bb_start(engine, rq,
+ batch->node.start,
+ PAGE_SIZE, 0);
i915_vma_unlock(batch);
i915_vma_unpin(batch);
@@ -3466,6 +3466,7 @@ static int smoke_submit(struct preempt_smoke *smoke,
struct i915_gem_context *ctx, int prio,
struct drm_i915_gem_object *batch)
{
+ struct intel_engine_cs *engine = smoke->engine;
struct i915_request *rq;
struct i915_vma *vma = NULL;
int err = 0;
@@ -3486,7 +3487,7 @@ static int smoke_submit(struct preempt_smoke *smoke,
ctx->sched.priority = prio;
- rq = igt_request_alloc(ctx, smoke->engine);
+ rq = igt_request_alloc(ctx, engine);
if (IS_ERR(rq)) {
err = PTR_ERR(rq);
goto unpin;
@@ -3498,9 +3499,9 @@ static int smoke_submit(struct preempt_smoke *smoke,
if (!err)
err = i915_vma_move_to_active(vma, rq, 0);
if (!err)
- err = rq->engine->emit_bb_start(rq,
- vma->node.start,
- PAGE_SIZE, 0);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start,
+ PAGE_SIZE, 0);
i915_vma_unlock(vma);
}
diff --git a/drivers/gpu/drm/i915/gt/selftest_hangcheck.c b/drivers/gpu/drm/i915/gt/selftest_hangcheck.c
index ea05892a4d94..3050fa68e188 100644
--- a/drivers/gpu/drm/i915/gt/selftest_hangcheck.c
+++ b/drivers/gpu/drm/i915/gt/selftest_hangcheck.c
@@ -246,7 +246,9 @@ hang_create_request(struct hang *h, struct intel_engine_cs *engine)
if (INTEL_GEN(gt->i915) <= 5)
flags |= I915_DISPATCH_SECURE;
- err = engine->emit_bb_start(rq, vma->node.start, PAGE_SIZE, flags);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start, PAGE_SIZE,
+ flags);
cancel_rq:
if (err) {
diff --git a/drivers/gpu/drm/i915/gt/selftest_rps.c b/drivers/gpu/drm/i915/gt/selftest_rps.c
index 967641fee42a..faefe1c5d1f1 100644
--- a/drivers/gpu/drm/i915/gt/selftest_rps.c
+++ b/drivers/gpu/drm/i915/gt/selftest_rps.c
@@ -654,9 +654,9 @@ int live_rps_frequency_cs(void *arg)
if (!err)
err = i915_vma_move_to_active(vma, rq, 0);
if (!err)
- err = rq->engine->emit_bb_start(rq,
- vma->node.start,
- PAGE_SIZE, 0);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start,
+ PAGE_SIZE, 0);
i915_request_add(rq);
if (err)
goto err_vma;
@@ -795,9 +795,9 @@ int live_rps_frequency_srm(void *arg)
if (!err)
err = i915_vma_move_to_active(vma, rq, 0);
if (!err)
- err = rq->engine->emit_bb_start(rq,
- vma->node.start,
- PAGE_SIZE, 0);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start,
+ PAGE_SIZE, 0);
i915_request_add(rq);
if (err)
goto err_vma;
diff --git a/drivers/gpu/drm/i915/gt/selftest_workarounds.c b/drivers/gpu/drm/i915/gt/selftest_workarounds.c
index 6f09716ce75c..98dd6fa56fcd 100644
--- a/drivers/gpu/drm/i915/gt/selftest_workarounds.c
+++ b/drivers/gpu/drm/i915/gt/selftest_workarounds.c
@@ -583,7 +583,7 @@ static int check_dirty_whitelist(struct intel_context *ce)
if (err)
goto err_request;
- err = engine->emit_bb_start(rq,
+ err = engine->emit_bb_start(engine, rq,
batch->node.start, PAGE_SIZE,
0);
if (err)
@@ -865,7 +865,7 @@ static int scrub_whitelisted_registers(struct intel_context *ce)
goto err_request;
/* Perform the writes from an unprivileged "user" batch */
- err = engine->emit_bb_start(rq, batch->node.start, 0, 0);
+ err = engine->emit_bb_start(engine, rq, batch->node.start, 0, 0);
err_request:
err = request_add_sync(rq, err);
diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c
index 93f3e2c5c89a..9d807b0a8ce2 100644
--- a/drivers/gpu/drm/i915/i915_perf.c
+++ b/drivers/gpu/drm/i915/i915_perf.c
@@ -1939,6 +1939,7 @@ emit_oa_config(struct i915_perf_stream *stream,
struct intel_context *ce,
struct i915_active *active)
{
+ struct intel_engine_cs *engine = ce->engine;
struct i915_request *rq;
struct i915_vma *vma;
struct i915_gem_ww_ctx ww;
@@ -1958,9 +1959,9 @@ emit_oa_config(struct i915_perf_stream *stream,
if (err)
goto err;
- intel_engine_pm_get(ce->engine);
+ intel_engine_pm_get(engine);
rq = i915_request_create(ce);
- intel_engine_pm_put(ce->engine);
+ intel_engine_pm_put(engine);
if (IS_ERR(rq)) {
err = PTR_ERR(rq);
goto err_vma_unpin;
@@ -1984,9 +1985,9 @@ emit_oa_config(struct i915_perf_stream *stream,
if (err)
goto err_add_request;
- err = rq->engine->emit_bb_start(rq,
- vma->node.start, 0,
- I915_DISPATCH_SECURE);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start, 0,
+ I915_DISPATCH_SECURE);
if (err)
goto err_add_request;
diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c b/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
index 401ee975bbe2..6b986ae888d2 100644
--- a/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
+++ b/drivers/gpu/drm/i915/selftests/i915_gem_gtt.c
@@ -1765,7 +1765,7 @@ submit_batch(struct intel_context *ce, u64 addr)
if (engine->emit_init_breadcrumb) /* detect a hang */
err = engine->emit_init_breadcrumb(engine, rq);
if (err == 0)
- err = engine->emit_bb_start(rq, addr, 0, 0);
+ err = engine->emit_bb_start(engine, rq, addr, 0, 0);
if (err == 0)
i915_request_get(rq);
diff --git a/drivers/gpu/drm/i915/selftests/i915_perf.c b/drivers/gpu/drm/i915/selftests/i915_perf.c
index 56bb7db79c23..c5f45afb9614 100644
--- a/drivers/gpu/drm/i915/selftests/i915_perf.c
+++ b/drivers/gpu/drm/i915/selftests/i915_perf.c
@@ -235,9 +235,9 @@ static int live_noa_delay(void *arg)
goto out;
}
- err = rq->engine->emit_bb_start(rq,
- i915_ggtt_offset(stream->noa_wait), 0,
- I915_DISPATCH_SECURE);
+ err = stream->engine->emit_bb_start(stream->engine, rq,
+ i915_ggtt_offset(stream->noa_wait), 0,
+ I915_DISPATCH_SECURE);
if (err) {
i915_request_add(rq);
goto out;
@@ -342,7 +342,7 @@ static int live_noa_gpr(void *arg)
intel_ring_advance(rq, cs);
/* Execute the GPU delay */
- err = stream->engine->emit_bb_start(rq,
+ err = stream->engine->emit_bb_start(stream->engine, rq,
i915_ggtt_offset(stream->noa_wait), 0,
I915_DISPATCH_SECURE);
if (err) {
diff --git a/drivers/gpu/drm/i915/selftests/i915_request.c b/drivers/gpu/drm/i915/selftests/i915_request.c
index c5d7427bd429..5d747c2d937b 100644
--- a/drivers/gpu/drm/i915/selftests/i915_request.c
+++ b/drivers/gpu/drm/i915/selftests/i915_request.c
@@ -667,7 +667,7 @@ empty_request(struct intel_engine_cs *engine,
if (IS_ERR(request))
return request;
- err = engine->emit_bb_start(request,
+ err = engine->emit_bb_start(engine, request,
batch->node.start,
batch->node.size,
I915_DISPATCH_SECURE);
@@ -879,7 +879,8 @@ static int live_all_engines(void *arg)
err = i915_vma_move_to_active(batch, request[idx], 0);
GEM_BUG_ON(err);
- err = engine->emit_bb_start(request[idx],
+ err = engine->emit_bb_start(engine,
+ request[idx],
batch->node.start,
batch->node.size,
0);
@@ -1009,7 +1010,8 @@ static int live_sequential_engines(void *arg)
err = i915_vma_move_to_active(batch, request[idx], 0);
GEM_BUG_ON(err);
- err = engine->emit_bb_start(request[idx],
+ err = engine->emit_bb_start(engine,
+ request[idx],
batch->node.start,
batch->node.size,
0);
diff --git a/drivers/gpu/drm/i915/selftests/igt_spinner.c b/drivers/gpu/drm/i915/selftests/igt_spinner.c
index 7e148e3975cc..5d83079bedd2 100644
--- a/drivers/gpu/drm/i915/selftests/igt_spinner.c
+++ b/drivers/gpu/drm/i915/selftests/igt_spinner.c
@@ -179,7 +179,9 @@ igt_spinner_create_request(struct igt_spinner *spin,
flags = 0;
if (INTEL_GEN(engine->i915) <= 5)
flags |= I915_DISPATCH_SECURE;
- err = engine->emit_bb_start(rq, vma->node.start, PAGE_SIZE, flags);
+ err = engine->emit_bb_start(engine, rq,
+ vma->node.start, PAGE_SIZE,
+ flags);
cancel_rq:
if (err) {
--
2.20.1
More information about the Intel-gfx-trybot
mailing list