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

Patchwork patchwork at emeril.freedesktop.org
Wed Jun 26 01:21:39 UTC 2024


== Series Details ==

Series: Convert multiple bind ops to 1 job (rev5)
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 1e5e69ddd7423e10d5fc4efb50e3ed731354f747
Author: Matthew Brost <matthew.brost at intel.com>
Date:   Tue Jun 25 17:39:20 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 edccdf441f1e061f81e0d1d172aa0f37511bbe4d drm-intel
46bae314e0c6 drm/xe: s/xe_tile_migrate_engine/xe_tile_migrate_exec_queue
4e6efc993696 drm/xe: Add xe_vm_pgtable_update_op to xe_vma_ops
8889cb9da126 drm/xe: Add xe_exec_queue_last_fence_test_dep
54c54cb8a8a6 drm/xe: Convert multiple bind ops into single job
-:13: WARNING:TYPO_SPELLING: 'implemenation' may be misspelled - perhaps 'implementation'?
#13: 
The implemenation is roughly:
    ^^^^^^^^^^^^^

-:671: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#671: FILE: drivers/gpu/drm/xe/xe_pt.c:992:
+
+}

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

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

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

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

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

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

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

total: 1 errors, 6 warnings, 2 checks, 2526 lines checked
f607e08e843d drm/xe: Update VM trace events
d002f88d7ead drm/xe: Update PT layer with better error handling
1e5e69ddd742 drm/xe: Add VM bind IOCTL error injection




More information about the Intel-xe mailing list