✗ 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