== Series Details == Series: series starting with [01/26] dma-buf: add dma_resv_for_each_fence_unlocked v2 URL : https://patchwork.freedesktop.org/series/94750/ State : warning == Summary == $ dim checkpatch origin/drm-tip 79becb99fccc dma-buf: add dma_resv_for_each_fence_unlocked v2 -:72: CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #72: FILE: drivers/dma-buf/dma-resv.c:368: + fence = rcu_dereference( -:134: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'obj' - possible side-effects? #134: FILE: include/linux/dma-resv.h:183: +#define dma_resv_for_each_fence_unlocked(obj, cursor, all_fences, fence) \ + for (fence = dma_resv_walk_unlocked(obj, cursor, all_fences, true); \ + fence; dma_fence_put(fence), \ + fence = dma_resv_walk_unlocked(obj, cursor, all_fences, false)) -:134: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cursor' - possible side-effects? #134: FILE: include/linux/dma-resv.h:183: +#define dma_resv_for_each_fence_unlocked(obj, cursor, all_fences, fence) \ + for (fence = dma_resv_walk_unlocked(obj, cursor, all_fences, true); \ + fence; dma_fence_put(fence), \ + fence = dma_resv_walk_unlocked(obj, cursor, all_fences, false)) -:134: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'all_fences' - possible side-effects? #134: FILE: include/linux/dma-resv.h:183: +#define dma_resv_for_each_fence_unlocked(obj, cursor, all_fences, fence) \ + for (fence = dma_resv_walk_unlocked(obj, cursor, all_fences, true); \ + fence; dma_fence_put(fence), \ + fence = dma_resv_walk_unlocked(obj, cursor, all_fences, false)) -:134: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fence' - possible side-effects? #134: FILE: include/linux/dma-resv.h:183: +#define dma_resv_for_each_fence_unlocked(obj, cursor, all_fences, fence) \ + for (fence = dma_resv_walk_unlocked(obj, cursor, all_fences, true); \ + fence; dma_fence_put(fence), \ + fence = dma_resv_walk_unlocked(obj, cursor, all_fences, false)) -:162: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 5 checks, 130 lines checked 43fd9d11fcd6 dma-buf: add dma_resv_for_each_fence -:81: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'obj' - possible side-effects? #81: FILE: include/linux/dma-resv.h:182: +#define dma_resv_for_each_fence(obj, cursor, all_fences, fence) \ + for (fence = dma_resv_walk(obj, cursor, all_fences, true); fence; \ + fence = dma_resv_walk(obj, cursor, all_fences, false)) -:81: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cursor' - possible side-effects? #81: FILE: include/linux/dma-resv.h:182: +#define dma_resv_for_each_fence(obj, cursor, all_fences, fence) \ + for (fence = dma_resv_walk(obj, cursor, all_fences, true); fence; \ + fence = dma_resv_walk(obj, cursor, all_fences, false)) -:81: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'all_fences' - possible side-effects? #81: FILE: include/linux/dma-resv.h:182: +#define dma_resv_for_each_fence(obj, cursor, all_fences, fence) \ + for (fence = dma_resv_walk(obj, cursor, all_fences, true); fence; \ + fence = dma_resv_walk(obj, cursor, all_fences, false)) -:81: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fence' - possible side-effects? #81: FILE: include/linux/dma-resv.h:182: +#define dma_resv_for_each_fence(obj, cursor, all_fences, fence) \ + for (fence = dma_resv_walk(obj, cursor, all_fences, true); fence; \ + fence = dma_resv_walk(obj, cursor, all_fences, false)) -:97: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 4 checks, 73 lines checked a8b725463938 dma-buf: use new iterator in dma_resv_copy_fences -:124: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 106 lines checked 20181be832c0 dma-buf: use new iterator in dma_resv_get_fences v2 -:156: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 136 lines checked d07cfed64ece dma-buf: use new iterator in dma_resv_wait_timeout -:99: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 81 lines checked f26d368dba9c dma-buf: use new iterator in dma_resv_test_signaled -:91: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 72 lines checked bd71df193561 drm/ttm: use the new iterator in ttm_bo_flush_all_fences -:39: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 21 lines checked 174500546b2b drm/amdgpu: use the new iterator in amdgpu_sync_resv -:72: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 55 lines checked f8f742ac41e1 drm/amdgpu: use new iterator in amdgpu_ttm_bo_eviction_valuable -:47: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 28 lines checked cc28ebf6dea3 drm/msm: use new iterator in msm_gem_describe -:55: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 34 lines checked 8fe08b573247 drm/radeon: use new iterator in radeon_sync_resv -:53: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 36 lines checked d5ec3add7f52 drm/scheduler: use new iterator in drm_sched_job_add_implicit_dependencies v2 -:56: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 36 lines checked f4e64617226b drm/i915: use the new iterator in i915_gem_busy_ioctl -:64: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 45 lines checked 77c552340892 drm/i915: use the new iterator in i915_sw_fence_await_reservation v2 -:86: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 65 lines checked 3b3d6edc3ec1 drm/i915: use the new iterator in i915_request_await_object v2 -:70: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 50 lines checked fbe3139fe0d5 drm/i915: use new iterator in i915_gem_object_wait_reservation v2 -:49: CHECK:BRACES: Blank lines aren't necessary after an open brace '{' #49: FILE: drivers/gpu/drm/i915/gem/i915_gem_wait.c:46: + fence) { + -:90: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 1 checks, 70 lines checked 01cb40f10c37 drm/i915: use new iterator in i915_gem_object_wait_priority v2 -:61: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 41 lines checked e4c67e16e633 drm/i915: use new iterator in i915_gem_object_last_write_engine v2 -:43: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 23 lines checked 36237da723df drm/i915: use new cursor in intel_prepare_plane_fb v2 -:45: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 25 lines checked 92c356444b7b drm: use new iterator in drm_gem_fence_array_add_implicit v2 -:63: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 44 lines checked ee7a40d3c32d drm: use new iterator in drm_gem_plane_helper_prepare_fb v2 -:46: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 25 lines checked cf5d087e77ef drm/nouveau: use the new iterator in nouveau_fence_sync -:96: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 77 lines checked aa6d699f8f59 drm/nouveau: use the new interator in nv50_wndw_prepare_fb v2 -:44: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 23 lines checked 10b96c40fe4f drm/etnaviv: use new iterator in etnaviv_gem_describe -:65: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 47 lines checked fc9b5ce671ec drm/etnaviv: replace dma_resv_get_excl_unlocked -:25: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 8 lines checked a648c89556c2 dma-buf: nuke dma_resv_get_excl_unlocked -:49: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" <ckoenig.leichtzumerken@xxxxxxxxx>' != 'Signed-off-by: Christian König <christian.koenig@xxxxxxx>' total: 0 errors, 1 warnings, 0 checks, 32 lines checked
