✗ CI.checkpatch: warning for drm/xe: xe-only patches from the multi-device GPUSVM series (rev4)
Patchwork
patchwork at emeril.freedesktop.org
Mon Mar 24 08:36:32 UTC 2025
== Series Details ==
Series: drm/xe: xe-only patches from the multi-device GPUSVM series (rev4)
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 514292657d5ba14b72d7267218da54de910be764
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>
Reviewed-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch 17f81b31a1d8e62f021bbb43e3744e5540e2ecb3 drm-intel
24ff1bb5518d 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
0fef0e6f51e6 drm/xe/svm: Fix a potential bo UAF
4b5971dd509b 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
7403b6dea167 drm/xe/migrate: Allow xe_migrate_vram() also on non-pagefault capable devices
514292657d5b 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