diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2019-05-07 13:25:44 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2019-05-07 17:40:20 +0100 |
| commit | 25d851adbf4c4faef5b1acbe81a92720d1bd173e (patch) | |
| tree | f7a8bd8b65a299d8b9f8e6f533f1cb4bac7765ae /drivers/gpu/drm/i915/intel_guc_submission.c | |
| parent | 3970564940ba0322bcefce7fd8fd35c2b85846bf (diff) | |
| download | linux-stable-25d851adbf4c4faef5b1acbe81a92720d1bd173e.tar.gz linux-stable-25d851adbf4c4faef5b1acbe81a92720d1bd173e.tar.bz2 linux-stable-25d851adbf4c4faef5b1acbe81a92720d1bd173e.zip | |
drm/i915: Only reschedule the submission tasklet if preemption is possible
If we couple the scheduler more tightly with the execlists policy, we
can apply the preemption policy to the question of whether we need to
kick the tasklet at all for this priority bump.
v2: Rephrase it as a core i915 policy and not an execlists foible.
v3: Pull the kick together.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190507122544.12698-1-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/intel_guc_submission.c')
| -rw-r--r-- | drivers/gpu/drm/i915/intel_guc_submission.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/intel_guc_submission.c b/drivers/gpu/drm/i915/intel_guc_submission.c index 57ed1dd4ae41..380d83a2bfb6 100644 --- a/drivers/gpu/drm/i915/intel_guc_submission.c +++ b/drivers/gpu/drm/i915/intel_guc_submission.c @@ -747,7 +747,8 @@ static bool __guc_dequeue(struct intel_engine_cs *engine) &engine->i915->guc.preempt_work[engine->id]; int prio = execlists->queue_priority_hint; - if (__execlists_need_preempt(prio, port_prio(port))) { + if (i915_scheduler_need_preempt(prio, + port_prio(port))) { execlists_set_active(execlists, EXECLISTS_ACTIVE_PREEMPT); queue_work(engine->i915->guc.preempt_wq, |