✗ Fi.CI.CHECKPATCH: warning for Fix i915 pmu on bind/unbind (rev2)

Patchwork patchwork at emeril.freedesktop.org
Fri Oct 11 23:55:02 UTC 2024


== Series Details ==

Series: Fix i915 pmu on bind/unbind (rev2)
URL   : https://patchwork.freedesktop.org/series/136362/
State : warning

== Summary ==

Error: dim checkpatch failed
73e46040952e squashed series - 20241008183501.1354695-1-lucas.demarchi at intel.com
-:7: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#7: 
Patches 2 and 3 of https://lore.kernel.org/all/20241008183501.1354695-1-lucas.demarchi@intel.com/

-:27: WARNING:SPACING: Unnecessary space before function pointer arguments
#27: FILE: include/linux/perf_event.h:571:
+	struct pmu *(*get)		(struct pmu *pmu);

-:30: WARNING:SPACING: Unnecessary space before function pointer arguments
#30: FILE: include/linux/perf_event.h:574:
+	 void (*put)			(struct pmu *pmu);

-:38: CHECK:AVOID_EXTERNS: extern prototypes should be avoided in .h files
#38: FILE: include/linux/perf_event.h:1118:
+extern void perf_pmu_free(struct pmu *pmu);

-:95: WARNING:LONG_LINE: line length of 111 exceeds 100 columns
#95: FILE: kernel/events/core.c:11769:
+	if (WARN_ONCE((!!pmu->get) ^ (!!pmu->put), "Can not register a pmu with only get or put defined.\n")) {

total: 0 errors, 4 warnings, 1 checks, 113 lines checked
1a85dc884b6d drm/i915/pmu: Let resource survive unbind
e7bb5806e940 drm/i915/pmu: Fix crash due to use-after-free
-:110: WARNING:MISSING_FIXES_TAG: The commit message has 'stable@', perhaps it also needs a 'Fixes:' tag?

total: 0 errors, 1 warnings, 0 checks, 66 lines checked
26c2507815e4 drm/i915/pmu: Rename cpuhp_slot to cpuhp_state
1b5adbd7c6e4 drm/i915/pmu: Stop setting event_init to NULL
c98937826402 drm/i915/pmu: Replace closed with registered
f21a1a02ada7 drm/i915/pmu: Remove pointless synchronize_rcu() call
ec2a46c1535d drm/i915/pmu: Release open events when unregistering




More information about the Intel-gfx mailing list