On Mon, Jan 23, 2017 at 12:14:25PM +0000, Chris Wilson wrote: > @@ -558,7 +558,8 @@ static bool execlists_elsp_ready(struct intel_engine_cs *engine) > int port; > > port = 1; /* wait for a free slot */ > - if (engine->disable_lite_restore_wa || engine->preempt_wa) > + if (engine->disable_lite_restore_wa || > + engine->execlist_port[0].count > 1) > port = 0; /* wait for GPU to be idle before continuing */ The disable_lite_restore_wa is a preproduction wa, right? Do we still want to keep it? If not this could be rewritten as return port[0].count + port[1].count < 2; -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
