== Series Details == Series: series starting with [01/39] drm/i915: Hold a ref to the ring while retiring URL : https://patchwork.freedesktop.org/series/57942/ State : warning == Summary == $ dim checkpatch origin/drm-tip 526bf4d5618e drm/i915: Hold a ref to the ring while retiring 848d7519ed33 drm/i915: Lock the gem_context->active_list while dropping the link 980a7574f55f drm/i915: Hold a reference to the active HW context 17195bbe8952 drm/i915: Stop needlessly acquiring wakeref for debugfs/drop_caches_set 8ab261c92dcc drm/i915/selftests: Provide stub reset functions 47ef34599402 drm/i915: Switch to use HWS indices rather than addresses 2ecbcbd82a8a drm/i915: Introduce the i915_user_extension_method -:70: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #70: new file mode 100644 -:75: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #75: FILE: drivers/gpu/drm/i915/i915_user_extensions.c:1: +/* -:140: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #140: FILE: drivers/gpu/drm/i915/i915_user_extensions.h:1: +/* -:168: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'ptr' - possible side-effects? #168: FILE: drivers/gpu/drm/i915/i915_utils.h:108: +#define container_of_user(ptr, type, member) ({ \ + void __user *__mptr = (void __user *)(ptr); \ + BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \ + !__same_type(*(ptr), void), \ + "pointer type mismatch in container_of()"); \ + ((type __user *)(__mptr - offsetof(type, member))); }) -:168: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'member' - possible side-effects? #168: FILE: drivers/gpu/drm/i915/i915_utils.h:108: +#define container_of_user(ptr, type, member) ({ \ + void __user *__mptr = (void __user *)(ptr); \ + BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \ + !__same_type(*(ptr), void), \ + "pointer type mismatch in container_of()"); \ + ((type __user *)(__mptr - offsetof(type, member))); }) -:168: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'member' may be better as '(member)' to avoid precedence issues #168: FILE: drivers/gpu/drm/i915/i915_utils.h:108: +#define container_of_user(ptr, type, member) ({ \ + void __user *__mptr = (void __user *)(ptr); \ + BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \ + !__same_type(*(ptr), void), \ + "pointer type mismatch in container_of()"); \ + ((type __user *)(__mptr - offsetof(type, member))); }) -:175: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'U' - possible side-effects? #175: FILE: drivers/gpu/drm/i915/i915_utils.h:115: +#define check_user_mbz(U) ({ \ + typeof(*(U)) mbz__; \ + get_user(mbz__, (U)) ? -EFAULT : mbz__ ? -EINVAL : 0; \ +}) total: 0 errors, 3 warnings, 4 checks, 132 lines checked 57178ebc3abd drm/i915: Create/destroy VM (ppGTT) for use with contexts -:45: CHECK:UNCOMMENTED_DEFINITION: struct mutex definition without comment #45: FILE: drivers/gpu/drm/i915/i915_drv.h:221: + struct mutex vm_lock; -:694: WARNING:LINE_SPACING: Missing a blank line after declarations #694: FILE: drivers/gpu/drm/i915/selftests/i915_gem_context.c:503: + struct drm_file *file; + IGT_TIMEOUT(end_time); -:756: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #756: FILE: drivers/gpu/drm/i915/selftests/i915_gem_context.c:565: + ncontexts = dw = 0; -:831: WARNING:LINE_SPACING: Missing a blank line after declarations #831: FILE: drivers/gpu/drm/i915/selftests/i915_gem_context.c:633: + struct i915_gem_context *ctx = NULL; + IGT_TIMEOUT(end_time); -:901: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #901: FILE: drivers/gpu/drm/i915/selftests/i915_gem_context.c:686: + ncontexts = dw = 0; -:1056: WARNING:LONG_LINE: line over 100 characters #1056: FILE: include/uapi/drm/i915_drm.h:407: +#define DRM_IOCTL_I915_GEM_VM_CREATE DRM_IOWR(DRM_COMMAND_BASE + DRM_I915_GEM_VM_CREATE, struct drm_i915_gem_vm_control) -:1057: WARNING:LONG_LINE: line over 100 characters #1057: FILE: include/uapi/drm/i915_drm.h:408: +#define DRM_IOCTL_I915_GEM_VM_DESTROY DRM_IOW (DRM_COMMAND_BASE + DRM_I915_GEM_VM_DESTROY, struct drm_i915_gem_vm_control) -:1057: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #1057: FILE: include/uapi/drm/i915_drm.h:408: +#define DRM_IOCTL_I915_GEM_VM_DESTROY DRM_IOW (DRM_COMMAND_BASE + DRM_I915_GEM_VM_DESTROY, struct drm_i915_gem_vm_control) -:1057: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #1057: FILE: include/uapi/drm/i915_drm.h:408: +#define DRM_IOCTL_I915_GEM_VM_DESTROY DRM_IOW (DRM_COMMAND_BASE + DRM_I915_GEM_VM_DESTROY, struct drm_i915_gem_vm_control) total: 1 errors, 5 warnings, 3 checks, 1004 lines checked eefcebbc441f drm/i915: Extend CONTEXT_CREATE to set parameters upon construction -:28: WARNING:LONG_LINE: line over 100 characters #28: FILE: drivers/gpu/drm/i915/i915_drv.c:3113: + DRM_IOCTL_DEF_DRV(I915_GEM_CONTEXT_CREATE_EXT, i915_gem_context_create_ioctl, DRM_RENDER_ALLOW), -:535: WARNING:LONG_LINE: line over 100 characters #535: FILE: include/uapi/drm/i915_drm.h:397: +#define DRM_IOCTL_I915_GEM_CONTEXT_CREATE_EXT DRM_IOWR (DRM_COMMAND_BASE + DRM_I915_GEM_CONTEXT_CREATE, struct drm_i915_gem_context_create_ext) -:535: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #535: FILE: include/uapi/drm/i915_drm.h:397: +#define DRM_IOCTL_I915_GEM_CONTEXT_CREATE_EXT DRM_IOWR (DRM_COMMAND_BASE + DRM_I915_GEM_CONTEXT_CREATE, struct drm_i915_gem_context_create_ext) -:535: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #535: FILE: include/uapi/drm/i915_drm.h:397: +#define DRM_IOCTL_I915_GEM_CONTEXT_CREATE_EXT DRM_IOWR (DRM_COMMAND_BASE + DRM_I915_GEM_CONTEXT_CREATE, struct drm_i915_gem_context_create_ext) total: 1 errors, 3 warnings, 0 checks, 694 lines checked 8f38f7ec1974 drm/i915: Allow contexts to share a single timeline across all engines 2d91c1810223 drm/i915: Allow userspace to clone contexts on creation fc4129cd0378 drm/i915: Allow a context to define its set of engines -:448: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'T' - possible side-effects? #448: FILE: drivers/gpu/drm/i915/i915_utils.h:94: +#define check_struct_size(T, A, C, SZ) \ + likely(__check_struct_size(sizeof(*(T)), \ + sizeof(*(T)->A) + __must_be_array((T)->A), \ + C, SZ)) -:448: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'A' - possible side-effects? #448: FILE: drivers/gpu/drm/i915/i915_utils.h:94: +#define check_struct_size(T, A, C, SZ) \ + likely(__check_struct_size(sizeof(*(T)), \ + sizeof(*(T)->A) + __must_be_array((T)->A), \ + C, SZ)) -:448: CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'A' may be better as '(A)' to avoid precedence issues #448: FILE: drivers/gpu/drm/i915/i915_utils.h:94: +#define check_struct_size(T, A, C, SZ) \ + likely(__check_struct_size(sizeof(*(T)), \ + sizeof(*(T)->A) + __must_be_array((T)->A), \ + C, SZ)) total: 0 errors, 0 warnings, 3 checks, 457 lines checked 2a504a76fd93 drm/i915: Extend I915_CONTEXT_PARAM_SSEU to support local ctx->engine[] aa184a5b051d drm/i915: Load balancing across a virtual engine -:962: WARNING:LINE_SPACING: Missing a blank line after declarations #962: FILE: drivers/gpu/drm/i915/intel_lrc.c:3392: + struct intel_engine_cs *actual = ve->siblings[0]; + intel_context_put(&ve->context); total: 0 errors, 1 warnings, 0 checks, 1163 lines checked f139fd08d886 drm/i915: Extend execution fence to support a callback 145305fa38f2 drm/i915/execlists: Virtual engine bonding -:472: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #472: FILE: drivers/gpu/drm/i915/selftests/intel_lrc.c:1318: + if (i915_request_wait(rq[0], + I915_WAIT_LOCKED, total: 0 errors, 0 warnings, 1 checks, 558 lines checked 641e83a992cb drm/i915: Allow specification of parallel execbuf 0ca268bda92a drm/i915/execlists: Skip direct submission if only lite-restore 78aa0598ecbe drm/i915: Split GEM object type definition to its own header -:34: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #34: new file mode 100644 -:39: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #39: FILE: drivers/gpu/drm/i915/gem/i915_gem_object_types.h:1: +/* -:157: WARNING:LINE_SPACING: Missing a blank line after declarations #157: FILE: drivers/gpu/drm/i915/gem/i915_gem_object_types.h:119: + struct list_head batch_pool_link; + I915_SELFTEST_DECLARE(struct list_head st_link); -:267: WARNING:BOOL_BITFIELD: Avoid using bool as bitfield. Prefer bool bitfields as unsigned int or u<8|16|32> #267: FILE: drivers/gpu/drm/i915/gem/i915_gem_object_types.h:229: + bool dirty:1; -:273: WARNING:BOOL_BITFIELD: Avoid using bool as bitfield. Prefer bool bitfields as unsigned int or u<8|16|32> #273: FILE: drivers/gpu/drm/i915/gem/i915_gem_object_types.h:235: + bool quirked:1; -:330: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #330: FILE: drivers/gpu/drm/i915/gem/test_i915_gem_object_types_standalone.c:1: +/* total: 0 errors, 6 warnings, 0 checks, 662 lines checked 27f0bf2b4439 drm/i915: Pull GEM ioctls interface to its own file -:24: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #24: new file mode 100644 -:29: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #29: FILE: drivers/gpu/drm/i915/gem/i915_gem_ioctls.h:1: +/* -:87: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #87: FILE: drivers/gpu/drm/i915/gem/test_i915_gem_ioctls_standalone.c:1: +/* total: 0 errors, 3 warnings, 0 checks, 169 lines checked 5b1b7f5ca28b drm/i915: Move object->pages API to i915_gem_object.[ch] -:43: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #43: rename from drivers/gpu/drm/i915/i915_gem_object.c -:207: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #207: FILE: drivers/gpu/drm/i915/gem/test_i915_gem_object_standalone.c:1: +/* total: 0 errors, 2 warnings, 0 checks, 326 lines checked 41944053d575 drm/i915: Move shmem object setup to its own file -:408: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #408: new file mode 100644 -:413: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #413: FILE: drivers/gpu/drm/i915/gem/i915_gem_shmem.c:1: +/* -:917: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #917: FILE: drivers/gpu/drm/i915/gvt/cmd_parser.c:1730: + bb->obj = i915_gem_object_create_shmem(s->vgpu->gvt->dev_priv, roundup(bb_size + gma_start_offset, PAGE_SIZE)); total: 0 errors, 2 warnings, 1 checks, 1937 lines checked 5620e146ae17 drm/i915: Move phys objects to its own file -:70: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #70: new file mode 100644 -:75: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #75: FILE: drivers/gpu/drm/i915/gem/i915_gem_pages.c:1: +/* -:167: WARNING:MEMORY_BARRIER: memory barrier without comment #167: FILE: drivers/gpu/drm/i915/gem/i915_gem_pages.c:93: + smp_mb__before_atomic(); -:239: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #239: FILE: drivers/gpu/drm/i915/gem/i915_gem_pages.c:165: + obj->mm.page_sizes.phys = obj->mm.page_sizes.sg = 0; -:365: WARNING:MEMORY_BARRIER: memory barrier without comment #365: FILE: drivers/gpu/drm/i915/gem/i915_gem_pages.c:291: + smp_mb__before_atomic(); -:384: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #384: FILE: drivers/gpu/drm/i915/gem/i915_gem_pages.c:310: + ptr = obj->mm.mapping = NULL; -:563: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #563: FILE: drivers/gpu/drm/i915/gem/i915_gem_phys.c:1: +/* -:821: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #821: FILE: drivers/gpu/drm/i915/gem/selftests/i915_gem_phys.c:1: +/* total: 0 errors, 6 warnings, 2 checks, 1651 lines checked f832228e1c87 drm/i915: Move mmap and friends to its own file -:25: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #25: new file mode 100644 -:30: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #30: FILE: drivers/gpu/drm/i915/gem/i915_gem_mman.c:1: +/* -:599: WARNING:MEMORY_BARRIER: memory barrier without comment #599: FILE: drivers/gpu/drm/i915/gem/i915_gem_object.c:400: + wmb(); -:642: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #642: FILE: drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c:1: +/* -:834: WARNING:LONG_LINE: line over 100 characters #834: FILE: drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c:193: + (1 + next_prime_number(i915->ggtt.vm.total >> PAGE_SHIFT)) << PAGE_SHIFT); -:953: WARNING:SPACING: space prohibited before semicolon #953: FILE: drivers/gpu/drm/i915/gem/selftests/i915_gem_mman.c:312: +next_tiling: ; total: 0 errors, 6 warnings, 0 checks, 2280 lines checked 02c9e4a87630 drm/i915: Move GEM domain management to its own file -:25: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #25: new file mode 100644 -:30: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #30: FILE: drivers/gpu/drm/i915/gem/i915_gem_domain.c:1: +/* -:113: WARNING:MEMORY_BARRIER: memory barrier without comment #113: FILE: drivers/gpu/drm/i915/gem/i915_gem_domain.c:84: + mb(); -:175: WARNING:MEMORY_BARRIER: memory barrier without comment #175: FILE: drivers/gpu/drm/i915/gem/i915_gem_domain.c:146: + mb(); total: 0 errors, 4 warnings, 0 checks, 1867 lines checked 4da92633aee7 drm/i915: Move more GEM objects under gem/ -:73: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #73: rename from drivers/gpu/drm/i915/i915_gem_clflush.c -:121: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #121: FILE: drivers/gpu/drm/i915/gem/i915_gem_clflush.h:1: +/* -:719: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #719: FILE: drivers/gpu/drm/i915/gem/i915_gemfs.h:1: +/* -:776: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #776: FILE: drivers/gpu/drm/i915/gem/selftests/huge_gem_object.h:1: +/* -:1102: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #1102: FILE: drivers/gpu/drm/i915/gem/selftests/mock_context.h:1: +/* -:1167: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #1167: FILE: drivers/gpu/drm/i915/gem/selftests/mock_dmabuf.h:1: +/* -:1198: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #1198: FILE: drivers/gpu/drm/i915/gem/selftests/mock_gem_object.h:1: +/* -:1213: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #1213: FILE: drivers/gpu/drm/i915/gem/test_i915_gem_clflush_standalone.c:1: +/* -:1226: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #1226: FILE: drivers/gpu/drm/i915/gem/test_i915_gem_context_standalone.c:1: +/* -:1243: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #1243: FILE: drivers/gpu/drm/i915/gem/test_i915_gemfs_standalone.c:1: +/* total: 0 errors, 10 warnings, 0 checks, 1330 lines checked 8c80a4f633c4 drm/i915: Pull scatterlist utils out of i915_gem.h -:347: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #347: new file mode 100644 -:352: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #352: FILE: drivers/gpu/drm/i915/i915_scatterlist.c:1: +/* -:397: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #397: FILE: drivers/gpu/drm/i915/i915_scatterlist.h:1: +/* -:427: CHECK:MULTIPLE_ASSIGNMENTS: multiple assignments should be avoided #427: FILE: drivers/gpu/drm/i915/i915_scatterlist.h:31: + s.max = s.curr = s.sgp->offset; -:471: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__iter' - possible side-effects? #471: FILE: drivers/gpu/drm/i915/i915_scatterlist.h:75: +#define for_each_sgt_dma(__dmap, __iter, __sgt) \ + for ((__iter) = __sgt_iter((__sgt)->sgl, true); \ + ((__dmap) = (__iter).dma + (__iter).curr); \ + (((__iter).curr += I915_GTT_PAGE_SIZE) >= (__iter).max) ? \ + (__iter) = __sgt_iter(__sg_next((__iter).sgp), true), 0 : 0) -:483: CHECK:MACRO_ARG_REUSE: Macro argument reuse '__iter' - possible side-effects? #483: FILE: drivers/gpu/drm/i915/i915_scatterlist.h:87: +#define for_each_sgt_page(__pp, __iter, __sgt) \ + for ((__iter) = __sgt_iter((__sgt)->sgl, false); \ + ((__pp) = (__iter).pfn == 0 ? NULL : \ + pfn_to_page((__iter).pfn + ((__iter).curr >> PAGE_SHIFT))); \ + (((__iter).curr += PAGE_SIZE) >= (__iter).max) ? \ + (__iter) = __sgt_iter(__sg_next((__iter).sgp), false), 0 : 0) total: 0 errors, 3 warnings, 3 checks, 443 lines checked 100f378e5545 drm/i915: Move GEM object domain management from struct_mutex to local -:534: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #534: new file mode 100644 -:539: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #539: FILE: drivers/gpu/drm/i915/gem/i915_gem_fence.c:1: +/* total: 0 errors, 2 warnings, 0 checks, 1621 lines checked ec1f3770a99a drm/i915: Move GEM object waiting to its own file -:41: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #41: new file mode 100644 -:46: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #46: FILE: drivers/gpu/drm/i915/gem/i915_gem_wait.c:1: +/* total: 0 errors, 2 warnings, 0 checks, 590 lines checked b983c7cdca32 drm/i915: Move GEM object busy checking to its own file -:24: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #24: new file mode 100644 -:29: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #29: FILE: drivers/gpu/drm/i915/gem/i915_gem_busy.c:1: +/* total: 0 errors, 2 warnings, 0 checks, 277 lines checked 3e47db72c86b drm/i915: Move GEM client throttling to its own file -:24: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #24: new file mode 100644 -:29: WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #29: FILE: drivers/gpu/drm/i915/gem/i915_gem_throttle.c:1: +/* total: 0 errors, 2 warnings, 0 checks, 163 lines checked 57012dbc5c4a drm/i915: Drop the deferred active reference 6e6db4867a8e drm/i915: Move object close under its own lock ce65f0cefd1e drm/i915: Rename intel_context.active to .inflight cee986fa6182 drm/i915: Keep contexts pinned until after the next kernel context switch -:222: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #222: FILE: drivers/gpu/drm/i915/i915_gem.c:84: + llist_for_each_safe(node, next, + llist_del_all(&engine->barrier_tasks)) { -:225: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #225: FILE: drivers/gpu/drm/i915/i915_gem.c:87: + container_of((struct list_head *)node, + typeof(*active), link); total: 0 errors, 0 warnings, 2 checks, 778 lines checked bf3e783b1eeb drm/i915: Stop retiring along engine a5c5069955ba drm/i915: Replace engine->timeline with a plain list -:551: CHECK:UNCOMMENTED_DEFINITION: spinlock_t definition without comment #551: FILE: drivers/gpu/drm/i915/intel_engine_types.h:273: + spinlock_t lock; total: 0 errors, 0 warnings, 1 checks, 925 lines checked 8e9f0402b1b1 drm/i915/execlists: Preempt-to-busy -:157: CHECK:MACRO_ARG_REUSE: Macro argume _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx