On Tue, Mar 28, 2017 at 08:00:26PM +0200, Michał Winiarski wrote: > Since request can be unsubmitted, we need to avoid overriding its priority > during submission. Otherwise we won't be able to resubmit it with > correct priority. > > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Signed-off-by: Michał Winiarski <michal.winiarski@xxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_guc_submission.c | 1 - > drivers/gpu/drm/i915/intel_lrc.c | 1 - > 2 files changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_guc_submission.c b/drivers/gpu/drm/i915/i915_guc_submission.c > index 6193ad7..082f8ae 100644 > --- a/drivers/gpu/drm/i915/i915_guc_submission.c > +++ b/drivers/gpu/drm/i915/i915_guc_submission.c > @@ -692,7 +692,6 @@ static bool i915_guc_dequeue(struct intel_engine_cs *engine) > rb = rb_next(rb); > rb_erase(&rq->priotree.node, &engine->execlist_queue); > RB_CLEAR_NODE(&rq->priotree.node); > - rq->priotree.priority = INT_MAX; This still breaks execlists_schedule(). -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
