[Intel-gfx] [PATCH 09/10] drm/i915: Enable semaphores for legacy submission on gen8

[Daniel Vetter]

On Thu, Apr 28, 2016 at 05:24:31PM +0100, Chris Wilson wrote:
> We have sufficient evidence from igt to support that semaphores are in
> a working state. Enabling semaphores now for legacy provides a better
> comparison of execlists against legacy ring submission.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

Somehow my reply didn't make it. Since still disabled by default, and mesa
CI folks confirmed that they stopped disabling execlist on bdw this seems
all safe. And useful for delta-benchmarking execlist.

Acked-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> ---
>  drivers/gpu/drm/i915/i915_drv.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c
> index d37c0a671eed..9fd221c97275 100644
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -542,10 +542,6 @@ bool i915_semaphore_is_enabled(struct drm_device *dev)
>  	if (i915.enable_execlists)
>  		return false;
>  
> -	/* Until we get further testing... */
> -	if (IS_GEN8(dev))
> -		return false;
> -
>  #ifdef CONFIG_INTEL_IOMMU
>  	/* Enable semaphores on SNB when IO remapping is off */
>  	if (INTEL_INFO(dev)->gen == 6 && intel_iommu_gfx_mapped)
> -- 
> 2.8.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch