== Series Details == Series: series starting with [01/19] drm/i915: Return immediately if trylock fails for direct-reclaim URL : https://patchwork.freedesktop.org/series/53953/ State : warning == Summary == $ dim checkpatch origin/drm-tip 916a3be212e3 drm/i915: Return immediately if trylock fails for direct-reclaim 26917cf8d163 drm/i915/userptr: Avoid struct_mutex recursion for mmu_invalidate_range_start -:23: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #23: References: 93065ac753e4 ("mm, oom: distinguish blockable mode for mmu notifiers") -:23: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 93065ac753e4 ("mm, oom: distinguish blockable mode for mmu notifiers")' #23: References: 93065ac753e4 ("mm, oom: distinguish blockable mode for mmu notifiers") -:240: ERROR:LOCKING: recursive locking is bad, do not use this ever. #240: FILE: drivers/gpu/drm/i915/i915_gem_userptr.c:108: + switch (mutex_trylock_recursive(m)) { total: 2 errors, 1 warnings, 0 checks, 444 lines checked c19a3aa46c49 drm/i915/userptr: Probe vma range before gup bde35602ca66 drm/i915/selftests: Check we can recover a wedged device 10e12b27ed33 drm/i915/selftests: Verify we can perform resets from atomic context ce3fa3096959 drm/i915/icl: Record the valid VDBoxes with SFC capability b5cc02d7c587 drm/i915/icl: Mind the SFC units when resetting VD or VEBox engines 6db7a50be19a drm/i915: Always try to reset the GPU on takeover 654c916de1ce drm/i915: Report the number of closed vma held by each context in debugfs -:61: WARNING:LONG_LINE: line over 100 characters #61: FILE: drivers/gpu/drm/i915/i915_debugfs.c:348: + seq_printf(m, "%s: %lu objects, %llu bytes (%llu active, %llu inactive, %llu global, %llu shared, %llu unbound, %llu closed)\n", \ total: 0 errors, 1 warnings, 0 checks, 201 lines checked 0e7e9b9d070d drm/i915: Remove debugfs/i915_ppgtt_info 12f7cb4ec2ad drm/i915: Track all held rpm wakerefs -:105: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment #105: FILE: drivers/gpu/drm/i915/i915_drv.h:1160: + spinlock_t debug_lock; total: 0 errors, 0 warnings, 1 checks, 571 lines checked c4143336103e drm/i915: Markup paired operations on wakerefs -:783: WARNING:NEW_TYPEDEFS: do not add new typedefs #783: FILE: drivers/gpu/drm/i915/i915_drv.h:134: +typedef depot_stack_handle_t intel_wakeref_t; total: 0 errors, 1 warnings, 0 checks, 2533 lines checked b72c3febcaf2 drm/i915: Syntatic sugar for using intel_runtime_pm -:509: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects? #509: FILE: drivers/gpu/drm/i915/intel_drv.h:2181: +#define with_intel_runtime_pm(i915, wf) \ + for (wf = intel_runtime_pm_get(i915); wf; \ + intel_runtime_pm_put(i915, wf), wf = 0) -:509: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects? #509: FILE: drivers/gpu/drm/i915/intel_drv.h:2181: +#define with_intel_runtime_pm(i915, wf) \ + for (wf = intel_runtime_pm_get(i915); wf; \ + intel_runtime_pm_put(i915, wf), wf = 0) -:513: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'i915' - possible side-effects? #513: FILE: drivers/gpu/drm/i915/intel_drv.h:2185: +#define with_intel_runtime_pm_if_in_use(i915, wf) \ + for (wf = intel_runtime_pm_get_if_in_use(i915); wf; \ + intel_runtime_pm_put(i915, wf), wf = 0) -:513: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects? #513: FILE: drivers/gpu/drm/i915/intel_drv.h:2185: +#define with_intel_runtime_pm_if_in_use(i915, wf) \ + for (wf = intel_runtime_pm_get_if_in_use(i915); wf; \ + intel_runtime_pm_put(i915, wf), wf = 0) total: 0 errors, 0 warnings, 4 checks, 732 lines checked 507dba016bbc drm/i915: Markup paired operations on display power domains e1be67c9baf0 drm/i915: Track the wakeref used to initialise display power domains -:213: WARNING:LINE_SPACING: Missing a blank line after declarations #213: FILE: drivers/gpu/drm/i915/intel_runtime_pm.c:4107: + struct i915_power_domains *power_domains = &i915->power_domains; + intel_wakeref_t wakeref __maybe_unused = total: 0 errors, 1 warnings, 0 checks, 324 lines checked 5b0a7f1a093b drm/i915: Combined gt.awake/gt.power wakerefs 7d9692b87640 drm/i915/dp: Markup pps lock power well -:57: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dp' - possible side-effects? #57: FILE: drivers/gpu/drm/i915/intel_dp.c:633: +#define with_pps_lock(dp, wf) \ + for (wf = pps_lock(dp); wf; wf = pps_unlock(dp, wf)) -:57: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wf' - possible side-effects? #57: FILE: drivers/gpu/drm/i915/intel_dp.c:633: +#define with_pps_lock(dp, wf) \ + for (wf = pps_lock(dp); wf; wf = pps_unlock(dp, wf)) total: 0 errors, 0 warnings, 2 checks, 430 lines checked 6649c0427ac7 drm/i915: Complain if hsw_get_pipe_config acquires the same power well twice 7d9e140908f7 drm/i915: Mark up Ironlake ips with rpm wakerefs _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
