[Intel-xe] ✗ CI.checkpatch: warning for drm/xe/bo: handle PL_TT -> PL_TT

[Patchwork]

== Series Details ==

Series: drm/xe/bo: handle PL_TT -> PL_TT
URL   : https://patchwork.freedesktop.org/series/119400/
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
c7d32770e3cd31d9fc134ce41f329b10aa33ee15
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit ec9a9303b36a6a25f42054ce3ef2cbe7caf5ee11
Author: Matthew Auld <matthew.auld at intel.com>
Date:   Thu Jun 15 19:18:40 2023 +0100

    drm/xe/bo: handle PL_TT -> PL_TT
    
    When moving between PL_VRAM <-> PL_SYSTEM we have to have use PL_TT in
    the middle as a temporary resource for the actual copy. In some GL
    workloads it can be seen that once the resource has been moved to the
    PL_TT we might have to bail out of the ttm_bo_validate(), before
    finishing the final hop. If this happens the resource is left as
    TTM_PL_FLAG_TEMPORARY, and when the ttm_bo_validate() is restarted the
    current placement is always seen as incompatible, requiring us to
    complete the move.  However if the BO allows PL_TT as a possible
    placement we can end up attempting a PL_TT -> PL_TT move (like when
    running out of VRAM) which leads to explosions in xe_bo_move(), like
    triggering the XE_BUG_ON(!tile).
    
    Going from TTM_PL_FLAG_TEMPORARY with PL_TT -> PL_VRAM should already
    work as-is, so it looks like we only need to worry about PL_TT -> PL_TT
    and it looks like we can just treat it as a dummy move, since no real
    move is needed.
    
    Reported-by: José Roberto de Souza <jose.souza at intel.com>
    Signed-off-by: Matthew Auld <matthew.auld at intel.com>
    Cc: Thomas Hellström <thomas.hellstrom at linux.intel.com>
+ /mt/dim checkpatch 6d2981d42cd891965094ec7d173203520cfa36f7 drm-intel
ec9a9303b drm/xe/bo: handle PL_TT -> PL_TT
-:26: WARNING:BAD_REPORTED_BY_LINK: Reported-by: should be immediately followed by Link: with a URL to the report
#26: 
Reported-by: José Roberto de Souza <jose.souza at intel.com>
Signed-off-by: Matthew Auld <matthew.auld at intel.com>

total: 0 errors, 1 warnings, 0 checks, 16 lines checked