✗ CI.checkpatch: warning for drm/xe: xe-only patches from the multi-device GPUSVM series (rev3)

Patchwork patchwork at emeril.freedesktop.org
Fri Mar 21 17:12:41 UTC 2025


== Series Details ==

Series: drm/xe: xe-only patches from the multi-device GPUSVM series (rev3)
URL   : https://patchwork.freedesktop.org/series/146383/
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
cbb4e4a079d89106c2736adc3c7de6f9dc56da07
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 1bd5e9b28228a2e4c13520bedce4a7edc7bc06c4
Author: Thomas Hellström <thomas.hellstrom at linux.intel.com>
Date:   Fri Mar 21 17:34:16 2025 +0100

    drm/xe/uapi, drm/xe: Make the PT code handle placement per PTE rather than per vma / range
    
    With SVM, ranges forwarded to the PT code for binding can, mostly
    due to races when migrating, point to both VRAM and system / foreign
    device memory. Make the PT code able to handle that by checking,
    for each PTE set up, whether it points to local VRAM or to system
    memory.
    
    The UAPI is changed implicitly in that before this patch,
    global atomics required a bo with VRAM/System placements. With
    this patch that is changed to requiring LR mode, and
    if the required placement is not available upon GPU atomic access
    pagefault, an error will be generated and the VM banned.
    
    v2:
    - Fix system memory GPU atomic access.
    
    Signed-off-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>
+ /mt/dim checkpatch 74f632d1bd3b90ed79883361ca25f1225c0aee58 drm-intel
76b1412c169c drm/xe: Introduce CONFIG_DRM_XE_GPUSVM
-:35: WARNING:CONFIG_DESCRIPTION: please write a help paragraph that fully describes the config symbol
#35: FILE: drivers/gpu/drm/xe/Kconfig:77:
+config DRM_XE_GPUSVM
+	bool "Enable CPU to GPU address mirroring"
+	depends on DRM_XE
+	depends on !UML
+	default y
+	select DEVICE_PRIVATE
+	help
+	  Enable this option if you want support for CPU to GPU address
+	  mirroring.
+
+	  If in doubut say "Y".
+

total: 0 errors, 1 warnings, 0 checks, 260 lines checked
ee0152523962 drm/xe/svm: Fix a potential bo UAF
66969780dfab drm/xe/bo: Add a bo remove callback
-:187: CHECK:SPACING: No space is necessary after a cast
#187: FILE: drivers/gpu/drm/xe/xe_bo_evict.c:200:
+	(void) xe_bo_apply_to_pinned(xe, &xe->pinned.external,

-:193: CHECK:SPACING: No space is necessary after a cast
#193: FILE: drivers/gpu/drm/xe/xe_bo_evict.c:206:
+	(void) xe_bo_apply_to_pinned(xe, &xe->pinned.kernel_bo_present,

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

-:199: CHECK:LINE_SPACING: Please don't use multiple blank lines
#199: FILE: drivers/gpu/drm/xe/xe_bo_evict.c:212:
+
+

total: 0 errors, 0 warnings, 4 checks, 228 lines checked
b02a337a61f7 drm/xe/migrate: Allow xe_migrate_vram() also on non-pagefault capable devices
1bd5e9b28228 drm/xe/uapi, drm/xe: Make the PT code handle placement per PTE rather than per vma / range




More information about the Intel-xe mailing list