[Intel-xe] ✗ CI.checkpatch: warning for Some eviction fixes
Patchwork
patchwork at emeril.freedesktop.org
Wed Oct 25 20:12:34 UTC 2023
== Series Details ==
Series: Some eviction fixes
URL : https://patchwork.freedesktop.org/series/125591/
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 f13d5939bab53fb22cd94058794021ee19986e8e
Author: Matthew Auld <matthew.auld at intel.com>
Date: Wed Oct 25 18:39:41 2023 +0100
drm/xe/bo: sync kernel fences for KMD buffers
With things like pipelined evictions, VRAM pages can be marked as free
and yet still have some active kernel fences, with the idea that the
next caller to allocate the memory will respect them. However it looks
like we are missing synchronisation for KMD internal buffers, like
page-tables, lrc etc. For userspace objects we should already have the
required serialisation for CPU access via the fault handler, and
likewise for GPU access when vm_binding them.
To fix this serialise against any kernel fences for all KMD objects at
creation. This seems to resolve some severe corruption seen during
evictions.
Closes: ?
Testcase: igt at xe-evict-ccs
Signed-off-by: Matthew Auld <matthew.auld at intel.com>
Cc: Thomas Hellström <thomas.hellstrom at linux.intel.com>
Cc: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch bd3d7e8aa312ce25c5797388f753151f024bec02 drm-intel
5f32a3f1f drm/xe/migrate: fix MI_ARB_ON_OFF usage
fbfe9ed32 drm/xe/bo: consider dma-resv fences for clear job
f13d5939b drm/xe/bo: sync kernel fences for KMD buffers
-:21: WARNING:COMMIT_LOG_WRONG_LINK: 'Closes:' should be followed by a public http(s) link
#21:
Closes: ?
total: 0 errors, 1 warnings, 0 checks, 16 lines checked
More information about the Intel-xe
mailing list