✗ CI.checkpatch: warning for Replace dma-fence chain with dma-fence array for media GT TLB invalidation

Patchwork patchwork at emeril.freedesktop.org
Fri Aug 23 05:04:26 UTC 2024


== Series Details ==

Series: Replace dma-fence chain with dma-fence array for media GT TLB invalidation
URL   : https://patchwork.freedesktop.org/series/137684/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
9fe5037901cabbcdf27a6fe0dfb047ca1474d363
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 8ec783004d432b3e19d2b08577e91d4b5a45641e
Author: Matthew Brost <matthew.brost at intel.com>
Date:   Thu Aug 22 21:54:43 2024 -0700

    drm/xe: Use dma-fence array for media GT TLB invalidations in PT code
    
    Using a chain fence is problematic as these cannot be installed in
    timeout drm sync objects. Use a dma-fence-array instead at the cost of
    an extra failure point.
    
    Also fixup reserve fence count to include media GT invalidation fence.
    
    v2:
     - Fix reserve fence count (Casey Bowman)
    v3:
     - Prealloc dma fence array (CI)
    
    Fixes: 40520283e0fd ("drm/xe: Invalidate media_gt TLBs in PT code")
    Signed-off-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch 7365d672ab38ee0eabb65fcd359e5774a8efbd70 drm-intel
2247aa4d6414 dma-buf: Split out dma fence array create into alloc and arm functions
-:59: WARNING:REPEATED_WORD: Possible repeated word: 'fence'
#59: FILE: drivers/dma-buf/dma-fence-array.c:170:
+ * preallocated dma fence fence in the path of reclaim or dma fence signaling.

-:68: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#68: FILE: drivers/dma-buf/dma-fence-array.c:175:
+			 u64 context, unsigned seqno,

-:110: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#110: FILE: drivers/dma-buf/dma-fence-array.c:229:
+					       u64 context, unsigned seqno,

-:129: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 0)
#129: FILE: include/linux/dma-fence-array.h:79:
 	for (index = 0, fence = dma_fence_array_first(head); fence;	\
[...]
+struct dma_fence_array *dma_fence_array_alloc(int num_fences);

-:136: WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#136: FILE: include/linux/dma-fence-array.h:86:
+			 u64 context, unsigned seqno,

total: 0 errors, 5 warnings, 0 checks, 114 lines checked
8ec783004d43 drm/xe: Use dma-fence array for media GT TLB invalidations in PT code




More information about the Intel-xe mailing list