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 7726543EB4 for ; Sat, 20 Apr 2024 01:09:22 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B19E642DEB; Sat, 20 Apr 2024 01:09:13 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 19B1540A70 for ; Sat, 20 Apr 2024 01:09:09 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 0C258124858; Sat, 20 Apr 2024 01:09:08 +0200 (CEST) Subject: |WARNING| pw139561 [PATCH v4 01/45] net/mlx5: use rte stdatomic API In-Reply-To: <1713568003-30453-2-git-send-email-roretzla@linux.microsoft.com> References: <1713568003-30453-2-git-send-email-roretzla@linux.microsoft.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Tyler Retzlaff Message-Id: <20240419230909.0C258124858@dpdk.org> Date: Sat, 20 Apr 2024 01:09:08 +0200 (CEST) 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/139561 _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: + alignas(RTE_CACHE_LINE_SIZE) RTE_ATOMIC(uint32_t) query_gen; 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