✗ CI.checkpatch: warning for Split out dma fence array and invalidate media_gt TLBs in PT code (rev2)

Patchwork patchwork at emeril.freedesktop.org
Mon Aug 26 18:04:02 UTC 2024


== Series Details ==

Series: Split out dma fence array and invalidate media_gt TLBs in PT code (rev2)
URL   : https://patchwork.freedesktop.org/series/137805/
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 82456ca6fc25a2b7fb58f96702a30e801d621156
Author: Matthew Brost <matthew.brost at intel.com>
Date:   Mon Aug 26 10:01:44 2024 -0700

    drm/xe: Invalidate media_gt TLBs in PT code
    
    Testing on LNL has shown media GT's TLBs need to be invalidated via the
    GuC, update PT code appropriately.
    
    v2:
     - Do dma_fence_get before first call of invalidation_fence_init (Himal)
     - No need to check for valid chain fence (Himal)
    v3:
     - Use dma-fence-array
    
    Fixes: 3330361543fc ("drm/xe/lnl: Add LNL platform definition")
    Signed-off-by: Matthew Brost <matthew.brost at intel.com>
    Acked-by: Christian König <christian.koenig at amd.com>
+ /mt/dim checkpatch 2c93383f8014669a19878de3293fd74d3e2474e4 drm-intel
45459327e146 dma-buf: Split out dma fence array create into alloc and arm functions
-:64: WARNING:REPEATED_WORD: Possible repeated word: 'fence'
#64: FILE: drivers/dma-buf/dma-fence-array.c:170:
+ * preallocated dma fence fence in the path of reclaim or dma fence signaling.

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

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

-:133: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 0)
#133: 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);

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

total: 0 errors, 5 warnings, 0 checks, 112 lines checked
82456ca6fc25 drm/xe: Invalidate media_gt TLBs in PT code




More information about the Intel-xe mailing list