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

Patchwork patchwork at emeril.freedesktop.org
Wed May 15 00:44:48 UTC 2024


== Series Details ==

Series: Convert multiple bind ops to 1 job (rev2)
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
57b97a66dd129aea93991dc66cd10477f7a05cf8
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit f7d9f9971bc4cb1b718d662cf0508f20bc14ddcc
Author: Matthew Brost <matthew.brost at intel.com>
Date:   Tue May 14 17:40:10 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.
    
    Signed-off-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch 4cb2d17202b74ed81c0b6f22e540d811458869df drm-intel
3a56e48610f5 drm/xe: s/xe_tile_migrate_engine/xe_tile_migrate_exec_queue
3240cf16b6e2 drm/xe: Add xe_vm_pgtable_update_op to xe_vma_ops
15c6214d947a drm/xe: Convert multiple bind ops into single job
-:13: WARNING:TYPO_SPELLING: 'implemenation' may be misspelled - perhaps 'implementation'?
#13: 
The implemenation is roughly:
    ^^^^^^^^^^^^^

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

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

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

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

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

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

total: 1 errors, 6 warnings, 0 checks, 2524 lines checked
4c6cbd0fd7a9 drm/xe: Update VM trace events
2e82846680c6 drm/xe: Update PT layer with better error handling
f7d9f9971bc4 drm/xe: Add VM bind IOCTL error injection
-:99: ERROR:TRAILING_WHITESPACE: trailing whitespace
#99: FILE: drivers/gpu/drm/xe/xe_vm.c:2718:
+^I DRM_XE_VM_BIND_FLAG_DUMPABLE) $

total: 1 errors, 0 warnings, 0 checks, 121 lines checked




More information about the Intel-xe mailing list