[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [01/27] drm/i915: Squelch very verbose error logging

[Patchwork]

== Series Details ==

Series: series starting with [01/27] drm/i915: Squelch very verbose error logging
URL   : https://patchwork.freedesktop.org/series/46051/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
47eefccc61dd drm/i915: Squelch very verbose error logging
-:10: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#10: 
References: 10195b1e4411 ("drm/i915: Show vma allocator stack when in doubt")

-:10: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 10195b1e4411 ("drm/i915: Show vma allocator stack when in doubt")'
#10: 
References: 10195b1e4411 ("drm/i915: Show vma allocator stack when in doubt")

total: 1 errors, 1 warnings, 0 checks, 26 lines checked
2d1ad507b837 drm/i915/selftests: Destroy partial tiling vma after use
7c6cc6272c70 drm/i915/selftests: Skip using the GPU if wedged
b284299c1f62 drm/i915/selftests: Skip making an object busy if the GPU is wedged
e92b9d733a9b drm/i915/selftests: Skip all request selftests when wedged
6b77796e5bff drm/i915/selftests: Skip workaround tests when wedged
54d9e3ec289c drm/i915/selftests: Skip live eviction tests when wedged
6dbafa1b7434 drm/i915/selftests: Skip huge pages live tests if wedged
f3362da2025d drm/i915/selftests: Skip over live context testing when wedged
2214c0198f3a drm/i915/selftests: Skip live context execution test without logical contexts
b8328c22eaf4 drm/i915: Refactor export_fence() after i915_vma_move_to_active()
7a224ca884d8 drm/i915: Export i915_request_skip()
9cc06ac3d8db drm/i915: Start returning an error from i915_vma_move_to_active()
8906e2e47a45 drm/i915: Move i915_vma_move_to_active() to i915_vma.c
7f27eec803fb drm/i915: Track vma activity per fence.context, not per engine
551ba11b1c18 drm/i915: Track the last-active inside the i915_vma
-:83: WARNING:TYPO_SPELLING: 'recieve' may be misspelled - perhaps 'receive'?
#83: FILE: drivers/gpu/drm/i915/i915_vma.c:952:
+	 * we must be careful as we now only expect to recieve one retire

total: 0 errors, 1 warnings, 0 checks, 93 lines checked
fb0482872396 drm/i915: Replace nested subclassing with explicit subclasses
83f43a217240 drm/i915: Only reset hangcheck at the start of an activity cycle
-:14: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#14: 
References: e21b141376f9 ("drm/i915: Mark the hangcheck as idle when unparking the engines")

-:14: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit e21b141376f9 ("drm/i915: Mark the hangcheck as idle when unparking the engines")'
#14: 
References: e21b141376f9 ("drm/i915: Mark the hangcheck as idle when unparking the engines")

-:15: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 1fd00c0faeec ("drm/i915: Declare the driver wedged if hangcheck makes no progress")'
#15: 
References: 1fd00c0faeec ("drm/i915: Declare the driver wedged if hangcheck makes no progress")

total: 2 errors, 1 warnings, 0 checks, 15 lines checked
d7f74c833562 drm/i915: Stop tracking MRU activity on VMA
618768236969 drm/i915: Introduce i915_address_space.mutex
2568bc465304 drm/i915: Move fence register tracking to GGTT
cd029a515a36 drm/i915: Convert fences to use a GGTT lock rather than struct_mutex
2c6fb6e1eb22 drm/i915: Tidy i915_gem_suspend()
d2aeb1979359 drm/i915: Move fence-reg interface to i915_gem_fence_reg.h
71ce31363138 drm/i915: Dynamically allocate the array of drm_i915_gem_fence_reg
eab94b91efeb drm/i915: Pull all the reset functionality together into i915_reset.c
-:1070: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#1070: 
new file mode 100644

-:1075: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#1075: FILE: drivers/gpu/drm/i915/i915_reset.c:1:
+/*

-:1216: WARNING:TYPO_SPELLING: 'acknowledgement' may be misspelled - perhaps 'acknowledgment'?
#1216: FILE: drivers/gpu/drm/i915/i915_reset.c:142:
+	/* Assert reset for at least 20 usec, and wait for acknowledgement. */

-:1793: WARNING:MEMORY_BARRIER: memory barrier without comment
#1793: FILE: drivers/gpu/drm/i915/i915_reset.c:719:
+	smp_mb__after_atomic();

-:2161: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'W' - possible side-effects?
#2161: FILE: drivers/gpu/drm/i915/i915_reset.c:1087:
+#define i915_wedge_on_timeout(W, DEV, TIMEOUT)				\
+	for (__init_wedge((W), (DEV), (TIMEOUT), __func__);		\
+	     (W)->i915;							\
+	     __fini_wedge((W)))

-:2172: WARNING:STATIC_CONST_CHAR_ARRAY: char * array declaration might be better as static const
#2172: FILE: drivers/gpu/drm/i915/i915_reset.c:1098:
+	char *error_event[] = { I915_ERROR_UEVENT "=1", NULL };

-:2173: WARNING:STATIC_CONST_CHAR_ARRAY: char * array declaration might be better as static const
#2173: FILE: drivers/gpu/drm/i915/i915_reset.c:1099:
+	char *reset_event[] = { I915_RESET_UEVENT "=1", NULL };

-:2174: WARNING:STATIC_CONST_CHAR_ARRAY: char * array declaration might be better as static const
#2174: FILE: drivers/gpu/drm/i915/i915_reset.c:1100:
+	char *reset_done_event[] = { I915_ERROR_UEVENT "=0", NULL };

-:2352: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1
#2352: FILE: drivers/gpu/drm/i915/i915_reset.h:1:
+/*

total: 0 errors, 8 warnings, 1 checks, 2806 lines checked
d45b72b8650c drm/i915: Remove GPU reset dependence on struct_mutex
-:639: WARNING:MEMORY_BARRIER: memory barrier without comment
#639: FILE: drivers/gpu/drm/i915/i915_reset.c:563:
+	smp_store_mb(i915->gpu_error.restart, NULL);

total: 0 errors, 1 warnings, 0 checks, 1264 lines checked