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 F4070459BC for ; Tue, 17 Sep 2024 11:48:02 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id EBE8F40261; Tue, 17 Sep 2024 11:48:02 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 0889C40A4B for ; Tue, 17 Sep 2024 11:48:02 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id DEE26121D10; Tue, 17 Sep 2024 11:48:01 +0200 (CEST) Subject: |WARNING| pw144109 [PATCH v7 6/6] eal: extend bitops to handle volatile pointers In-Reply-To: <20240917093646.723777-7-mattias.ronnblom@ericsson.com> References: <20240917093646.723777-7-mattias.ronnblom@ericsson.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: =?utf-8?q?Mattias_R=C3=B6nnblom?= Message-Id: <20240917094801.DEE26121D10@dpdk.org> Date: Tue, 17 Sep 2024 11:48:01 +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/144109 _coding style issues_ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #186: FILE: app/test/test_bitops.c:44: + const mod uint ## size ## _t *const_ptr = &word; \ ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #195: 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) #199: 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) #214: FILE: app/test/test_bitops.c:90: + bit_atomic_flip, bit_atomic_test, 32,) ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #219: FILE: app/test/test_bitops.c:94: + bit_atomic_flip, bit_atomic_test, 64,) ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #260: FILE: lib/eal/include/rte_bitops.h:132: + uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #260: FILE: lib/eal/include/rte_bitops.h:132: + uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #261: FILE: lib/eal/include/rte_bitops.h:133: + const uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #261: FILE: lib/eal/include/rte_bitops.h:133: + const uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #262: FILE: lib/eal/include/rte_bitops.h:134: + volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #262: FILE: lib/eal/include/rte_bitops.h:134: + volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #263: FILE: lib/eal/include/rte_bitops.h:135: + const volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #263: FILE: lib/eal/include/rte_bitops.h:135: + const volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #264: FILE: lib/eal/include/rte_bitops.h:136: + uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #264: FILE: lib/eal/include/rte_bitops.h:136: + uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #265: FILE: lib/eal/include/rte_bitops.h:137: + const uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #265: FILE: lib/eal/include/rte_bitops.h:137: + const uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #266: FILE: lib/eal/include/rte_bitops.h:138: + volatile uint64_t *: __rte_bit_v_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #266: FILE: lib/eal/include/rte_bitops.h:138: + volatile uint64_t *: __rte_bit_v_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #267: 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) #267: 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) #281: FILE: lib/eal/include/rte_bitops.h:161: + uint32_t *: __rte_bit_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #281: FILE: lib/eal/include/rte_bitops.h:161: + uint32_t *: __rte_bit_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #282: FILE: lib/eal/include/rte_bitops.h:162: + volatile uint32_t *: __rte_bit_v_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #282: FILE: lib/eal/include/rte_bitops.h:162: + volatile uint32_t *: __rte_bit_v_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #283: FILE: lib/eal/include/rte_bitops.h:163: + uint64_t *: __rte_bit_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #283: FILE: lib/eal/include/rte_bitops.h:163: + uint64_t *: __rte_bit_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #284: 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) #284: 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) #298: FILE: lib/eal/include/rte_bitops.h:186: + uint32_t *: __rte_bit_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #298: FILE: lib/eal/include/rte_bitops.h:186: + uint32_t *: __rte_bit_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #299: FILE: lib/eal/include/rte_bitops.h:187: + volatile uint32_t *: __rte_bit_v_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #299: FILE: lib/eal/include/rte_bitops.h:187: + volatile uint32_t *: __rte_bit_v_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #300: FILE: lib/eal/include/rte_bitops.h:188: + uint64_t *: __rte_bit_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #300: FILE: lib/eal/include/rte_bitops.h:188: + uint64_t *: __rte_bit_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #301: 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) #301: 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) #310: FILE: lib/eal/include/rte_bitops.h:213: + volatile uint32_t *: __rte_bit_v_assign32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #310: FILE: lib/eal/include/rte_bitops.h:213: + volatile uint32_t *: __rte_bit_v_assign32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #311: FILE: lib/eal/include/rte_bitops.h:214: + uint64_t *: __rte_bit_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #311: FILE: lib/eal/include/rte_bitops.h:214: + uint64_t *: __rte_bit_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #312: 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) #312: 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) #321: FILE: lib/eal/include/rte_bitops.h:238: + volatile uint32_t *: __rte_bit_v_flip32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #321: FILE: lib/eal/include/rte_bitops.h:238: + volatile uint32_t *: __rte_bit_v_flip32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #322: FILE: lib/eal/include/rte_bitops.h:239: + uint64_t *: __rte_bit_flip64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #322: FILE: lib/eal/include/rte_bitops.h:239: + uint64_t *: __rte_bit_flip64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #323: 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) #323: 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) #331: FILE: lib/eal/include/rte_bitops.h:265: + volatile uint32_t *: __rte_bit_atomic_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #331: FILE: lib/eal/include/rte_bitops.h:265: + volatile uint32_t *: __rte_bit_atomic_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #332: 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) #332: 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) #336: FILE: lib/eal/include/rte_bitops.h:268: + const uint64_t *: __rte_bit_atomic_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #336: FILE: lib/eal/include/rte_bitops.h:268: + const uint64_t *: __rte_bit_atomic_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #337: FILE: lib/eal/include/rte_bitops.h:269: + volatile uint64_t *: __rte_bit_atomic_v_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #337: FILE: lib/eal/include/rte_bitops.h:269: + volatile uint64_t *: __rte_bit_atomic_v_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #338: 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) #338: 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) #348: FILE: lib/eal/include/rte_bitops.h:293: + volatile uint32_t *: __rte_bit_atomic_v_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #348: FILE: lib/eal/include/rte_bitops.h:293: + volatile uint32_t *: __rte_bit_atomic_v_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #349: FILE: lib/eal/include/rte_bitops.h:294: + uint64_t *: __rte_bit_atomic_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #349: FILE: lib/eal/include/rte_bitops.h:294: + uint64_t *: __rte_bit_atomic_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #350: 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) #350: 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) #360: FILE: lib/eal/include/rte_bitops.h:318: + volatile uint32_t *: __rte_bit_atomic_v_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #360: FILE: lib/eal/include/rte_bitops.h:318: + volatile uint32_t *: __rte_bit_atomic_v_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #361: FILE: lib/eal/include/rte_bitops.h:319: + uint64_t *: __rte_bit_atomic_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #361: FILE: lib/eal/include/rte_bitops.h:319: + uint64_t *: __rte_bit_atomic_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #362: 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) #362: 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) #373: FILE: lib/eal/include/rte_bitops.h:345: + volatile uint32_t *: __rte_bit_atomic_v_assign32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #373: FILE: lib/eal/include/rte_bitops.h:345: + volatile uint32_t *: __rte_bit_atomic_v_assign32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #374: FILE: lib/eal/include/rte_bitops.h:346: + uint64_t *: __rte_bit_atomic_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #374: FILE: lib/eal/include/rte_bitops.h:346: + uint64_t *: __rte_bit_atomic_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #375: 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) #375: 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) #386: FILE: lib/eal/include/rte_bitops.h:372: + volatile uint32_t *: __rte_bit_atomic_v_flip32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #386: FILE: lib/eal/include/rte_bitops.h:372: + volatile uint32_t *: __rte_bit_atomic_v_flip32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #387: FILE: lib/eal/include/rte_bitops.h:373: + uint64_t *: __rte_bit_atomic_flip64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #387: FILE: lib/eal/include/rte_bitops.h:373: + uint64_t *: __rte_bit_atomic_flip64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #388: 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) #388: 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) #399: 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) #399: 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) #400: FILE: lib/eal/include/rte_bitops.h:400: + uint64_t *: __rte_bit_atomic_test_and_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #400: FILE: lib/eal/include/rte_bitops.h:400: + uint64_t *: __rte_bit_atomic_test_and_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #401: 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) #401: 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) #412: 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) #412: 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) #413: FILE: lib/eal/include/rte_bitops.h:427: + uint64_t *: __rte_bit_atomic_test_and_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #413: FILE: lib/eal/include/rte_bitops.h:427: + uint64_t *: __rte_bit_atomic_test_and_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #414: 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) #414: 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) #426: 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) #426: 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) #427: FILE: lib/eal/include/rte_bitops.h:457: + uint64_t *: __rte_bit_atomic_test_and_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #427: FILE: lib/eal/include/rte_bitops.h:457: + uint64_t *: __rte_bit_atomic_test_and_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #428: 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) #428: 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) #438: FILE: lib/eal/include/rte_bitops.h:529: + __RTE_GEN_BIT_OPS(,, size) \ ^ ERROR:SPACING: space required after that ',' (ctx:BxO) #448: FILE: lib/eal/include/rte_bitops.h:670: + __RTE_GEN_BIT_ATOMIC_OPS(,, size) \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #462: 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 #472: 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) #473: 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 #480: 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 #484: 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) #488: 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 #498: 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) #500: 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 #505: 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) #519: 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 #528: 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) #530: 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 #535: 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 #546: 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) #551: 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 #563: 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) #567: 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 #575: 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) #587: 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 #601: 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) #609: 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 #616: 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 #625: 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) #631: 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 #646: 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) #660: 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 #667: 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) #677: FILE: lib/eal/include/rte_bitops.h:1535: +__RTE_BIT_OVERLOAD_2(, set,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #678: FILE: lib/eal/include/rte_bitops.h:1536: +__RTE_BIT_OVERLOAD_2(, clear,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #679: 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) #680: FILE: lib/eal/include/rte_bitops.h:1538: +__RTE_BIT_OVERLOAD_2(, flip,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #687: 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) #688: 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) #689: 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) #693: 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) #694: 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) #697: 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) #700: 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