[Intel-xe] ✗ CI.checkpatch: warning for Some eviction fixes (rev4)

Patchwork patchwork at emeril.freedesktop.org
Tue Oct 31 08:17:50 UTC 2023


== Series Details ==

Series: Some eviction fixes (rev4)
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 d86842f6e9ae944972807dd8287696da98104ec9
Author: Matthew Auld <matthew.auld at intel.com>
Date:   Mon Oct 30 16:10:20 2023 +0000

    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 synchronisation for CPU access via the fault handler, and
    likewise for GPU access when vm_binding them.
    
    To fix this synchronise against any kernel fences for all KMD objects at
    creation. This should resolve some severe corruption seen during
    evictions.
    
    v2 (Matt B):
      - Revamp the comment explaining this. Also mention why USAGE_KERNEL is
        correct here.
    v3 (Thomas):
      - Make sure to use ctx.interruptible for the wait.
    
    Closes: ?
    Testcase: igt at xe-evict-ccs
    Reported-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
    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>
    Reviewed-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>
+ /mt/dim checkpatch 1e652bcf7cc139244f4f8c0cd1fc184029240050 drm-intel
fdc7b0e36 drm/xe/migrate: fix MI_ARB_ON_OFF usage
1a24c69f2 drm/xe/bo: consider dma-resv fences for clear job
d86842f6e drm/xe/bo: sync kernel fences for KMD buffers
-:27: WARNING:COMMIT_LOG_WRONG_LINK: 'Closes:' should be followed by a public http(s) link
#27: 
Closes: ?

-:29: WARNING:BAD_REPORTED_BY_LINK: Reported-by: should be immediately followed by Closes: with a URL to the report
#29: 
Reported-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
Signed-off-by: Matthew Auld <matthew.auld at intel.com>

total: 0 errors, 2 warnings, 0 checks, 37 lines checked




More information about the Intel-xe mailing list