[Intel-gfx] [PATCH] drm/i915: Revert workaround for disabling L3 cache aging on IVB

[Ville Syrjälä]

On Fri, Feb 14, 2014 at 10:34:43PM +0000, Chris Wilson wrote:
> In commit e4e0c058a19c41150d12ad2d3023b3cf09c5de67
> Author: Eugeni Dodonov <eugeni.dodonov at intel.com>
> Date:   Wed Feb 8 12:53:50 2012 -0800
> 
>     drm/i915: gen7: Implement an L3 caching workaround.
> 
> the L3 cache aging was disabled. This was part of a shotgun response
> to a number of GPU hang bugs, but there appears to be no documentation
> to suggest that disabling the L3 cache age was ever required (to prevent
> the GPU hangs).

Bspec still lists the 0xF value in the w/a list, but the actual register
description doesn't say that this bit needs to be set. All the other
bits match either the default values, or specific notes about which bits
need to be set.

So I get the feeling this should be fine:
Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

> 
> Restoring the L3 cache age is a minor performance win of around 2%
> on IVB:GT2. (Note that this value seems to be consistent across a number
> of tests and so appears to be above the usual noise.)
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Kenneth Graunke <kenneth at whitecape.org>
> ---
>  drivers/gpu/drm/i915/i915_reg.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
> index a2c68c9492ba..d40d71bb66b6 100644
> --- a/drivers/gpu/drm/i915/i915_reg.h
> +++ b/drivers/gpu/drm/i915/i915_reg.h
> @@ -4171,7 +4171,7 @@
>  #define  VLV_B0_WA_L3SQCREG1_VALUE		0x00D30000
>  
>  #define GEN7_L3CNTLREG1				0xB01C
> -#define  GEN7_WA_FOR_GEN7_L3_CONTROL			0x3C4FFF8C
> +#define  GEN7_WA_FOR_GEN7_L3_CONTROL			0x3C47FF8C
>  #define  GEN7_L3AGDIS				(1<<19)
>  
>  #define GEN7_L3_CHICKEN_MODE_REGISTER		0xB030
> -- 
> 1.9.0.rc3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrjälä
Intel OTC