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

[Patchwork]

== Series Details ==

Series: TTM shrinker helpers and xe buffer object shrinker (rev9)
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
c62d7e164862503a3662a095da1c6c9014248cb2
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 8a3085a7b959e7205a933412abfd25f2f3e1d9cf
Author: Thomas Hellström <thomas.hellstrom at linux.intel.com>
Date:   Mon Sep 16 16:51:12 2024 +0200

    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 cb4252b3107f36d565422099fdde8d7e34ffbf1d drm-intel
2cc2ddc967e0 drm/ttm: Balance ttm_resource_cursor_init() and ttm_resource_cursor_fini()
-:154: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cursor' - possible side-effects?
#154: 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))

-:154: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'res' - possible side-effects?
#154: 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
d0c6dc9a050e drm/ttm: Add a virtual base class for graphics memory backup
-:53: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#53: 
new file mode 100644

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

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

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

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

-:357: ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#357: FILE: include/drm/ttm/ttm_bo.h:549:
+#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))

-:357: CHECK:MACRO_ARG_REUSE: Macro argument reuse '_cursor' - possible side-effects?
#357: FILE: include/drm/ttm/ttm_bo.h:549:
+#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))

-:357: CHECK:MACRO_ARG_REUSE: Macro argument reuse '_bo' - possible side-effects?
#357: FILE: include/drm/ttm/ttm_bo.h:549:
+#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, 320 lines checked
ec366ca8adc3 drm/xe: Add a shrinker for xe bos
-:531: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#531: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 695 lines checked
8a3085a7b959 drm/xe: Increase the XE_PL_TT watermark