== Series Details == Series: drm/i915: use ref_tracker library for tracking wakerefs (rev3) URL : https://patchwork.freedesktop.org/series/100327/ State : warning == Summary == $ dim checkpatch origin/drm-tip eec39ef01f2c ref_tracker: implement use-after-free detection e7636c41be32 ref_tracker: add a count of untracked references 414f58d98e73 ref_tracker: remove filter_irq_stacks() call 5a389d9482be lib/ref_tracker: add unlocked leak print helper -:6: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #6: To have reliable detection of leaks, caller must be able to check under the same -:23: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #23: FILE: include/linux/ref_tracker.h:40: +void __ref_tracker_dir_print(struct ref_tracker_dir *dir, + unsigned int display_limit); -:49: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #49: FILE: lib/ref_tracker.c:18: +void __ref_tracker_dir_print(struct ref_tracker_dir *dir, + unsigned int display_limit) total: 0 errors, 1 warnings, 2 checks, 105 lines checked 36bd9e9d6260 lib/ref_tracker: __ref_tracker_dir_print improve printing -:37: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #37: FILE: include/linux/ref_tracker.h:31: +static inline void __ref_tracker_dir_init(struct ref_tracker_dir *dir, + unsigned int quarantine_count, -:46: WARNING:STRLCPY: Prefer strscpy over strlcpy - see: https://lore.kernel.org/r/CAHk-=wgfRnXz0W3D37d01q3JFkr_i_uTL=V6A6G1oUZcprmknw@xxxxxxxxxxxxxx/ #46: FILE: include/linux/ref_tracker.h:41: + strlcpy(dir->name, name, sizeof(dir->name)); -:140: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #140: FILE: lib/ref_tracker.c:66: +void __ref_tracker_dir_print(struct ref_tracker_dir *dir, + unsigned int display_limit) total: 0 errors, 1 warnings, 2 checks, 151 lines checked bd88fd5cf435 lib/ref_tracker: add printing to memory buffer -:52: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fmt' - possible side-effects? #52: FILE: lib/ref_tracker.c:70: +#define pr_ostream(stream, fmt, args...) \ +({ \ + struct ostream *_s = (stream); \ +\ + if (!_s->buf) { \ + pr_err(fmt, ##args); \ + } else { \ + int ret, len = _s->size - _s->used; \ + ret = snprintf(_s->buf + _s->used, len, pr_fmt(fmt), ##args); \ + _s->used += min(ret, len); \ + } \ +}) -:105: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #105: FILE: lib/ref_tracker.c:125: +void __ref_tracker_dir_print(struct ref_tracker_dir *dir, + unsigned int display_limit) total: 0 errors, 0 warnings, 2 checks, 109 lines checked 359c7ae0d260 lib/ref_tracker: remove warnings in case of allocation failure e1a8bc8a2d3a drm/i915: Separate wakeref tracking -:7: WARNING:REPEATED_WORD: Possible repeated word: 'that' #7: utility so that that we can reuse it for other online debugging of -:453: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #453: new file mode 100644 -:710: WARNING:NEW_TYPEDEFS: do not add new typedefs #710: FILE: drivers/gpu/drm/i915/intel_wakeref_tracker.h:13: +typedef depot_stack_handle_t intel_wakeref_t; -:715: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment #715: FILE: drivers/gpu/drm/i915/intel_wakeref_tracker.h:18: + spinlock_t lock; -:731: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #731: FILE: drivers/gpu/drm/i915/intel_wakeref_tracker.h:34: +void intel_wakeref_tracker_remove(struct intel_wakeref_tracker *w, + intel_wakeref_t handle); -:768: WARNING:LONG_LINE: line length of 112 exceeds 100 columns #768: FILE: drivers/gpu/drm/i915/intel_wakeref_tracker.h:71: +static inline void __intel_wakeref_tracker_show(const struct intel_wakeref_tracker *w, struct drm_printer *p) {} -:769: WARNING:LONG_LINE: line length of 104 exceeds 100 columns #769: FILE: drivers/gpu/drm/i915/intel_wakeref_tracker.h:72: +static inline void intel_wakeref_tracker_show(struct intel_wakeref_tracker *w, struct drm_printer *p) {} -:773: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: Chris Wilson <chris.p.wilson@xxxxxxxxx>' != 'Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>' total: 0 errors, 6 warnings, 2 checks, 713 lines checked 29a2d4b63b37 drm/i915: Track leaked gt->wakerefs -:450: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'gt' - possible side-effects? #450: FILE: drivers/gpu/drm/i915/gt/intel_gt_pm.h:75: +#define with_intel_gt_pm(gt, wf) \ + for (wf = intel_gt_pm_get(gt); wf; intel_gt_pm_put(gt, wf), wf = 0) -:450: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects? #450: FILE: drivers/gpu/drm/i915/gt/intel_gt_pm.h:75: +#define with_intel_gt_pm(gt, wf) \ + for (wf = intel_gt_pm_get(gt); wf; intel_gt_pm_put(gt, wf), wf = 0) -:798: ERROR:ASSIGN_IN_IF: do not use assignment in if condition #798: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c:1280: + if (!in_reset && (wakeref = intel_gt_pm_get_if_awake(gt))) { total: 1 errors, 0 warnings, 2 checks, 775 lines checked fdce0e9f8058 drm/i915: Correct type of wakeref variable 79bf0307dd7d drm/i915: replace Intel internal tracker with kernel core ref_tracker -:254: WARNING:NEW_TYPEDEFS: do not add new typedefs #254: FILE: drivers/gpu/drm/i915/intel_wakeref.h:24: +typedef unsigned long intel_wakeref_t; -:357: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #357: deleted file mode 100644 total: 0 errors, 2 warnings, 0 checks, 286 lines checked
