From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 0062543D55 for ; Wed, 27 Mar 2024 23:41:32 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6489D42DB6; Wed, 27 Mar 2024 23:41:31 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id B851342831 for ; Wed, 27 Mar 2024 23:40:22 +0100 (CET) Received: by dpdk.org (Postfix, from userid 65534) id B41F71223FA; Wed, 27 Mar 2024 23:40:22 +0100 (CET) Subject: |WARNING| pw138878 [PATCH v3 01/45] net/mlx5: use rte stdatomic API In-Reply-To: <1711579078-10624-2-git-send-email-roretzla@linux.microsoft.com> References: <1711579078-10624-2-git-send-email-roretzla@linux.microsoft.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Tyler Retzlaff Message-Id: <20240327224022.B41F71223FA@dpdk.org> Date: Wed, 27 Mar 2024 23:40:22 +0100 (CET) X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/138878 _coding style issues_ CHECK:SPACING: No space is necessary after a cast #222: FILE: drivers/net/mlx5/mlx5.h:381: + RTE_ATOMIC(uint16_t) refcnt; /* Reference count for representors. */ CHECK:SPACING: No space is necessary after a cast #231: FILE: drivers/net/mlx5/mlx5.h:484: + RTE_ATOMIC(uint16_t) state; /**< Age state (atomically accessed). */ CHECK:SPACING: No space is necessary after a cast #235: FILE: drivers/net/mlx5/mlx5.h:487: + RTE_ATOMIC(uint32_t) sec_since_last_hit; CHECK:SPACING: No space is necessary after a cast #244: FILE: drivers/net/mlx5/mlx5.h:500: + RTE_ATOMIC(uint32_t) refcnt; /* Only for shared action management. */ CHECK:SPACING: No space is necessary after a cast #253: FILE: drivers/net/mlx5/mlx5.h:591: + volatile RTE_ATOMIC(uint16_t) n_valid; /* Number of valid pools. */ CHECK:SPACING: No space is necessary after a cast #262: FILE: drivers/net/mlx5/mlx5.h:657: + RTE_ATOMIC(uint32_t) refcnt; CHECK:SPACING: No space is necessary after a cast #271: FILE: drivers/net/mlx5/mlx5.h:691: + RTE_ATOMIC(uint32_t) refcnt; /* geneve tlv object reference counter */ CHECK:SPACING: No space is necessary after a cast #280: FILE: drivers/net/mlx5/mlx5.h:906: + RTE_ATOMIC(uint32_t) ref_cnt; CHECK:SPACING: No space is necessary after a cast #289: FILE: drivers/net/mlx5/mlx5.h:1041: + RTE_ATOMIC(uint32_t) ref_cnt; /**< Use count. */ CHECK:SPACING: No space is necessary after a cast #298: FILE: drivers/net/mlx5/mlx5.h:1081: + RTE_ATOMIC(uint8_t) state; /**< ASO flow meter state. */ CHECK:SPACING: No space is necessary after a cast #307: FILE: drivers/net/mlx5/mlx5.h:1127: + RTE_ATOMIC(uint32_t) def_policy_ref_cnt; CHECK:SPACING: No space is necessary after a cast #317: FILE: drivers/net/mlx5/mlx5.h:1200: + RTE_ATOMIC(uint64_t) ci_ts; CHECK:SPACING: No space is necessary after a cast #318: FILE: drivers/net/mlx5/mlx5.h:1201: + RTE_ATOMIC(uint64_t) ts; CHECK:SPACING: No space is necessary after a cast #332: FILE: drivers/net/mlx5/mlx5.h:1224: + RTE_ATOMIC(uint64_t) err_miss_int; /* Missed service interrupt. */ CHECK:SPACING: No space is necessary after a cast #333: FILE: drivers/net/mlx5/mlx5.h:1225: + RTE_ATOMIC(uint64_t) err_rearm_queue; /* Rearm Queue errors. */ CHECK:SPACING: No space is necessary after a cast #334: FILE: drivers/net/mlx5/mlx5.h:1226: + RTE_ATOMIC(uint64_t) err_clock_queue; /* Clock Queue errors. */ CHECK:SPACING: No space is necessary after a cast #335: FILE: drivers/net/mlx5/mlx5.h:1227: + RTE_ATOMIC(uint64_t) err_ts_past; /* Timestamp in the past. */ CHECK:SPACING: No space is necessary after a cast #336: FILE: drivers/net/mlx5/mlx5.h:1228: + RTE_ATOMIC(uint64_t) err_ts_future; /* Timestamp in the distant future. */ CHECK:SPACING: No space is necessary after a cast #337: FILE: drivers/net/mlx5/mlx5.h:1229: + RTE_ATOMIC(uint64_t) err_ts_order; /* Timestamp not in ascending order. */ CHECK:SPACING: No space is necessary after a cast #346: FILE: drivers/net/mlx5/mlx5.h:1290: + RTE_ATOMIC(uint32_t) refcnt; /* Action used count in device flows. */ CHECK:SPACING: No space is necessary after a cast #350: FILE: drivers/net/mlx5/mlx5.h:1293: + RTE_ATOMIC(enum mlx5_aso_ct_state) state; /* ASO CT state. */ CHECK:SPACING: No space is necessary after a cast #366: FILE: drivers/net/mlx5/mlx5.h:1373: + RTE_ATOMIC(uint32_t) refcnt; /* Atomically accessed refcnt by flows. */ CHECK:SPACING: No space is necessary after a cast #375: FILE: drivers/net/mlx5/mlx5.h:1386: + RTE_ATOMIC(uint32_t) refcnt; CHECK:SPACING: No space is necessary after a cast #386: FILE: drivers/net/mlx5/mlx5.h:1515: + RTE_ATOMIC(struct mlx5_hlist *) encaps_decaps; /* Encap/decap action hash list. */ CHECK:SPACING: No space is necessary after a cast #387: FILE: drivers/net/mlx5/mlx5.h:1516: + RTE_ATOMIC(struct mlx5_hlist *) modify_cmds; CHECK:SPACING: No space is necessary after a cast #388: FILE: drivers/net/mlx5/mlx5.h:1517: + RTE_ATOMIC(struct mlx5_hlist *) tag_table; CHECK:SPACING: No space is necessary after a cast #397: FILE: drivers/net/mlx5/mlx5.h:1528: + RTE_ATOMIC(struct mlx5_indexed_pool *) mdh_ipools[MLX5_MAX_MODIFY_NUM]; CHECK:SPACING: No space is necessary after a cast #406: FILE: drivers/net/mlx5/mlx5.h:1573: + RTE_ATOMIC(void *) hca_bar; CHECK:SPACING: No space is necessary after a cast #415: FILE: drivers/net/mlx5/mlx5.h:1638: + RTE_ATOMIC(uint32_t) refcnt; /* Reference counter. */ CHECK:SPACING: No space is necessary after a cast #424: FILE: drivers/net/mlx5/mlx5.h:1829: + RTE_ATOMIC(uint8_t) state; /* object state */ CHECK:SPACING: No space is necessary after a cast #433: FILE: drivers/net/mlx5/mlx5.h:1958: + RTE_ATOMIC(uint32_t) hws_mark_refcnt; /* HWS mark action reference counter. */ CHECK:SPACING: No space is necessary after a cast #442: FILE: drivers/net/mlx5/mlx5.h:2010: + RTE_ATOMIC(uint16_t) shared_refcnt; /* HW steering host reference counter. */ CHECK:SPACING: No space is necessary after a cast #559: FILE: drivers/net/mlx5/mlx5_flow.h:1052: + RTE_ATOMIC(uint32_t) refctn; CHECK:SPACING: No space is necessary after a cast #568: FILE: drivers/net/mlx5/mlx5_flow.h:1473: + RTE_ATOMIC(uint32_t) refcnt; /* Reference counter. */ CHECK:SPACING: No space is necessary after a cast #577: FILE: drivers/net/mlx5/mlx5_flow.h:1505: + RTE_ATOMIC(uint32_t) refcnt; /* Reference counter. */ CHECK:SPACING: No space is necessary after a cast #586: FILE: drivers/net/mlx5/mlx5_flow.h:1858: + RTE_ATOMIC(uint32_t) refcnt; /**< Atomically accessed refcnt. */ CHECK:SPACING: No space is necessary after a cast #734: FILE: drivers/net/mlx5/mlx5_flow_dv.c:316: +flow_dv_hlist_prepare(struct mlx5_dev_ctx_shared *sh, RTE_ATOMIC(struct mlx5_hlist *) *phl, CHECK:SPACING: spaces preferred around that '*' (ctx:WxV) #734: FILE: drivers/net/mlx5/mlx5_flow_dv.c:316: +flow_dv_hlist_prepare(struct mlx5_dev_ctx_shared *sh, RTE_ATOMIC(struct mlx5_hlist *) *phl, ^ CHECK:SPACING: No space is necessary after a cast #1679: FILE: drivers/net/mlx5/mlx5_hws_cnt.h:104: + RTE_ATOMIC(uint32_t) query_gen __rte_cache_aligned; CHECK:SPACING: No space is necessary after a cast #1689: FILE: drivers/net/mlx5/mlx5_hws_cnt.h:137: + RTE_ATOMIC(uint32_t) timeout; /* Aging timeout in seconds (atomically accessed). */ CHECK:SPACING: No space is necessary after a cast #1690: FILE: drivers/net/mlx5/mlx5_hws_cnt.h:138: + RTE_ATOMIC(uint32_t) sec_since_last_hit; CHECK:SPACING: No space is necessary after a cast #1693: FILE: drivers/net/mlx5/mlx5_hws_cnt.h:140: + RTE_ATOMIC(uint16_t) state; /* AGE state (atomically accessed). */ CHECK:SPACING: No space is necessary after a cast #1715: FILE: drivers/net/mlx5/mlx5_rx.h:176: + RTE_ATOMIC(uint32_t) refcnt; /* Reference counter. */ CHECK:SPACING: No space is necessary after a cast #1724: FILE: drivers/net/mlx5/mlx5_rx.h:191: + RTE_ATOMIC(uint32_t) refcnt; /* Reference counter. */ CHECK:SPACING: No space is necessary after a cast #1899: FILE: drivers/net/mlx5/mlx5_tx.h:182: + RTE_ATOMIC(uint32_t) refcnt; /* Reference counter. */ CHECK:SPACING: No space is necessary after a cast #2221: FILE: drivers/net/mlx5/mlx5_utils.h:243: + volatile RTE_ATOMIC(uint32_t) n_trunk_valid; /* Trunks allocated. */ CHECK:SPACING: No space is necessary after a cast #2230: FILE: drivers/net/mlx5/mlx5_utils.h:269: + RTE_ATOMIC(struct mlx5_indexed_cache *) gc; total: 0 errors, 0 warnings, 47 checks, 1840 lines checked Please use __rte_cache_aligned only for struct or union types alignment.