== 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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
