== Series Details == Series: series starting with [01/22] drm/i915: Restore -Wunused-but-set-variable URL : https://patchwork.freedesktop.org/series/62184/ State : warning == Summary == $ dim checkpatch origin/drm-tip f9d84faad457 drm/i915: Restore -Wunused-but-set-variable 9a9b7608d0bb drm/i915/gtt: Serialise both updates to PDE and our shadow 5084e77a82e6 drm/i915: Skip shrinking already freed pages 90ca29dca482 drm/i915: Stop passing I915_WAIT_LOCKED to i915_request_wait() -:11: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #11: References: eb8d0f5af4ec ("drm/i915: Remove GPU reset dependence on struct_mutex") -:11: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit eb8d0f5af4ec ("drm/i915: Remove GPU reset dependence on struct_mutex")' #11: References: eb8d0f5af4ec ("drm/i915: Remove GPU reset dependence on struct_mutex") total: 1 errors, 1 warnings, 0 checks, 328 lines checked f8ec22798621 drm/i915: Flush the execution-callbacks on retiring 68499f942c04 drm/i915/execlists: Preempt-to-busy -:1494: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p_ptr' - possible side-effects? #1494: FILE: drivers/gpu/drm/i915/i915_utils.h:134: +#define ptr_count_dec(p_ptr) do { \ + typeof(p_ptr) __p = (p_ptr); \ + unsigned long __v = (unsigned long)(*__p); \ + *__p = (typeof(*p_ptr))(--__v); \ +} while (0) -:1500: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'p_ptr' - possible side-effects? #1500: FILE: drivers/gpu/drm/i915/i915_utils.h:140: +#define ptr_count_inc(p_ptr) do { \ + typeof(p_ptr) __p = (p_ptr); \ + unsigned long __v = (unsigned long)(*__p); \ + *__p = (typeof(*p_ptr))(++__v); \ +} while (0) -:1783: WARNING:LINE_SPACING: Missing a blank line after declarations #1783: FILE: drivers/gpu/drm/i915/intel_guc_submission.c:820: + int rem = ARRAY_SIZE(execlists->inflight) - idx; + memmove(execlists->inflight, port, rem * sizeof(*port)); total: 0 errors, 1 warnings, 2 checks, 1682 lines checked 0e05621dacc3 drm/i915/execlists: Minimalistic timeslicing -:345: WARNING:LONG_LINE: line over 100 characters #345: FILE: drivers/gpu/drm/i915/gt/selftest_lrc.c:211: + 2 * RUNTIME_INFO(outer->i915)->num_engines * (count + 2) * (count + 3)) < 0) { total: 0 errors, 1 warnings, 0 checks, 426 lines checked dcf969ca734d drm/i915/execlists: Force preemption 411a6ede312e drm/i915: Make the semaphore saturation mask global 9492cff546bc dma-fence: Propagate errors to dma-fence-array container 22dc1810a223 dma-fence: Report the composite sync_file status a9fdb112a5db dma-fence: Refactor signaling for manual invocation -:33: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #33: new file mode 100644 -:38: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #38: FILE: drivers/dma-buf/dma-fence-trace.c:1: +/* -:293: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #293: FILE: include/linux/dma-fence-types.h:1: +/* -:368: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment #368: FILE: include/linux/dma-fence-types.h:76: + spinlock_t *lock; total: 0 errors, 3 warnings, 1 checks, 728 lines checked f76e40a91686 dma-fence: Always execute signal callbacks fedfd3888d5c drm/i915: Throw away the active object retirement complexity 5b4a2eca4f49 drm/i915: Provide an i915_active.acquire callback -:684: CHECK:UNCOMMENTED_DEFINITION: struct mutex definition without comment #684: FILE: drivers/gpu/drm/i915/i915_active_types.h:36: + struct mutex mutex; total: 0 errors, 0 warnings, 1 checks, 730 lines checked 04f835537ecd drm/i915: Push the i915_active.retire into a worker c703a4145665 drm/i915/overlay: Switch to using i915_active tracking 0dd26f77aba5 drm/i915: Forgo last_fence active request tracking 70d8c211e754 drm/i915: Extract intel_frontbuffer active tracking 14f5d2753d66 drm/i915: Coordinate i915_active with its own mutex 437346375b8c drm/i915: Replace struct_mutex for batch pool serialisation -:304: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #304: new file mode 100644 -:309: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #309: FILE: drivers/gpu/drm/i915/gt/intel_engine_pool.c:1: +/* -:310: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead #310: FILE: drivers/gpu/drm/i915/gt/intel_engine_pool.c:2: + * SPDX-License-Identifier: MIT -:478: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #478: FILE: drivers/gpu/drm/i915/gt/intel_engine_pool.h:1: +/* -:479: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead #479: FILE: drivers/gpu/drm/i915/gt/intel_engine_pool.h:2: + * SPDX-License-Identifier: MIT -:517: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #517: FILE: drivers/gpu/drm/i915/gt/intel_engine_pool_types.h:1: +/* -:518: WARNING:SPDX_LICENSE_TAG: Misplaced SPDX-License-Identifier tag - use line 1 instead #518: FILE: drivers/gpu/drm/i915/gt/intel_engine_pool_types.h:2: + * SPDX-License-Identifier: MIT -:534: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment #534: FILE: drivers/gpu/drm/i915/gt/intel_engine_pool_types.h:18: + spinlock_t lock; total: 0 errors, 7 warnings, 1 checks, 574 lines checked d05834adffc1 drm/i915: Move idle barrier cleanup into engine-pm _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx