✗ CI.checkpatch: warning for drm/ttm: improve idle/busy handling v5

Patchwork patchwork at emeril.freedesktop.org
Thu Feb 29 19:37:18 UTC 2024


== Series Details ==

Series: drm/ttm: improve idle/busy handling v5
URL   : https://patchwork.freedesktop.org/series/130576/
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
9ab59fc4b7cb7663906ac4471a262434aef57f03
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit de956161f7c92743c40e5db717ee4100a96ba608
Author: Christian König <ckoenig.leichtzumerken at gmail.com>
Date:   Thu Feb 29 20:32:01 2024 +0100

    drm/ttm: improve idle/busy handling v5
    
    Previously we would never try to move a BO into the preferred placements
    when it ever landed in a busy placement since those were considered
    compatible.
    
    Rework the whole handling and finally unify the idle and busy handling.
    ttm_bo_validate() is now responsible to try idle placement first and then
    use the busy placement if that didn't worked.
    
    Drawback is that we now always try the idle placement first for each
    validation which might cause some additional CPU overhead on overcommit.
    
    v2: fix kerneldoc warning and coding style
    v3: take care of XE as well
    v4: keep the ttm_bo_mem_space functionality as it is for now, only add
        new handling for ttm_bo_validate as suggested by Thomas
    v5: fix bug pointed out by Matthew
    
    Signed-off-by: Christian König <christian.koenig at amd.com>
    Reviewed-by: Zack Rusin <zack.rusin at broadcom.com> v3
+ /mt/dim checkpatch 879fd254f581ab57486aae46f4a027e92642feb6 drm-intel
de956161f7c9 drm/ttm: improve idle/busy handling v5
-:27: WARNING:BAD_SIGN_OFF: Unexpected content after email: 'Zack Rusin <zack.rusin at broadcom.com> v3', should be: 'Zack Rusin <zack.rusin at broadcom.com> (v3)'
#27: 
Reviewed-by: Zack Rusin <zack.rusin at broadcom.com> v3

-:388: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken at gmail.com>' != 'Signed-off-by: Christian König <christian.koenig at amd.com>'

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




More information about the Intel-xe mailing list