Re: [PATCH 6.14.y and others] cpufreq: Reference count policy in cpufreq_update_limits()

[Sasha Levin <sashal@kernel.org>]

[ Sasha's backport helper bot ]

Hi,

Summary of potential issues:
⚠️ Found matching upstream commit but patch is missing proper reference to it

Found matching upstream commit: 9e4e249018d208678888bdf22f6b652728106528

WARNING: Author mismatch between patch and found commit:
Backport author: <marmarek@invisiblethingslab.com>
Commit author: Rafael J. Wysocki<rafael.j.wysocki@intel.com>

Note: The patch differs from the upstream commit:
---
1:  9e4e249018d20 ! 1:  cd95c6b965176 cpufreq: Reference count policy in cpufreq_update_limits()
    @@ Commit message
         Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
         Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
         Link: https://patch.msgid.link/1928789.tdWV9SEqCh@rjwysocki.net
    +    (cherry picked from commit 9e4e249018d208678888bdf22f6b652728106528)
    +    [do not use __free(cpufreq_cpu_put) in a backport]
    +    Signed-off-by: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>
     
      ## drivers/cpufreq/cpufreq.c ##
     @@ drivers/cpufreq/cpufreq.c: EXPORT_SYMBOL(cpufreq_update_policy);
       */
      void cpufreq_update_limits(unsigned int cpu)
      {
    -+	struct cpufreq_policy *policy __free(put_cpufreq_policy);
    ++	struct cpufreq_policy *policy;
     +
     +	policy = cpufreq_cpu_get(cpu);
     +	if (!policy)
    @@ drivers/cpufreq/cpufreq.c: EXPORT_SYMBOL(cpufreq_update_policy);
      	if (cpufreq_driver->update_limits)
      		cpufreq_driver->update_limits(cpu);
      	else
    + 		cpufreq_update_policy(cpu);
    ++
    ++	cpufreq_cpu_put(policy);
    + }
    + EXPORT_SYMBOL_GPL(cpufreq_update_limits);
    + 
---

Results of testing on various branches:

| Branch                    | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| stable/linux-6.14.y       |  Success    |  Success   |