✗ CI.checkpatch: warning for TTM shrinker helpers and xe buffer object shrinker (rev17)

[Patchwork]

== Series Details ==

Series: TTM shrinker helpers and xe buffer object shrinker (rev17)
URL   : https://patchwork.freedesktop.org/series/131815/
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
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 1ce29bb61f8837a7f81aad9117cc2f6b0e22654a
Author: Thomas Hellström <thomas.hellstrom at linux.intel.com>
Date:   Thu Jan 30 11:13:25 2025 +0100

    drm/xe: Increase the XE_PL_TT watermark
    
    The XE_PL_TT watermark was set to 50% of system memory.
    The idea behind that was unclear since the net effect is that
    TT memory will be evicted to TTM_PL_SYSTEM memory if that
    watermark is exceeded, requiring PPGTT rebinds and dma
    remapping. But there is no similar watermark for TTM_PL_1SYSTEM
    memory.
    
    The TTM functionality that tries to swap out system memory to
    shmem objects if a 50% limit of total system memory is reached
    is orthogonal to this, and with the shrinker added, it's no
    longer in effect.
    
    Replace the 50% TTM_PL_TT limit with a 100% limit, in effect
    allowing all graphics memory to be bound to the device unless it
    has been swapped out by the shrinker.
    
    Signed-off-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>
    Reviewed-by: Matthew Brost <matthew.brost at intel.com>
+ /mt/dim checkpatch c2a5da40b8b1c5af77dcdabed8516069949fea3b drm-intel
66458b481e4d drm/ttm: Provide a shmem backup implementation
-:56: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#56: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 289 lines checked
b2c00a58e480 drm/ttm/pool, drm/ttm/tt: Provide a helper to shrink pages
c7e4d970ff7c drm/ttm: Use fault-injection to test error paths
15ed3011bf88 drm/ttm: Add a macro to perform LRU iteration
-:11: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#11: 
https://lore.kernel.org/linux-mm/b7491378-defd-4f1c-31e2-29e4c77e2d67@amd.com/T/#ma918844aa8a6efe8768fdcda0c6590d5c93850c9

-:256: WARNING:TABSTOP: Statements should start on a tabstop
#256: FILE: include/drm/ttm/ttm_bo.h:510:
+	     if (_T) {ttm_bo_lru_cursor_fini(_T); },

-:256: ERROR:TRAILING_STATEMENTS: trailing statements should be on next line
#256: FILE: include/drm/ttm/ttm_bo.h:510:
+	     if (_T) {ttm_bo_lru_cursor_fini(_T); },

-:256: WARNING:BRACES: braces {} are not necessary for single statement blocks
#256: FILE: include/drm/ttm/ttm_bo.h:510:
+	     if (_T) {ttm_bo_lru_cursor_fini(_T); },

-:283: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#283: FILE: include/drm/ttm/ttm_bo.h:537:
+#define ttm_bo_lru_for_each_reserved_guarded(_cursor, _man, _ctx, _bo)	\
+	scoped_guard(ttm_bo_lru_cursor, _cursor, _man, _ctx)		\
+		for ((_bo) = ttm_bo_lru_cursor_first(_cursor); (_bo);	\
+		     (_bo) = ttm_bo_lru_cursor_next(_cursor))

-:283: CHECK:MACRO_ARG_REUSE: Macro argument reuse '_cursor' - possible side-effects?
#283: FILE: include/drm/ttm/ttm_bo.h:537:
+#define ttm_bo_lru_for_each_reserved_guarded(_cursor, _man, _ctx, _bo)	\
+	scoped_guard(ttm_bo_lru_cursor, _cursor, _man, _ctx)		\
+		for ((_bo) = ttm_bo_lru_cursor_first(_cursor); (_bo);	\
+		     (_bo) = ttm_bo_lru_cursor_next(_cursor))

-:283: CHECK:MACRO_ARG_REUSE: Macro argument reuse '_bo' - possible side-effects?
#283: FILE: include/drm/ttm/ttm_bo.h:537:
+#define ttm_bo_lru_for_each_reserved_guarded(_cursor, _man, _ctx, _bo)	\
+	scoped_guard(ttm_bo_lru_cursor, _cursor, _man, _ctx)		\
+		for ((_bo) = ttm_bo_lru_cursor_first(_cursor); (_bo);	\
+		     (_bo) = ttm_bo_lru_cursor_next(_cursor))

total: 2 errors, 3 warnings, 2 checks, 234 lines checked
36e548cf803f drm/ttm: Add helpers for shrinking
19a1147867dc drm/xe: Add a shrinker for xe bos
-:548: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#548: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 711 lines checked
1ce29bb61f88 drm/xe: Increase the XE_PL_TT watermark