From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <test-report-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 2213245778
	for <public@inbox.dpdk.org>; Fri,  9 Aug 2024 12:09:14 +0200 (CEST)
Received: from mails.dpdk.org (localhost [127.0.0.1])
	by mails.dpdk.org (Postfix) with ESMTP id 1D7BF42EE4;
	Fri,  9 Aug 2024 12:09:14 +0200 (CEST)
Received: from dpdk.org (dpdk.org [92.243.24.197])
 by mails.dpdk.org (Postfix) with ESMTP id 297A842D78
 for <test-report@dpdk.org>; Fri,  9 Aug 2024 12:09:11 +0200 (CEST)
Received: by dpdk.org (Postfix, from userid 65534)
 id 14F2F123EF6; Fri,  9 Aug 2024 12:09:11 +0200 (CEST)
Subject: |WARNING| pw143048 [PATCH v2 5/5] eal: extend bitops to handle
 volatile pointers
In-Reply-To: <20240809095829.589396-6-mattias.ronnblom@ericsson.com>
References: <20240809095829.589396-6-mattias.ronnblom@ericsson.com>
To: test-report@dpdk.org
From: checkpatch@dpdk.org
Cc: =?utf-8?q?Mattias_R=C3=B6nnblom?= <mattias.ronnblom@ericsson.com>
Message-Id: <20240809100911.14F2F123EF6@dpdk.org>
Date: Fri,  9 Aug 2024 12:09:11 +0200 (CEST)
X-BeenThere: test-report@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: automatic DPDK test reports <test-report.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/test-report>,
 <mailto:test-report-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/test-report/>
List-Post: <mailto:test-report@dpdk.org>
List-Help: <mailto:test-report-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/test-report>,
 <mailto:test-report-request@dpdk.org?subject=subscribe>
Errors-To: test-report-bounces@dpdk.org

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/143048

_coding style issues_


ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#179: FILE: app/test/test_bitops.c:44:
+			const mod uint ## size ## _t *const_ptr = &word; \
 			                             ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#188: FILE: app/test/test_bitops.c:62:
+		    rte_bit_assign, rte_bit_flip, rte_bit_test, 32,)
 		                                                  ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#192: FILE: app/test/test_bitops.c:65:
+		    rte_bit_assign, rte_bit_flip, rte_bit_test, 64,)
 		                                                  ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#207: FILE: app/test/test_bitops.c:90:
+		    bit_atomic_flip, bit_atomic_test, 32,)
 		                                        ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#212: FILE: app/test/test_bitops.c:94:
+		    bit_atomic_flip, bit_atomic_test, 64,)
 		                                        ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#253: FILE: lib/eal/include/rte_bitops.h:132:
+		 uint32_t *: __rte_bit_test32,				\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#253: FILE: lib/eal/include/rte_bitops.h:132:
+		 uint32_t *: __rte_bit_test32,				\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#254: FILE: lib/eal/include/rte_bitops.h:133:
+		 const uint32_t *: __rte_bit_test32,			\
 		                ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#254: FILE: lib/eal/include/rte_bitops.h:133:
+		 const uint32_t *: __rte_bit_test32,			\
 		                 ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#255: FILE: lib/eal/include/rte_bitops.h:134:
+		 volatile uint32_t *: __rte_bit_v_test32,		\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#255: FILE: lib/eal/include/rte_bitops.h:134:
+		 volatile uint32_t *: __rte_bit_v_test32,		\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#256: FILE: lib/eal/include/rte_bitops.h:135:
+		 const volatile uint32_t *: __rte_bit_v_test32,		\
 		                         ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#256: FILE: lib/eal/include/rte_bitops.h:135:
+		 const volatile uint32_t *: __rte_bit_v_test32,		\
 		                          ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#257: FILE: lib/eal/include/rte_bitops.h:136:
+		 uint64_t *: __rte_bit_test64,				\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#257: FILE: lib/eal/include/rte_bitops.h:136:
+		 uint64_t *: __rte_bit_test64,				\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#258: FILE: lib/eal/include/rte_bitops.h:137:
+		 const uint64_t *: __rte_bit_test64,			\
 		                ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#258: FILE: lib/eal/include/rte_bitops.h:137:
+		 const uint64_t *: __rte_bit_test64,			\
 		                 ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#259: FILE: lib/eal/include/rte_bitops.h:138:
+		 volatile uint64_t *: __rte_bit_v_test64,		\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#259: FILE: lib/eal/include/rte_bitops.h:138:
+		 volatile uint64_t *: __rte_bit_v_test64,		\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#260: FILE: lib/eal/include/rte_bitops.h:139:
+		 const volatile uint64_t *: __rte_bit_v_test64)(addr, nr)
 		                         ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#260: FILE: lib/eal/include/rte_bitops.h:139:
+		 const volatile uint64_t *: __rte_bit_v_test64)(addr, nr)
 		                          ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#274: FILE: lib/eal/include/rte_bitops.h:161:
+		 uint32_t *: __rte_bit_set32,				\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#274: FILE: lib/eal/include/rte_bitops.h:161:
+		 uint32_t *: __rte_bit_set32,				\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#275: FILE: lib/eal/include/rte_bitops.h:162:
+		 volatile uint32_t *: __rte_bit_v_set32,		\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#275: FILE: lib/eal/include/rte_bitops.h:162:
+		 volatile uint32_t *: __rte_bit_v_set32,		\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#276: FILE: lib/eal/include/rte_bitops.h:163:
+		 uint64_t *: __rte_bit_set64,				\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#276: FILE: lib/eal/include/rte_bitops.h:163:
+		 uint64_t *: __rte_bit_set64,				\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#277: FILE: lib/eal/include/rte_bitops.h:164:
+		 volatile uint64_t *: __rte_bit_v_set64)(addr, nr)
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#277: FILE: lib/eal/include/rte_bitops.h:164:
+		 volatile uint64_t *: __rte_bit_v_set64)(addr, nr)
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#291: FILE: lib/eal/include/rte_bitops.h:186:
+		 uint32_t *: __rte_bit_clear32,				\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#291: FILE: lib/eal/include/rte_bitops.h:186:
+		 uint32_t *: __rte_bit_clear32,				\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#292: FILE: lib/eal/include/rte_bitops.h:187:
+		 volatile uint32_t *: __rte_bit_v_clear32,		\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#292: FILE: lib/eal/include/rte_bitops.h:187:
+		 volatile uint32_t *: __rte_bit_v_clear32,		\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#293: FILE: lib/eal/include/rte_bitops.h:188:
+		 uint64_t *: __rte_bit_clear64,				\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#293: FILE: lib/eal/include/rte_bitops.h:188:
+		 uint64_t *: __rte_bit_clear64,				\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#294: FILE: lib/eal/include/rte_bitops.h:189:
+		 volatile uint64_t *: __rte_bit_v_clear64)(addr, nr)
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#294: FILE: lib/eal/include/rte_bitops.h:189:
+		 volatile uint64_t *: __rte_bit_v_clear64)(addr, nr)
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#303: FILE: lib/eal/include/rte_bitops.h:213:
+		 volatile uint32_t *: __rte_bit_v_assign32,		\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#303: FILE: lib/eal/include/rte_bitops.h:213:
+		 volatile uint32_t *: __rte_bit_v_assign32,		\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#304: FILE: lib/eal/include/rte_bitops.h:214:
+		 uint64_t *: __rte_bit_assign64,			\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#304: FILE: lib/eal/include/rte_bitops.h:214:
+		 uint64_t *: __rte_bit_assign64,			\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#305: FILE: lib/eal/include/rte_bitops.h:215:
+		 volatile uint64_t *: __rte_bit_v_assign64)(addr, nr, value)
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#305: FILE: lib/eal/include/rte_bitops.h:215:
+		 volatile uint64_t *: __rte_bit_v_assign64)(addr, nr, value)
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#314: FILE: lib/eal/include/rte_bitops.h:238:
+		 volatile uint32_t *: __rte_bit_v_flip32,		\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#314: FILE: lib/eal/include/rte_bitops.h:238:
+		 volatile uint32_t *: __rte_bit_v_flip32,		\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#315: FILE: lib/eal/include/rte_bitops.h:239:
+		 uint64_t *: __rte_bit_flip64,				\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#315: FILE: lib/eal/include/rte_bitops.h:239:
+		 uint64_t *: __rte_bit_flip64,				\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#316: FILE: lib/eal/include/rte_bitops.h:240:
+		 volatile uint64_t *: __rte_bit_v_flip64)(addr, nr)
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#316: FILE: lib/eal/include/rte_bitops.h:240:
+		 volatile uint64_t *: __rte_bit_v_flip64)(addr, nr)
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#324: FILE: lib/eal/include/rte_bitops.h:265:
+		 volatile uint32_t *: __rte_bit_atomic_v_test32,	\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#324: FILE: lib/eal/include/rte_bitops.h:265:
+		 volatile uint32_t *: __rte_bit_atomic_v_test32,	\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#325: FILE: lib/eal/include/rte_bitops.h:266:
+		 const volatile uint32_t *: __rte_bit_atomic_v_test32,	\
 		                         ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#325: FILE: lib/eal/include/rte_bitops.h:266:
+		 const volatile uint32_t *: __rte_bit_atomic_v_test32,	\
 		                          ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#329: FILE: lib/eal/include/rte_bitops.h:268:
+		 const uint64_t *: __rte_bit_atomic_test64,		\
 		                ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#329: FILE: lib/eal/include/rte_bitops.h:268:
+		 const uint64_t *: __rte_bit_atomic_test64,		\
 		                 ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#330: FILE: lib/eal/include/rte_bitops.h:269:
+		 volatile uint64_t *: __rte_bit_atomic_v_test64,	\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#330: FILE: lib/eal/include/rte_bitops.h:269:
+		 volatile uint64_t *: __rte_bit_atomic_v_test64,	\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#331: FILE: lib/eal/include/rte_bitops.h:270:
+		 const volatile uint64_t *: __rte_bit_atomic_v_test64) \
 		                         ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#331: FILE: lib/eal/include/rte_bitops.h:270:
+		 const volatile uint64_t *: __rte_bit_atomic_v_test64) \
 		                          ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#341: FILE: lib/eal/include/rte_bitops.h:293:
+		 volatile uint32_t *: __rte_bit_atomic_v_set32,		\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#341: FILE: lib/eal/include/rte_bitops.h:293:
+		 volatile uint32_t *: __rte_bit_atomic_v_set32,		\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#342: FILE: lib/eal/include/rte_bitops.h:294:
+		 uint64_t *: __rte_bit_atomic_set64,			\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#342: FILE: lib/eal/include/rte_bitops.h:294:
+		 uint64_t *: __rte_bit_atomic_set64,			\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#343: FILE: lib/eal/include/rte_bitops.h:295:
+		 volatile uint64_t *: __rte_bit_atomic_v_set64)(addr, nr, \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#343: FILE: lib/eal/include/rte_bitops.h:295:
+		 volatile uint64_t *: __rte_bit_atomic_v_set64)(addr, nr, \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#353: FILE: lib/eal/include/rte_bitops.h:318:
+		 volatile uint32_t *: __rte_bit_atomic_v_clear32,	\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#353: FILE: lib/eal/include/rte_bitops.h:318:
+		 volatile uint32_t *: __rte_bit_atomic_v_clear32,	\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#354: FILE: lib/eal/include/rte_bitops.h:319:
+		 uint64_t *: __rte_bit_atomic_clear64,			\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#354: FILE: lib/eal/include/rte_bitops.h:319:
+		 uint64_t *: __rte_bit_atomic_clear64,			\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#355: FILE: lib/eal/include/rte_bitops.h:320:
+		 volatile uint64_t *: __rte_bit_atomic_v_clear64)(addr, nr, \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#355: FILE: lib/eal/include/rte_bitops.h:320:
+		 volatile uint64_t *: __rte_bit_atomic_v_clear64)(addr, nr, \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#366: FILE: lib/eal/include/rte_bitops.h:345:
+		 volatile uint32_t *: __rte_bit_atomic_v_assign32,	\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#366: FILE: lib/eal/include/rte_bitops.h:345:
+		 volatile uint32_t *: __rte_bit_atomic_v_assign32,	\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#367: FILE: lib/eal/include/rte_bitops.h:346:
+		 uint64_t *: __rte_bit_atomic_assign64,			\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#367: FILE: lib/eal/include/rte_bitops.h:346:
+		 uint64_t *: __rte_bit_atomic_assign64,			\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#368: FILE: lib/eal/include/rte_bitops.h:347:
+		 volatile uint64_t *: __rte_bit_atomic_v_assign64)(addr, nr, \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#368: FILE: lib/eal/include/rte_bitops.h:347:
+		 volatile uint64_t *: __rte_bit_atomic_v_assign64)(addr, nr, \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#379: FILE: lib/eal/include/rte_bitops.h:372:
+		 volatile uint32_t *: __rte_bit_atomic_v_flip32,	\
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#379: FILE: lib/eal/include/rte_bitops.h:372:
+		 volatile uint32_t *: __rte_bit_atomic_v_flip32,	\
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#380: FILE: lib/eal/include/rte_bitops.h:373:
+		 uint64_t *: __rte_bit_atomic_flip64,			\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#380: FILE: lib/eal/include/rte_bitops.h:373:
+		 uint64_t *: __rte_bit_atomic_flip64,			\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#381: FILE: lib/eal/include/rte_bitops.h:374:
+		 volatile uint64_t *: __rte_bit_atomic_v_flip64)(addr, nr, \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#381: FILE: lib/eal/include/rte_bitops.h:374:
+		 volatile uint64_t *: __rte_bit_atomic_v_flip64)(addr, nr, \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#392: FILE: lib/eal/include/rte_bitops.h:399:
+		 volatile uint32_t *: __rte_bit_atomic_v_test_and_set32, \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#392: FILE: lib/eal/include/rte_bitops.h:399:
+		 volatile uint32_t *: __rte_bit_atomic_v_test_and_set32, \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#393: FILE: lib/eal/include/rte_bitops.h:400:
+		 uint64_t *: __rte_bit_atomic_test_and_set64,		\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#393: FILE: lib/eal/include/rte_bitops.h:400:
+		 uint64_t *: __rte_bit_atomic_test_and_set64,		\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#394: FILE: lib/eal/include/rte_bitops.h:401:
+		 volatile uint64_t *: __rte_bit_atomic_v_test_and_set64) \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#394: FILE: lib/eal/include/rte_bitops.h:401:
+		 volatile uint64_t *: __rte_bit_atomic_v_test_and_set64) \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#405: FILE: lib/eal/include/rte_bitops.h:426:
+		 volatile uint32_t *: __rte_bit_atomic_v_test_and_clear32, \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#405: FILE: lib/eal/include/rte_bitops.h:426:
+		 volatile uint32_t *: __rte_bit_atomic_v_test_and_clear32, \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#406: FILE: lib/eal/include/rte_bitops.h:427:
+		 uint64_t *: __rte_bit_atomic_test_and_clear64,		\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#406: FILE: lib/eal/include/rte_bitops.h:427:
+		 uint64_t *: __rte_bit_atomic_test_and_clear64,		\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#407: FILE: lib/eal/include/rte_bitops.h:428:
+		 volatile uint64_t *: __rte_bit_atomic_v_test_and_clear64) \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#407: FILE: lib/eal/include/rte_bitops.h:428:
+		 volatile uint64_t *: __rte_bit_atomic_v_test_and_clear64) \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#419: FILE: lib/eal/include/rte_bitops.h:456:
+		 volatile uint32_t *: __rte_bit_atomic_v_test_and_assign32, \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#419: FILE: lib/eal/include/rte_bitops.h:456:
+		 volatile uint32_t *: __rte_bit_atomic_v_test_and_assign32, \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#420: FILE: lib/eal/include/rte_bitops.h:457:
+		 uint64_t *: __rte_bit_atomic_test_and_assign64,	\
 		          ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#420: FILE: lib/eal/include/rte_bitops.h:457:
+		 uint64_t *: __rte_bit_atomic_test_and_assign64,	\
 		           ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#421: FILE: lib/eal/include/rte_bitops.h:458:
+		 volatile uint64_t *: __rte_bit_atomic_v_test_and_assign64) \
 		                   ^

ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#421: FILE: lib/eal/include/rte_bitops.h:458:
+		 volatile uint64_t *: __rte_bit_atomic_v_test_and_assign64) \
 		                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#454: FILE: lib/eal/include/rte_bitops.h:548:
+	__rte_bit_atomic_ ## v ## test ## size(const qualifier uint ## size ## _t *addr, \
 	                                                                          ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#461: FILE: lib/eal/include/rte_bitops.h:553:
+		const qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr = \
 		                                               ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#473: FILE: lib/eal/include/rte_bitops.h:562:
+	__rte_bit_atomic_ ## v ## set ## size(qualifier uint ## size ## _t *addr, \
 	                                                                   ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#480: FILE: lib/eal/include/rte_bitops.h:567:
+		qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr =	\
 		                                         ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#492: FILE: lib/eal/include/rte_bitops.h:576:
+	__rte_bit_atomic_ ## v ## clear ## size(qualifier uint ## size ## _t *addr,	\
 	                                                                     ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#499: FILE: lib/eal/include/rte_bitops.h:581:
+		qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr =	\
 		                                         ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#511: FILE: lib/eal/include/rte_bitops.h:590:
+	__rte_bit_atomic_ ## v ## flip ## size(qualifier uint ## size ## _t *addr, \
 	                                                                    ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#518: FILE: lib/eal/include/rte_bitops.h:595:
+		qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr =	\
 		                                         ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#531: FILE: lib/eal/include/rte_bitops.h:604:
+	__rte_bit_atomic_## v ## assign ## size(qualifier uint ## size ## _t *addr, \
 	                                                                     ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#552: FILE: lib/eal/include/rte_bitops.h:618:
+	__rte_bit_atomic_ ## v ## test_and_set ## size(qualifier uint ## size ## _t *addr, \
 	                                                                            ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#560: FILE: lib/eal/include/rte_bitops.h:624:
+		qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr =	\
 		                                         ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#576: FILE: lib/eal/include/rte_bitops.h:638:
+	__rte_bit_atomic_ ## v ## test_and_clear ## size(qualifier uint ## size ## _t *addr, \
 	                                                                              ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#584: FILE: lib/eal/include/rte_bitops.h:644:
+		qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr =	\
 		                                         ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#601: FILE: lib/eal/include/rte_bitops.h:658:
+	__rte_bit_atomic_ ## v ## test_and_assign ## size(qualifier uint ## size ## _t *addr, \
 	                                                                               ^

WARNING:LONG_LINE: line length of 104 exceeds 100 columns
#609: FILE: lib/eal/include/rte_bitops.h:664:
+			return __rte_bit_atomic_ ## v ## test_and_set ## size(addr, nr, memory_order); \

WARNING:LONG_LINE: line length of 106 exceeds 100 columns
#613: FILE: lib/eal/include/rte_bitops.h:666:
+			return __rte_bit_atomic_ ## v ## test_and_clear ## size(addr, nr, memory_order); \

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#625: FILE: lib/eal/include/rte_bitops.h:669:
+#define __RTE_GEN_BIT_ATOMIC_OPS(v, qualifier, size)	\
+	__RTE_GEN_BIT_ATOMIC_TEST(v, qualifier, size)	\
+	__RTE_GEN_BIT_ATOMIC_SET(v, qualifier, size)	\
+	__RTE_GEN_BIT_ATOMIC_CLEAR(v, qualifier, size)	\
+	__RTE_GEN_BIT_ATOMIC_ASSIGN(v, qualifier, size)	\
+	__RTE_GEN_BIT_ATOMIC_TEST_AND_SET(v, qualifier, size) \
+	__RTE_GEN_BIT_ATOMIC_TEST_AND_CLEAR(v, qualifier, size) \
+	__RTE_GEN_BIT_ATOMIC_TEST_AND_ASSIGN(v, qualifier, size) \
+	__RTE_GEN_BIT_ATOMIC_FLIP(v, qualifier, size)

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#637: FILE: lib/eal/include/rte_bitops.h:679:
+#define __RTE_GEN_BIT_ATOMIC_OPS_SIZE(size) \
+	__RTE_GEN_BIT_ATOMIC_OPS(,, size) \
+	__RTE_GEN_BIT_ATOMIC_OPS(v_, volatile, size)

ERROR:SPACING: space required after that ',' (ctx:BxO)
#638: FILE: lib/eal/include/rte_bitops.h:680:
+	__RTE_GEN_BIT_ATOMIC_OPS(,, size) \
 	                         ^

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#655: FILE: lib/eal/include/rte_bitops.h:1392:
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
 	                                               ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#665: FILE: lib/eal/include/rte_bitops.h:1398:
+#define __RTE_BIT_OVERLOAD_SZ_2(family, fun, c, size, arg1_type, arg1_name) \
+	__RTE_BIT_OVERLOAD_V_2(family,, fun, c, size, arg1_type,	\
+			       arg1_name)				\
+	__RTE_BIT_OVERLOAD_V_2(family, v_, fun, c volatile, size, \
+			       arg1_type, arg1_name)

ERROR:SPACING: space required after that ',' (ctx:VxO)
#666: FILE: lib/eal/include/rte_bitops.h:1399:
+	__RTE_BIT_OVERLOAD_V_2(family,, fun, c, size, arg1_type,	\
 	                             ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#673: FILE: lib/eal/include/rte_bitops.h:1404:
+#define __RTE_BIT_OVERLOAD_2(family, fun, c, arg1_type, arg1_name)	\
+	__RTE_BIT_OVERLOAD_SZ_2(family, fun, c, 32, arg1_type, arg1_name) \
+	__RTE_BIT_OVERLOAD_SZ_2(family, fun, c, 64, arg1_type, arg1_name)

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#677: FILE: lib/eal/include/rte_bitops.h:1408:
+#define __RTE_BIT_OVERLOAD_V_2R(family, v, fun, c, size, ret_type, arg1_type, \
+				arg1_name)				\
 	static inline ret_type						\
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
 			arg1_type arg1_name)				\
 	{								\
+		return __rte_bit_ ## family ## v ## fun ## size(addr,	\
+								arg1_name); \
 	}

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#681: FILE: lib/eal/include/rte_bitops.h:1411:
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
 	                                               ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#691: FILE: lib/eal/include/rte_bitops.h:1418:
+#define __RTE_BIT_OVERLOAD_SZ_2R(family, fun, c, size, ret_type, arg1_type, \
+				 arg1_name)				\
+	__RTE_BIT_OVERLOAD_V_2R(family,, fun, c, size, ret_type, arg1_type, \
+				arg1_name)				\
+	__RTE_BIT_OVERLOAD_V_2R(family, v_, fun, c volatile,		\
+				size, ret_type, arg1_type, arg1_name)

ERROR:SPACING: space required after that ',' (ctx:VxO)
#693: FILE: lib/eal/include/rte_bitops.h:1420:
+	__RTE_BIT_OVERLOAD_V_2R(family,, fun, c, size, ret_type, arg1_type, \
 	                              ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#698: FILE: lib/eal/include/rte_bitops.h:1425:
+#define __RTE_BIT_OVERLOAD_2R(family, fun, c, ret_type, arg1_type, arg1_name) \
+	__RTE_BIT_OVERLOAD_SZ_2R(family, fun, c, 32, ret_type, arg1_type, \
 				 arg1_name)				\
+	__RTE_BIT_OVERLOAD_SZ_2R(family, fun, c, 64, ret_type, arg1_type, \
 				 arg1_name)

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#712: FILE: lib/eal/include/rte_bitops.h:1434:
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
 	                                               ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#721: FILE: lib/eal/include/rte_bitops.h:1441:
+#define __RTE_BIT_OVERLOAD_SZ_3(family, fun, c, size, arg1_type, arg1_name, \
+				arg2_type, arg2_name)			\
+	__RTE_BIT_OVERLOAD_V_3(family,, fun, c, size, arg1_type, arg1_name, \
+			       arg2_type, arg2_name)			\
+	__RTE_BIT_OVERLOAD_V_3(family, v_, fun, c volatile, size, arg1_type, \
+			       arg1_name, arg2_type, arg2_name)

ERROR:SPACING: space required after that ',' (ctx:VxO)
#723: FILE: lib/eal/include/rte_bitops.h:1443:
+	__RTE_BIT_OVERLOAD_V_3(family,, fun, c, size, arg1_type, arg1_name, \
 	                             ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#728: FILE: lib/eal/include/rte_bitops.h:1448:
+#define __RTE_BIT_OVERLOAD_3(family, fun, c, arg1_type, arg1_name, arg2_type, \
 			     arg2_name)					\
+	__RTE_BIT_OVERLOAD_SZ_3(family, fun, c, 32, arg1_type, arg1_name, \
 				arg2_type, arg2_name)			\
+	__RTE_BIT_OVERLOAD_SZ_3(family, fun, c, 64, arg1_type, arg1_name, \
 				arg2_type, arg2_name)

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#739: FILE: lib/eal/include/rte_bitops.h:1455:
+#define __RTE_BIT_OVERLOAD_V_3R(family, v, fun, c, size, ret_type, arg1_type, \
+				arg1_name, arg2_type, arg2_name)	\
 	static inline ret_type						\
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
+				  arg1_type arg1_name, arg2_type arg2_name) \
 	{								\
+		return __rte_bit_ ## family ## v ## fun ## size(addr,	\
+								arg1_name, \
+								arg2_name); \
 	}

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#744: FILE: lib/eal/include/rte_bitops.h:1458:
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
 	                                               ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#756: FILE: lib/eal/include/rte_bitops.h:1466:
+#define __RTE_BIT_OVERLOAD_SZ_3R(family, fun, c, size, ret_type, arg1_type, \
 				 arg1_name, arg2_type, arg2_name)	\
+	__RTE_BIT_OVERLOAD_V_3R(family,, fun, c, size, ret_type, \
+				arg1_type, arg1_name, arg2_type, arg2_name) \
+	__RTE_BIT_OVERLOAD_V_3R(family, v_, fun, c volatile, size, \
+				ret_type, arg1_type, arg1_name, arg2_type, \
+				arg2_name)

ERROR:SPACING: space required after that ',' (ctx:VxO)
#760: FILE: lib/eal/include/rte_bitops.h:1468:
+	__RTE_BIT_OVERLOAD_V_3R(family,, fun, c, size, ret_type, \
 	                              ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#768: FILE: lib/eal/include/rte_bitops.h:1474:
+#define __RTE_BIT_OVERLOAD_3R(family, fun, c, ret_type, arg1_type, arg1_name, \
+			      arg2_type, arg2_name)			\
+	__RTE_BIT_OVERLOAD_SZ_3R(family, fun, c, 32, ret_type,		\
+				 arg1_type, arg1_name, arg2_type, arg2_name) \
+	__RTE_BIT_OVERLOAD_SZ_3R(family, fun, c, 64, ret_type, \
+				 arg1_type, arg1_name, arg2_type, arg2_name)

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#780: FILE: lib/eal/include/rte_bitops.h:1484:
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
 	                                               ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#794: FILE: lib/eal/include/rte_bitops.h:1493:
+#define __RTE_BIT_OVERLOAD_SZ_4(family, fun, c, size, arg1_type, arg1_name, \
 				arg2_type, arg2_name, arg3_type, arg3_name) \
+	__RTE_BIT_OVERLOAD_V_4(family,, fun, c, size, arg1_type,	\
+			       arg1_name, arg2_type, arg2_name, arg3_type, \
+			       arg3_name)				\
+	__RTE_BIT_OVERLOAD_V_4(family, v_, fun, c volatile, size,	\
+			       arg1_type, arg1_name, arg2_type, arg2_name, \
+			       arg3_type, arg3_name)

ERROR:SPACING: space required after that ',' (ctx:VxO)
#802: FILE: lib/eal/include/rte_bitops.h:1495:
+	__RTE_BIT_OVERLOAD_V_4(family,, fun, c, size, arg1_type,	\
 	                             ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#809: FILE: lib/eal/include/rte_bitops.h:1502:
+#define __RTE_BIT_OVERLOAD_4(family, fun, c, arg1_type, arg1_name, arg2_type, \
+			     arg2_name, arg3_type, arg3_name)		\
+	__RTE_BIT_OVERLOAD_SZ_4(family, fun, c, 32, arg1_type,		\
+				arg1_name, arg2_type, arg2_name, arg3_type, \
+				arg3_name)				\
+	__RTE_BIT_OVERLOAD_SZ_4(family, fun, c, 64, arg1_type,		\
+				arg1_name, arg2_type, arg2_name, arg3_type, \
+				arg3_name)

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#818: FILE: lib/eal/include/rte_bitops.h:1511:
+#define __RTE_BIT_OVERLOAD_V_4R(family, v, fun, c, size, ret_type, arg1_type, \
+				arg1_name, arg2_type, arg2_name, arg3_type, \
+				arg3_name)				\
 	static inline ret_type						\
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
+				  arg1_type arg1_name, arg2_type arg2_name, \
+				  arg3_type arg3_name)			\
 	{								\
+		return __rte_bit_ ## family ## v ## fun ## size(addr,	\
+								arg1_name, \
+								arg2_name, \
+								arg3_name); \
 	}

ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#824: FILE: lib/eal/include/rte_bitops.h:1515:
+	rte_bit_ ## family ## fun(c uint ## size ## _t *addr,		\
 	                                               ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#839: FILE: lib/eal/include/rte_bitops.h:1525:
+#define __RTE_BIT_OVERLOAD_SZ_4R(family, fun, c, size, ret_type, arg1_type, \
 				 arg1_name, arg2_type, arg2_name, arg3_type, \
 				 arg3_name)				\
+	__RTE_BIT_OVERLOAD_V_4R(family,, fun, c, size, ret_type, arg1_type, \
+				arg1_name, arg2_type, arg2_name, arg3_type, \
+				arg3_name)				\
+	__RTE_BIT_OVERLOAD_V_4R(family, v_, fun, c volatile, size,	\
+				ret_type, arg1_type, arg1_name, arg2_type, \
+				arg2_name, arg3_type, arg3_name)

ERROR:SPACING: space required after that ',' (ctx:VxO)
#853: FILE: lib/eal/include/rte_bitops.h:1528:
+	__RTE_BIT_OVERLOAD_V_4R(family,, fun, c, size, ret_type, arg1_type, \
 	                              ^

ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#860: FILE: lib/eal/include/rte_bitops.h:1535:
+#define __RTE_BIT_OVERLOAD_4R(family, fun, c, ret_type, arg1_type, arg1_name, \
+			      arg2_type, arg2_name, arg3_type, arg3_name) \
+	__RTE_BIT_OVERLOAD_SZ_4R(family, fun, c, 32, ret_type,		\
+				 arg1_type, arg1_name, arg2_type, arg2_name, \
+				 arg3_type, arg3_name)			\
+	__RTE_BIT_OVERLOAD_SZ_4R(family, fun, c, 64, ret_type,		\
+				 arg1_type, arg1_name, arg2_type, arg2_name, \
+				 arg3_type, arg3_name)

ERROR:SPACING: space required after that ',' (ctx:VxO)
#870: FILE: lib/eal/include/rte_bitops.h:1545:
+__RTE_BIT_OVERLOAD_2(, set,, unsigned int, nr)
                           ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#871: FILE: lib/eal/include/rte_bitops.h:1546:
+__RTE_BIT_OVERLOAD_2(, clear,, unsigned int, nr)
                             ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#872: FILE: lib/eal/include/rte_bitops.h:1547:
+__RTE_BIT_OVERLOAD_3(, assign,, unsigned int, nr, bool, value)
                              ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#873: FILE: lib/eal/include/rte_bitops.h:1548:
+__RTE_BIT_OVERLOAD_2(, flip,, unsigned int, nr)
                            ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#880: FILE: lib/eal/include/rte_bitops.h:1552:
+__RTE_BIT_OVERLOAD_3(atomic_, set,, unsigned int, nr, int, memory_order)
                                  ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#881: FILE: lib/eal/include/rte_bitops.h:1553:
+__RTE_BIT_OVERLOAD_3(atomic_, clear,, unsigned int, nr, int, memory_order)
                                    ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#882: FILE: lib/eal/include/rte_bitops.h:1554:
+__RTE_BIT_OVERLOAD_4(atomic_, assign,, unsigned int, nr, bool, value,
                                     ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#886: FILE: lib/eal/include/rte_bitops.h:1556:
+__RTE_BIT_OVERLOAD_3(atomic_, flip,, unsigned int, nr, int, memory_order)
                                   ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#887: FILE: lib/eal/include/rte_bitops.h:1557:
+__RTE_BIT_OVERLOAD_3R(atomic_, test_and_set,, bool, unsigned int, nr,
                                            ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#890: FILE: lib/eal/include/rte_bitops.h:1559:
+__RTE_BIT_OVERLOAD_3R(atomic_, test_and_clear,, bool, unsigned int, nr,
                                              ^

ERROR:SPACING: space required after that ',' (ctx:VxO)
#893: FILE: lib/eal/include/rte_bitops.h:1561:
+__RTE_BIT_OVERLOAD_4R(atomic_, test_and_assign,, bool, unsigned int, nr,
                                               ^

total: 156 errors, 2 warnings, 713 lines checked