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

[Patchwork]

== Series Details ==

Series: TTM shrinker helpers and xe buffer object shrinker (rev16)
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 51c1509c74d7525876363b7a5e2d93d6d55319ab
Author: Thomas Hellström <thomas.hellstrom at linux.intel.com>
Date:   Tue Dec 17 15:58:52 2024 +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 1984b26a25a506f3437f63fd201cec704b99336a drm-intel
469728109937 drm/ttm: Balance ttm_resource_cursor_init() and ttm_resource_cursor_fini()
-:155: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cursor' - possible side-effects?
#155: FILE: include/drm/ttm/ttm_resource.h:476:
+#define ttm_resource_manager_for_each_res(cursor, res)	\
+	for (res = ttm_resource_manager_first(cursor); res;	\
 	     res = ttm_resource_manager_next(cursor))

-:155: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'res' - possible side-effects?
#155: FILE: include/drm/ttm/ttm_resource.h:476:
+#define ttm_resource_manager_for_each_res(cursor, res)	\
+	for (res = ttm_resource_manager_first(cursor); res;	\
 	     res = ttm_resource_manager_next(cursor))

total: 0 errors, 0 warnings, 2 checks, 114 lines checked
81c97b5ab14d 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
8eaa85135ed0 drm/ttm/pool: Restructure the pool allocation code
c2367aaed978 drm/ttm/pool, drm/ttm/tt: Provide a helper to shrink pages
e66070f7d538 drm/ttm: Use fault-injection to test error paths
0d5891ed097d 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
8cc36a03a449 drm/ttm: Add helpers for shrinking
7f90f6944677 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
51c1509c74d7 drm/xe: Increase the XE_PL_TT watermark