✗ CI.checkpatch: warning for series starting with [1/6] drm/vmwgfx: drop printing the TTM refcount for debugging (rev2)
Patchwork
patchwork at emeril.freedesktop.org
Mon Jun 23 09:35:31 UTC 2025
== Series Details ==
Series: series starting with [1/6] drm/vmwgfx: drop printing the TTM refcount for debugging (rev2)
URL : https://patchwork.freedesktop.org/series/150325/
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
f8ff75ae1d2127635239b134695774ed4045d05b
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 5c0d086110fa2a2f331908ea42d44878a5a055cb
Author: Christian König <ckoenig.leichtzumerken at gmail.com>
Date: Mon Jun 16 15:07:26 2025 +0200
drm/ttm: replace TTMs refcount with the DRM refcount
Instead of keeping a separate reference count for the TTM object also use
the reference count for DRM GEM objects inside TTM.
Apart from avoiding two reference counts for one object this approach has
the clear advantage of being able to use drm_exec inside TTM.
Signed-off-by: Christian König <christian.koenig at amd.com>
+ /mt/dim checkpatch 5aa80933c50abb7ebb5776aea2d14762d8449c5e drm-intel
0faac69a1154 drm/vmwgfx: drop printing the TTM refcount for debugging
-:30: 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, 1 warnings, 0 checks, 13 lines checked
2f457d998a9e drm/xe: stop asserting on the TTM refcount
-:25: 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, 1 warnings, 0 checks, 8 lines checked
060edd4dc736 drm/ttm: fix error handling in ttm_buffer_object_transfer
-:45: 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, 1 warnings, 0 checks, 25 lines checked
e3dc95355b13 drm/ttm: rename ttm_bo_put to _fini
-:501: 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, 1 warnings, 0 checks, 388 lines checked
a444c041f2ec drm/ttm: disable ttm_bo_validate_deleted_evict for now
-:22: WARNING:IF_0: Consider removing the code enclosed by this #if 0 and its #endif
#22: FILE: drivers/gpu/drm/ttm/tests/ttm_bo_validate_test.c:980:
+#if 0
-:42: 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, 22 lines checked
5c0d086110fa drm/ttm: replace TTMs refcount with the DRM refcount
-:163: CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'bo->ttm != NULL'
#163: FILE: drivers/gpu/drm/ttm/ttm_bo.c:292:
+ if (!dma_resv_test_signaled(&bo->base._resv, DMA_RESV_USAGE_BOOKKEEP) ||
+ (want_init_on_free() && (bo->ttm != NULL)) ||
+ bo->type == ttm_bo_type_sg ||
+ !dma_resv_trylock(bo->base.resv)) {
-:164: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "bo->ttm"
#164: FILE: drivers/gpu/drm/ttm/ttm_bo.c:293:
+ (want_init_on_free() && (bo->ttm != NULL)) ||
-:350: 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, 1 warnings, 2 checks, 294 lines checked
More information about the Intel-xe
mailing list