Re: [PATCH] drm/i915/slpc: Fix inconsistent locked return

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 30 Aug 2022 08:02:29 -0700, Rodrigo Vivi wrote:
>
> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> index 9d49ccef03bb..f8a2bbcdf14f 100644
> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_slpc.c
> @@ -477,7 +477,7 @@ int intel_guc_slpc_set_min_freq(struct intel_guc_slpc *slpc, u32 val)
>		if (unlikely(ret)) {
>			i915_probe_error(i915, "Failed to toggle efficient freq (%pe)\n",
>					 ERR_PTR(ret));
> -			return ret;
> +			goto unlock;

I think leaking runtime_pm wakeref now...

>		}
>
>		ret = slpc_set_param(slpc,
> @@ -492,6 +492,7 @@ int intel_guc_slpc_set_min_freq(struct intel_guc_slpc *slpc, u32 val)
>	if (!ret)
>		slpc->min_freq_softlimit = val;
>
> +unlock:
>	mutex_unlock(&slpc->lock);
>
>	return ret;
> --
> 2.37.2
>



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux