On 12/12/2013 03:28 PM, Ben Widawsky wrote:
> WaVSRefCountFullforceMissDisable and
> WaDSRefCountFullforceMissDisable
> 
> VS is a carry-over from HSW, and DS is likely not used by anyone yet.

You're correct that no one uses DS; we don't support GL 4.0's
tessellation shaders yet.

I was about to comment that the Haswell workaround is missing, but it's
just implemented in a different function: gen7_setup_fixed_func_scheduler.

Both patches are:
Reviewed-by: Kenneth Graunke <[email protected]>

Too late to get tagged, but still under 24 hours after posting...

> 
> Cc: Kenneth Graunke <[email protected]>
> Signed-off-by: Ben Widawsky <[email protected]>
> ---
>  drivers/gpu/drm/i915/i915_reg.h |  1 +
>  drivers/gpu/drm/i915/intel_pm.c | 11 ++++++++---
>  2 files changed, 9 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
> index 3259e83..f1eece4 100644
> --- a/drivers/gpu/drm/i915/i915_reg.h
> +++ b/drivers/gpu/drm/i915/i915_reg.h
> @@ -1003,6 +1003,7 @@
>  
>  #define GEN7_FF_THREAD_MODE          0x20a0
>  #define   GEN7_FF_SCHED_MASK         0x0077070
> +#define   GEN8_FF_DS_REF_CNT_FFME    (1 << 19)
>  #define   GEN7_FF_TS_SCHED_HS1               (0x5<<16)
>  #define   GEN7_FF_TS_SCHED_HS0               (0x3<<16)
>  #define   GEN7_FF_TS_SCHED_LOAD_BALANCE      (0x1<<16)
> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> index 7e2a0e9..424a9d8 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -5266,14 +5266,14 @@ static void gen8_init_clock_gating(struct drm_device 
> *dev)
>       I915_WRITE(GEN7_HALF_SLICE_CHICKEN1,
>                  _MASKED_BIT_ENABLE(GEN7_SINGLE_SUBSCAN_DISPATCH_ENABLE));
>  
> -     /* WaSwitchSolVfFArbitrationPriority */
> +     /* WaSwitchSolVfFArbitrationPriority:bdw */
>       I915_WRITE(GAM_ECOCHK, I915_READ(GAM_ECOCHK) | HSW_ECOCHK_ARB_PRIO_SOL);
>  
> -     /* WaPsrDPAMaskVBlankInSRD */
> +     /* WaPsrDPAMaskVBlankInSRD:bdw */
>       I915_WRITE(CHICKEN_PAR1_1,
>                  I915_READ(CHICKEN_PAR1_1) | DPA_MASK_VBLANK_SRD);
>  
> -     /* WaPsrDPRSUnmaskVBlankInSRD */
> +     /* WaPsrDPRSUnmaskVBlankInSRD:bdw */
>       for_each_pipe(i) {
>               I915_WRITE(CHICKEN_PIPESL_1(i),
>                          I915_READ(CHICKEN_PIPESL_1(i) |
> @@ -5286,6 +5286,11 @@ static void gen8_init_clock_gating(struct drm_device 
> *dev)
>        */
>       I915_WRITE(HDC_FORCE_NON_COHERENT,
>                  I915_READ(HDC_CHICKEN0) | HDC_FORCE_NON_COHERENT);
> +
> +     /* WaVSRefCountFullforceMissDisable:bdw */
> +     /* WaDSRefCountFullforceMissDisable:bdw */
> +     I915_WRITE(GEN7_FF_THREAD_MODE,
> +                I915_READ(GEN7_FF_THREAD_MODE) & ~(GEN8_FF_DS_REF_CNT_FFME | 
> GEN7_FF_VS_REF_CNT_FFME));
>  }
>  
>  static void haswell_init_clock_gating(struct drm_device *dev)
> 

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to