✗ CI.checkpatch: warning for Convert multiple bind ops to 1 job (rev4)

Patchwork patchwork at emeril.freedesktop.org
Tue Jun 18 17:19:31 UTC 2024


== Series Details ==

Series: Convert multiple bind ops to 1 job (rev4)
URL   : https://patchwork.freedesktop.org/series/133034/
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
51ce9f6cd981d42d7467409d7dbc559a450abc1e
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit bd4d7d4d0a250cd8303200f9a25b417ab19862dd
Author: Matthew Brost <matthew.brost at intel.com>
Date:   Tue Jun 18 10:15:09 2024 -0700

    drm/xe: Add VM bind IOCTL error injection
    
    Add VM bind IOCTL error injection which steals MSB of the bind flags
    field which if set injects errors at various points in the VM bind
    IOCTL. Intended to validate error paths. Enabled by CONFIG_DRM_XE_DEBUG.
    
    v4:
     - Change define layout (Jonathan)
    
    Signed-off-by: Matthew Brost <matthew.brost at intel.com>
    Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
+ /mt/dim checkpatch fd8ad6cbb953a61cdbb7d7a10a13a2b5c8752c4c drm-intel
0b921daa3de4 drm/xe: s/xe_tile_migrate_engine/xe_tile_migrate_exec_queue
a2dc71ee40c4 drm/xe: Add xe_vm_pgtable_update_op to xe_vma_ops
95c4c26fc6db drm/xe: Add xe_exec_queue_last_fence_test_dep
3c7f57ad2c9b drm/xe: Convert multiple bind ops into single job
-:13: WARNING:TYPO_SPELLING: 'implemenation' may be misspelled - perhaps 'implementation'?
#13: 
The implemenation is roughly:
    ^^^^^^^^^^^^^

-:878: WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 15)
#878: FILE: drivers/gpu/drm/xe/xe_pt.c:1163:
+	if (uvma->userptr.initial_bind && !xe_vm_in_fault_mode(vm))
+               return 0;

-:879: ERROR:CODE_INDENT: code indent should use tabs where possible
#879: FILE: drivers/gpu/drm/xe/xe_pt.c:1164:
+               return 0;$

-:879: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#879: FILE: drivers/gpu/drm/xe/xe_pt.c:1164:
+               return 0;$

-:1332: CHECK:LINE_SPACING: Please don't use multiple blank lines
#1332: FILE: drivers/gpu/drm/xe/xe_pt.c:1620:
+
+

-:1662: WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#1662: FILE: drivers/gpu/drm/xe/xe_pt.c:1902:
+ * installing job fence in various places.
+  *

-:1663: WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#1663: FILE: drivers/gpu/drm/xe/xe_pt.c:1903:
+  *
+ * Return: fence on success, negative ERR_PTR on error.

-:1795: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1795: FILE: drivers/gpu/drm/xe/xe_pt.c:2035:
+ }$

total: 1 errors, 6 warnings, 1 checks, 2525 lines checked
12dbb169266f drm/xe: Update VM trace events
1537c7985bfa drm/xe: Update PT layer with better error handling
bd4d7d4d0a25 drm/xe: Add VM bind IOCTL error injection




More information about the Intel-xe mailing list