[Intel-gfx] [PATCH v2 1/5] drm/i915: Only disable execlist preemption for the duration of the request

Chris Wilson chris at chris-wilson.co.uk
Mon Jan 23 12:24:20 UTC 2017


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


More information about the Intel-gfx mailing list