From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.7 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CB6A4C433F5 for ; Fri, 17 Sep 2021 14:01:44 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 821EF60E08 for ; Fri, 17 Sep 2021 14:01:44 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 821EF60E08 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=emeril.freedesktop.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 1F15D6EC82; Fri, 17 Sep 2021 14:01:44 +0000 (UTC) Received: from emeril.freedesktop.org (emeril.freedesktop.org [131.252.210.167]) by gabe.freedesktop.org (Postfix) with ESMTP id 13AEF6EC82; Fri, 17 Sep 2021 14:01:42 +0000 (UTC) Received: from emeril.freedesktop.org (localhost [127.0.0.1]) by emeril.freedesktop.org (Postfix) with ESMTP id 08BABA01BB; Fri, 17 Sep 2021 14:01:42 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Patchwork To: =?utf-8?q?Christian_K=C3=B6nig?= Cc: intel-gfx@lists.freedesktop.org Date: Fri, 17 Sep 2021 14:01:42 -0000 Message-ID: <163188730203.808.5975023716700859071@emeril.freedesktop.org> X-Patchwork-Hint: ignore References: <20210917123513.1106-1-christian.koenig@amd.com> In-Reply-To: <20210917123513.1106-1-christian.koenig@amd.com> Subject: [Intel-gfx] =?utf-8?b?4pyXIEZpLkNJLkNIRUNLUEFUQ0g6IHdhcm5pbmcg?= =?utf-8?q?for_series_starting_with_=5B01/26=5D_dma-buf=3A_add_dma=5Fresv?= =?utf-8?q?=5Ffor=5Feach=5Ffence=5Funlocked_v2?= X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: intel-gfx@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" == Series Details == Series: series starting with [01/26] dma-buf: add dma_resv_for_each_fence_unlocked v2 URL : https://patchwork.freedesktop.org/series/94805/ State : warning == Summary == $ dim checkpatch origin/drm-tip d9db5bae923c dma-buf: add dma_resv_for_each_fence_unlocked v2 -:70: CHECK:OPEN_ENDED_LINE: Lines should not end with a '(' #70: FILE: drivers/dma-buf/dma-resv.c:366: + cursor->fence = rcu_dereference( -:140: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #140: FILE: include/linux/dma-resv.h:190: +static inline void dma_resv_iter_begin(struct dma_resv_iter *cursor, + struct dma_resv *obj, -:182: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cursor' - possible side-effects? #182: FILE: include/linux/dma-resv.h:232: +#define dma_resv_for_each_fence_unlocked(cursor, fence) \ + for (fence = dma_resv_iter_walk_unlocked(cursor, true); \ + fence; fence = dma_resv_iter_walk_unlocked(cursor, false)) -:182: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fence' - possible side-effects? #182: FILE: include/linux/dma-resv.h:232: +#define dma_resv_for_each_fence_unlocked(cursor, fence) \ + for (fence = dma_resv_iter_walk_unlocked(cursor, true); \ + fence; fence = dma_resv_iter_walk_unlocked(cursor, false)) -:188: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 4 checks, 157 lines checked a903605ee308 dma-buf: add dma_resv_for_each_fence -:85: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cursor' - possible side-effects? #85: FILE: include/linux/dma-resv.h:248: +#define dma_resv_for_each_fence(cursor, obj, all_fences, fence) \ + for (dma_resv_iter_begin(cursor, obj, all_fences), \ + fence = dma_resv_iter_walk(cursor, true); fence; \ + fence = dma_resv_iter_walk(cursor, false)) -:85: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fence' - possible side-effects? #85: FILE: include/linux/dma-resv.h:248: +#define dma_resv_for_each_fence(cursor, obj, all_fences, fence) \ + for (dma_resv_iter_begin(cursor, obj, all_fences), \ + fence = dma_resv_iter_walk(cursor, true); fence; \ + fence = dma_resv_iter_walk(cursor, false)) -:92: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 2 checks, 68 lines checked 61845118125a dma-buf: use new iterator in dma_resv_copy_fences -:127: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 109 lines checked 3d471d53ec2b dma-buf: use new iterator in dma_resv_get_fences v2 -:159: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 139 lines checked 600bac37a0f1 dma-buf: use new iterator in dma_resv_wait_timeout -:102: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 84 lines checked b93771325f9f dma-buf: use new iterator in dma_resv_test_signaled -:93: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 74 lines checked 05ffba354ecc drm/ttm: use the new iterator in ttm_bo_flush_all_fences -:44: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 26 lines checked bbdc9793743a 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" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 55 lines checked 511232e4b5ca 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" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 28 lines checked 58e24849df68 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" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 34 lines checked ab00e34c795a 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" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 36 lines checked 749c88a44413 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" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 36 lines checked aa613855df42 drm/i915: use the new iterator in i915_gem_busy_ioctl -:65: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 46 lines checked 5fac734e6483 drm/i915: use the new iterator in i915_sw_fence_await_reservation v3 -:93: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 71 lines checked 59ae4d4e8254 drm/i915: use the new iterator in i915_request_await_object v2 -:64: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 43 lines checked c7c263d3cc20 drm/i915: use new iterator in i915_gem_object_wait_reservation v2 -:89: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 69 lines checked a2269c6c31d4 drm/i915: use new iterator in i915_gem_object_wait_priority v2 -:62: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 42 lines checked 1b7252cd01a4 drm/i915: use new iterator in i915_gem_object_last_write_engine v2 -:44: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 24 lines checked cd25bcfa7b95 drm/i915: use new cursor in intel_prepare_plane_fb v2 -:46: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 26 lines checked 82c3476a212c drm: use new iterator in drm_gem_fence_array_add_implicit v2 -:62: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 43 lines checked 8ad97e523761 drm: use new iterator in drm_gem_plane_helper_prepare_fb v2 -:48: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 27 lines checked a6220344b108 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" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 77 lines checked 2be87d3cd2c8 drm/nouveau: use the new interator in nv50_wndw_prepare_fb v2 -:45: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 24 lines checked 6851abceef53 drm/etnaviv: use new iterator in etnaviv_gem_describe -:67: WARNING:FROM_SIGN_OFF_MISMATCH: From:/Signed-off-by: email address mismatch: 'From: "Christian König" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 49 lines checked f7eb003ad813 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" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 8 lines checked 3ed2dc6e4f12 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" ' != 'Signed-off-by: Christian König ' total: 0 errors, 1 warnings, 0 checks, 32 lines checked