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 2626F44191 for ; Wed, 18 Sep 2024 11:17:00 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1FA8742E57; Wed, 18 Sep 2024 11:17:00 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id BDADE42D0B for ; Wed, 18 Sep 2024 11:16:58 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id A6B13121D10; Wed, 18 Sep 2024 11:16:58 +0200 (CEST) Subject: |WARNING| pw144247 [PATCH v9 6/6] eal: extend bitops to handle volatile pointers In-Reply-To: <20240918090445.725283-7-mattias.ronnblom@ericsson.com> References: <20240918090445.725283-7-mattias.ronnblom@ericsson.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: =?utf-8?q?Mattias_R=C3=B6nnblom?= Message-Id: <20240918091658.A6B13121D10@dpdk.org> Date: Wed, 18 Sep 2024 11:16:58 +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/144247 _coding style issues_ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #187: FILE: app/test/test_bitops.c:44: + const mod uint ## size ## _t *const_ptr = &word; \ ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #196: 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) #200: 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) #215: FILE: app/test/test_bitops.c:90: + bit_atomic_flip, bit_atomic_test, 32,) ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #220: FILE: app/test/test_bitops.c:94: + bit_atomic_flip, bit_atomic_test, 64,) ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #261: FILE: lib/eal/include/rte_bitops.h:132: + uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #261: FILE: lib/eal/include/rte_bitops.h:132: + uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #262: FILE: lib/eal/include/rte_bitops.h:133: + const uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #262: FILE: lib/eal/include/rte_bitops.h:133: + const uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #263: FILE: lib/eal/include/rte_bitops.h:134: + volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #263: FILE: lib/eal/include/rte_bitops.h:134: + volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #264: FILE: lib/eal/include/rte_bitops.h:135: + const volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #264: FILE: lib/eal/include/rte_bitops.h:135: + const volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #265: FILE: lib/eal/include/rte_bitops.h:136: + uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #265: FILE: lib/eal/include/rte_bitops.h:136: + uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #266: FILE: lib/eal/include/rte_bitops.h:137: + const uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #266: FILE: lib/eal/include/rte_bitops.h:137: + const uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #267: FILE: lib/eal/include/rte_bitops.h:138: + volatile uint64_t *: __rte_bit_v_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #267: FILE: lib/eal/include/rte_bitops.h:138: + volatile uint64_t *: __rte_bit_v_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #268: 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) #268: 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) #282: FILE: lib/eal/include/rte_bitops.h:161: + uint32_t *: __rte_bit_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #282: FILE: lib/eal/include/rte_bitops.h:161: + uint32_t *: __rte_bit_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #283: FILE: lib/eal/include/rte_bitops.h:162: + volatile uint32_t *: __rte_bit_v_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #283: FILE: lib/eal/include/rte_bitops.h:162: + volatile uint32_t *: __rte_bit_v_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #284: FILE: lib/eal/include/rte_bitops.h:163: + uint64_t *: __rte_bit_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #284: FILE: lib/eal/include/rte_bitops.h:163: + uint64_t *: __rte_bit_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #285: 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) #285: 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) #299: FILE: lib/eal/include/rte_bitops.h:186: + uint32_t *: __rte_bit_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #299: FILE: lib/eal/include/rte_bitops.h:186: + uint32_t *: __rte_bit_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #300: FILE: lib/eal/include/rte_bitops.h:187: + volatile uint32_t *: __rte_bit_v_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #300: FILE: lib/eal/include/rte_bitops.h:187: + volatile uint32_t *: __rte_bit_v_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #301: FILE: lib/eal/include/rte_bitops.h:188: + uint64_t *: __rte_bit_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #301: FILE: lib/eal/include/rte_bitops.h:188: + uint64_t *: __rte_bit_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #302: 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) #302: 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) #311: FILE: lib/eal/include/rte_bitops.h:213: + volatile uint32_t *: __rte_bit_v_assign32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #311: FILE: lib/eal/include/rte_bitops.h:213: + volatile uint32_t *: __rte_bit_v_assign32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #312: FILE: lib/eal/include/rte_bitops.h:214: + uint64_t *: __rte_bit_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #312: FILE: lib/eal/include/rte_bitops.h:214: + uint64_t *: __rte_bit_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #313: 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) #313: 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) #322: FILE: lib/eal/include/rte_bitops.h:238: + volatile uint32_t *: __rte_bit_v_flip32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #322: FILE: lib/eal/include/rte_bitops.h:238: + volatile uint32_t *: __rte_bit_v_flip32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #323: FILE: lib/eal/include/rte_bitops.h:239: + uint64_t *: __rte_bit_flip64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #323: FILE: lib/eal/include/rte_bitops.h:239: + uint64_t *: __rte_bit_flip64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #324: 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) #324: 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) #332: FILE: lib/eal/include/rte_bitops.h:265: + volatile uint32_t *: __rte_bit_atomic_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #332: FILE: lib/eal/include/rte_bitops.h:265: + volatile uint32_t *: __rte_bit_atomic_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #333: 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) #333: 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) #337: FILE: lib/eal/include/rte_bitops.h:268: + const uint64_t *: __rte_bit_atomic_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #337: FILE: lib/eal/include/rte_bitops.h:268: + const uint64_t *: __rte_bit_atomic_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #338: FILE: lib/eal/include/rte_bitops.h:269: + volatile uint64_t *: __rte_bit_atomic_v_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #338: FILE: lib/eal/include/rte_bitops.h:269: + volatile uint64_t *: __rte_bit_atomic_v_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #339: 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) #339: 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) #349: FILE: lib/eal/include/rte_bitops.h:293: + volatile uint32_t *: __rte_bit_atomic_v_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #349: FILE: lib/eal/include/rte_bitops.h:293: + volatile uint32_t *: __rte_bit_atomic_v_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #350: FILE: lib/eal/include/rte_bitops.h:294: + uint64_t *: __rte_bit_atomic_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #350: FILE: lib/eal/include/rte_bitops.h:294: + uint64_t *: __rte_bit_atomic_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #351: 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) #351: 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) #361: FILE: lib/eal/include/rte_bitops.h:318: + volatile uint32_t *: __rte_bit_atomic_v_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #361: FILE: lib/eal/include/rte_bitops.h:318: + volatile uint32_t *: __rte_bit_atomic_v_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #362: FILE: lib/eal/include/rte_bitops.h:319: + uint64_t *: __rte_bit_atomic_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #362: FILE: lib/eal/include/rte_bitops.h:319: + uint64_t *: __rte_bit_atomic_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #363: 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) #363: 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) #374: FILE: lib/eal/include/rte_bitops.h:345: + volatile uint32_t *: __rte_bit_atomic_v_assign32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #374: FILE: lib/eal/include/rte_bitops.h:345: + volatile uint32_t *: __rte_bit_atomic_v_assign32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #375: FILE: lib/eal/include/rte_bitops.h:346: + uint64_t *: __rte_bit_atomic_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #375: FILE: lib/eal/include/rte_bitops.h:346: + uint64_t *: __rte_bit_atomic_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #376: 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) #376: 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) #387: FILE: lib/eal/include/rte_bitops.h:372: + volatile uint32_t *: __rte_bit_atomic_v_flip32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #387: FILE: lib/eal/include/rte_bitops.h:372: + volatile uint32_t *: __rte_bit_atomic_v_flip32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #388: FILE: lib/eal/include/rte_bitops.h:373: + uint64_t *: __rte_bit_atomic_flip64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #388: FILE: lib/eal/include/rte_bitops.h:373: + uint64_t *: __rte_bit_atomic_flip64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #389: 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) #389: 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) #400: 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) #400: 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) #401: FILE: lib/eal/include/rte_bitops.h:400: + uint64_t *: __rte_bit_atomic_test_and_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #401: FILE: lib/eal/include/rte_bitops.h:400: + uint64_t *: __rte_bit_atomic_test_and_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #402: 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) #402: 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) #413: 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) #413: 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) #414: FILE: lib/eal/include/rte_bitops.h:427: + uint64_t *: __rte_bit_atomic_test_and_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #414: FILE: lib/eal/include/rte_bitops.h:427: + uint64_t *: __rte_bit_atomic_test_and_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #415: 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) #415: 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) #427: 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) #427: 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) #428: FILE: lib/eal/include/rte_bitops.h:457: + uint64_t *: __rte_bit_atomic_test_and_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #428: FILE: lib/eal/include/rte_bitops.h:457: + uint64_t *: __rte_bit_atomic_test_and_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #429: 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) #429: FILE: lib/eal/include/rte_bitops.h:458: + volatile uint64_t *: __rte_bit_atomic_v_test_and_assign64) \ ^ ERROR:SPACING: space required after that ',' (ctx:BxO) #439: FILE: lib/eal/include/rte_bitops.h:529: + __RTE_GEN_BIT_OPS(,, size) \ ^ ERROR:SPACING: space required after that ',' (ctx:BxO) #449: FILE: lib/eal/include/rte_bitops.h:670: + __RTE_GEN_BIT_ATOMIC_OPS(,, size) \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #463: FILE: lib/eal/include/rte_bitops.h:1382: + rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #473: FILE: lib/eal/include/rte_bitops.h:1388: +#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) #474: FILE: lib/eal/include/rte_bitops.h:1389: + __RTE_BIT_OVERLOAD_V_2(family,, fun, c, size, arg1_type, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #481: FILE: lib/eal/include/rte_bitops.h:1394: +#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 #485: FILE: lib/eal/include/rte_bitops.h:1398: +#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) #489: FILE: lib/eal/include/rte_bitops.h:1401: + rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #499: FILE: lib/eal/include/rte_bitops.h:1408: +#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) #501: FILE: lib/eal/include/rte_bitops.h:1410: + __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 #506: FILE: lib/eal/include/rte_bitops.h:1415: +#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) #520: FILE: lib/eal/include/rte_bitops.h:1424: + rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #529: FILE: lib/eal/include/rte_bitops.h:1431: +#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) #531: FILE: lib/eal/include/rte_bitops.h:1433: + __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 #536: FILE: lib/eal/include/rte_bitops.h:1438: +#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 #547: FILE: lib/eal/include/rte_bitops.h:1445: +#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) #552: FILE: lib/eal/include/rte_bitops.h:1448: + rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #564: FILE: lib/eal/include/rte_bitops.h:1456: +#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) #568: FILE: lib/eal/include/rte_bitops.h:1458: + __RTE_BIT_OVERLOAD_V_3R(family,, fun, c, size, ret_type, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #576: FILE: lib/eal/include/rte_bitops.h:1464: +#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) #588: FILE: lib/eal/include/rte_bitops.h:1474: + rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #602: FILE: lib/eal/include/rte_bitops.h:1483: +#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) #610: FILE: lib/eal/include/rte_bitops.h:1485: + __RTE_BIT_OVERLOAD_V_4(family,, fun, c, size, arg1_type, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #617: FILE: lib/eal/include/rte_bitops.h:1492: +#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 #626: FILE: lib/eal/include/rte_bitops.h:1501: +#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) #632: FILE: lib/eal/include/rte_bitops.h:1505: + rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \ ^ ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #647: FILE: lib/eal/include/rte_bitops.h:1515: +#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) #661: FILE: lib/eal/include/rte_bitops.h:1518: + __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 #668: FILE: lib/eal/include/rte_bitops.h:1525: +#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) #678: FILE: lib/eal/include/rte_bitops.h:1535: +__RTE_BIT_OVERLOAD_2(, set,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #679: FILE: lib/eal/include/rte_bitops.h:1536: +__RTE_BIT_OVERLOAD_2(, clear,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #680: FILE: lib/eal/include/rte_bitops.h:1537: +__RTE_BIT_OVERLOAD_3(, assign,, unsigned int, nr, bool, value) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #681: FILE: lib/eal/include/rte_bitops.h:1538: +__RTE_BIT_OVERLOAD_2(, flip,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #688: FILE: lib/eal/include/rte_bitops.h:1542: +__RTE_BIT_OVERLOAD_3(atomic_, set,, unsigned int, nr, int, memory_order) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #689: FILE: lib/eal/include/rte_bitops.h:1543: +__RTE_BIT_OVERLOAD_3(atomic_, clear,, unsigned int, nr, int, memory_order) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #690: FILE: lib/eal/include/rte_bitops.h:1544: +__RTE_BIT_OVERLOAD_4(atomic_, assign,, unsigned int, nr, bool, value, ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #694: FILE: lib/eal/include/rte_bitops.h:1546: +__RTE_BIT_OVERLOAD_3(atomic_, flip,, unsigned int, nr, int, memory_order) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #695: FILE: lib/eal/include/rte_bitops.h:1547: +__RTE_BIT_OVERLOAD_3R(atomic_, test_and_set,, bool, unsigned int, nr, ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #698: FILE: lib/eal/include/rte_bitops.h:1549: +__RTE_BIT_OVERLOAD_3R(atomic_, test_and_clear,, bool, unsigned int, nr, ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #701: FILE: lib/eal/include/rte_bitops.h:1551: +__RTE_BIT_OVERLOAD_4R(atomic_, test_and_assign,, bool, unsigned int, nr, ^ total: 141 errors, 0 warnings, 514 lines checked