[Intel-gfx] [PATCH 2/2] drm/i915: Enable rpm wakeref tracking whether runtime pm is enabled or not

Imre Deak imre.deak at intel.com
Wed Jan 26 12:40:32 UTC 2022


On Wed, Jan 26, 2022 at 10:15:39AM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Don't see why we should skip the wakeref tracking when the
> platform doesn't support runtime pm. We still want all the
> code to be 100% leak free so let's track this unconditionally.
> 
> Cc: Vlastimil Babka <vbabka at suse.cz>
> Cc: Dmitry Vyukov <dvyukov at google.com>
> Cc: Marco Elver <elver at google.com> # stackdepot
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Imre Deak <imre.deak at intel.com>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

Reviewed-by: Imre Deak <imre.deak at intel.com>

> ---
>  drivers/gpu/drm/i915/intel_runtime_pm.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_runtime_pm.c b/drivers/gpu/drm/i915/intel_runtime_pm.c
> index 64c2708efc9e..3293ac71bcf8 100644
> --- a/drivers/gpu/drm/i915/intel_runtime_pm.c
> +++ b/drivers/gpu/drm/i915/intel_runtime_pm.c
> @@ -77,9 +77,6 @@ track_intel_runtime_pm_wakeref(struct intel_runtime_pm *rpm)
>  	depot_stack_handle_t stack, *stacks;
>  	unsigned long flags;
>  
> -	if (!rpm->available)
> -		return -1;
> -
>  	stack = __save_depot_stack();
>  	if (!stack)
>  		return -1;
> -- 
> 2.34.1
> 


More information about the Intel-gfx mailing list