[Intel-gfx] [PATCH] drm/i915/gt: Use gt locals for accessing rc6

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Fri Nov 15 12:04:39 UTC 2019


On 15/11/2019 11:48, Chris Wilson wrote:
> RC6 is tracked underneath the intel_gt, so use our local pointers.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at linux.intel.com>
> Cc: Andi Shyti <andi.shyti at intel.com>
> ---
>   drivers/gpu/drm/i915/gt/intel_gt_pm.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gt/intel_gt_pm.c b/drivers/gpu/drm/i915/gt/intel_gt_pm.c
> index 6187cdd06646..e61f752a3cd5 100644
> --- a/drivers/gpu/drm/i915/gt/intel_gt_pm.c
> +++ b/drivers/gpu/drm/i915/gt/intel_gt_pm.c
> @@ -62,7 +62,7 @@ static int __gt_unpark(struct intel_wakeref *wf)
>   	GEM_BUG_ON(!gt->awake);
>   
>   	if (NEEDS_RC6_CTX_CORRUPTION_WA(i915))
> -		intel_uncore_forcewake_get(&i915->uncore, FORCEWAKE_ALL);
> +		intel_uncore_forcewake_get(gt->uncore, FORCEWAKE_ALL);
>   
>   	intel_rps_unpark(&gt->rps);
>   	i915_pmu_gt_unparked(i915);
> @@ -90,8 +90,8 @@ static int __gt_park(struct intel_wakeref *wf)
>   	intel_synchronize_irq(i915);
>   
>   	if (NEEDS_RC6_CTX_CORRUPTION_WA(i915)) {
> -		intel_rc6_ctx_wa_check(&i915->gt.rc6);
> -		intel_uncore_forcewake_put(&i915->uncore, FORCEWAKE_ALL);
> +		intel_rc6_ctx_wa_check(&gt->rc6);
> +		intel_uncore_forcewake_put(gt->uncore, FORCEWAKE_ALL);
>   	}
>   
>   	GEM_BUG_ON(!wakeref);
> 

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>

Regards,

Tvrtko


More information about the Intel-gfx mailing list