[Intel-xe] ✗ CI.checkpatch: warning for drm/xe: Only set xe_vma_op.map fields for GPUVA map operations

Patchwork patchwork at emeril.freedesktop.org
Wed Nov 15 03:37:24 UTC 2023


== Series Details ==

Series: drm/xe: Only set xe_vma_op.map fields for GPUVA map operations
URL   : https://patchwork.freedesktop.org/series/126438/
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
63c2b6b160bca2df6efc7bc4cea6f442097d7854
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 8fcbc7eb123d8c066e018b136701165edbeae1d7
Author: Matthew Brost <matthew.brost at intel.com>
Date:   Tue Nov 14 19:15:08 2023 -0800

    drm/xe: Only set xe_vma_op.map fields for GPUVA map operations
    
    XE_VM_BIND_OP_MAP_* IOCTL operations can result in GPUVA unmap, remap,
    or map operations in vm_bind_ioctl_ops_create. The xe_vma_op.map fields
    are blindly set which is incorrect for GPUVA unmap or remap operations.
    Fix this by only setting xe_vma_op.map for GPUVA map operations. Also
    restructure a bit vm_bind_ioctl_ops_create to make the code a bit more
    readable.
    
    Reported-by: Dafna Hirschfeld <dhirschfeld at habana.ai>
    Signec-off-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch 2fe82053d916d09ffcc92bc27294c9eed4e4804b drm-intel
8fcbc7eb1 drm/xe: Only set xe_vma_op.map fields for GPUVA map operations
-:14: WARNING:BAD_REPORTED_BY_LINK: Reported-by: should be immediately followed by Closes: with a URL to the report
#14: 
Reported-by: Dafna Hirschfeld <dhirschfeld at habana.ai>
Signec-off-by: Matthew Brost <matthew.brost at intel.com>

-:15: WARNING:BAD_SIGN_OFF: Non-standard signature: 'Signec-off-by:' - perhaps 'Signed-off-by:'?
#15: 
Signec-off-by: Matthew Brost <matthew.brost at intel.com>

-:111: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 1 errors, 2 warnings, 0 checks, 88 lines checked




More information about the Intel-xe mailing list