✗ CI.checkpatch: warning for cpufreq: intel_pstate: Rearrange locking in hybrid_init_cpu_capacity_scaling()

Patchwork patchwork at emeril.freedesktop.org
Tue Nov 12 10:25:38 UTC 2024


== Series Details ==

Series: cpufreq: intel_pstate: Rearrange locking in hybrid_init_cpu_capacity_scaling()
URL   : https://patchwork.freedesktop.org/series/141195/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 156014f3f3122d5669232833ab2943f1f7e7d3a3
Author: Rafael J. Wysocki <rafael.j.wysocki at intel.com>
Date:   Tue Nov 12 12:19:37 2024 +0530

    cpufreq: intel_pstate: Rearrange locking in hybrid_init_cpu_capacity_scaling()
    
    Notice that hybrid_init_cpu_capacity_scaling() only needs to hold
    hybrid_capacity_lock around __hybrid_init_cpu_capacity_scaling()
    calls, so introduce a "locked" wrapper around the latter and call
    it from the former.  This allows to drop a local variable and a
    label that are not needed any more.
    
    Also, rename __hybrid_init_cpu_capacity_scaling() to
    __hybrid_refresh_cpu_capacity_scaling() for consistency.
    
    No intentional functional impact.
    
    This is Core-for-CI patch for regression seen [1]
    
    [1] https://lore.kernel.org/linux-pm/SJ1PR11MB6129EDBF22F8A90FC3A3EDC8B9582@SJ1PR11MB6129.namprd11.prod.outlook.com/
    
    Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki at intel.com>
    References: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12763
    Link: https://lore.kernel.org/linux-pm/12554508.O9o76ZdvQC@rjwysocki.net/
    Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah at intel.com>
+ /mt/dim checkpatch bb3e6edc085808aa7d7ea238c6bff1130e893224 drm-intel
156014f3f312 cpufreq: intel_pstate: Rearrange locking in hybrid_init_cpu_capacity_scaling()
-:20: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#20: 
[1] https://lore.kernel.org/linux-pm/SJ1PR11MB6129EDBF22F8A90FC3A3EDC8B9582@SJ1PR11MB6129.namprd11.prod.outlook.com/

total: 0 errors, 1 warnings, 0 checks, 68 lines checked




More information about the Intel-xe mailing list