== Series Details == Series: drm/i915: use ref_tracker library for tracking wakerefs (rev8) URL : https://patchwork.freedesktop.org/series/100327/ State : warning == Summary == Error: dim checkpatch failed 4cc3dfa0b243 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 total: 0 errors, 1 warnings, 0 checks, 105 lines checked de51d28343e8 lib/ref_tracker: improve printing stats -:40: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #40: FILE: include/linux/ref_tracker.h:31: +static inline void __ref_tracker_dir_init(struct ref_tracker_dir *dir, + unsigned int quarantine_count, -:49: WARNING:STRLCPY: Prefer strscpy over strlcpy - see: https://lore.kernel.org/r/CAHk-=wgfRnXz0W3D37d01q3JFkr_i_uTL=V6A6G1oUZcprmknw@xxxxxxxxxxxxxx/ #49: FILE: include/linux/ref_tracker.h:41: + strlcpy(dir->name, name, sizeof(dir->name)); total: 0 errors, 1 warnings, 1 checks, 151 lines checked 87f8a2f62e75 lib/ref_tracker: add printing to memory buffer -:54: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fmt' - possible side-effects? #54: 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); \ + } \ +}) total: 0 errors, 0 warnings, 1 checks, 109 lines checked be584c3f69e9 lib/ref_tracker: remove warnings in case of allocation failure 04d9ce2b2595 drm/i915: Correct type of wakeref variable e1f156958d75 drm/i915: Replace custom intel runtime_pm tracker with ref_tracker library -:404: WARNING:NEW_TYPEDEFS: do not add new typedefs #404: FILE: drivers/gpu/drm/i915/intel_wakeref.h:24: +typedef unsigned long intel_wakeref_t; total: 0 errors, 1 warnings, 0 checks, 426 lines checked c1995f5d11d1 drm/i915: track gt pm wakerefs -:514: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'gt' - possible side-effects? #514: 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) -:514: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects? #514: 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) -:845: ERROR:ASSIGN_IN_IF: do not use assignment in if condition #845: FILE: drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c:1328: + if (!in_reset && (wakeref = intel_gt_pm_get_if_awake(gt))) { total: 1 errors, 0 warnings, 2 checks, 826 lines checked
