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 A3DF345954 for ; Tue, 10 Sep 2024 10:43:24 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9B0F7427B6; Tue, 10 Sep 2024 10:43:24 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 8F4E7427A7 for ; Tue, 10 Sep 2024 10:43:23 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 65CB2121D10; Tue, 10 Sep 2024 10:43:23 +0200 (CEST) Subject: |WARNING| pw143855 [PATCH v6 6/6] eal: extend bitops to handle volatile pointers In-Reply-To: <20240910083139.699291-7-mattias.ronnblom@ericsson.com> References: <20240910083139.699291-7-mattias.ronnblom@ericsson.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: =?utf-8?q?Mattias_R=C3=B6nnblom?= Message-Id: <20240910084323.65CB2121D10@dpdk.org> Date: Tue, 10 Sep 2024 10:43:23 +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/143855 _coding style issues_ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #188: FILE: app/test/test_bitops.c:44: + const mod uint ## size ## _t *const_ptr = &word; \ ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #197: 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) #201: 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) #216: FILE: app/test/test_bitops.c:90: + bit_atomic_flip, bit_atomic_test, 32,) ^ ERROR:SPACING: space required after that ',' (ctx:VxB) #221: FILE: app/test/test_bitops.c:94: + bit_atomic_flip, bit_atomic_test, 64,) ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #262: FILE: lib/eal/include/rte_bitops.h:132: + uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #262: FILE: lib/eal/include/rte_bitops.h:132: + uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #263: FILE: lib/eal/include/rte_bitops.h:133: + const uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #263: FILE: lib/eal/include/rte_bitops.h:133: + const uint32_t *: __rte_bit_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #264: FILE: lib/eal/include/rte_bitops.h:134: + volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #264: FILE: lib/eal/include/rte_bitops.h:134: + volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #265: FILE: lib/eal/include/rte_bitops.h:135: + const volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #265: FILE: lib/eal/include/rte_bitops.h:135: + const volatile uint32_t *: __rte_bit_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #266: FILE: lib/eal/include/rte_bitops.h:136: + uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #266: FILE: lib/eal/include/rte_bitops.h:136: + uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #267: FILE: lib/eal/include/rte_bitops.h:137: + const uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #267: FILE: lib/eal/include/rte_bitops.h:137: + const uint64_t *: __rte_bit_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #268: FILE: lib/eal/include/rte_bitops.h:138: + volatile uint64_t *: __rte_bit_v_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #268: FILE: lib/eal/include/rte_bitops.h:138: + volatile uint64_t *: __rte_bit_v_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #269: 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) #269: 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) #283: FILE: lib/eal/include/rte_bitops.h:161: + uint32_t *: __rte_bit_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #283: FILE: lib/eal/include/rte_bitops.h:161: + uint32_t *: __rte_bit_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #284: FILE: lib/eal/include/rte_bitops.h:162: + volatile uint32_t *: __rte_bit_v_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #284: FILE: lib/eal/include/rte_bitops.h:162: + volatile uint32_t *: __rte_bit_v_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #285: FILE: lib/eal/include/rte_bitops.h:163: + uint64_t *: __rte_bit_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #285: FILE: lib/eal/include/rte_bitops.h:163: + uint64_t *: __rte_bit_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #286: 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) #286: 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) #300: FILE: lib/eal/include/rte_bitops.h:186: + uint32_t *: __rte_bit_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #300: FILE: lib/eal/include/rte_bitops.h:186: + uint32_t *: __rte_bit_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #301: FILE: lib/eal/include/rte_bitops.h:187: + volatile uint32_t *: __rte_bit_v_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #301: FILE: lib/eal/include/rte_bitops.h:187: + volatile uint32_t *: __rte_bit_v_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #302: FILE: lib/eal/include/rte_bitops.h:188: + uint64_t *: __rte_bit_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #302: FILE: lib/eal/include/rte_bitops.h:188: + uint64_t *: __rte_bit_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #303: 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) #303: 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) #312: FILE: lib/eal/include/rte_bitops.h:213: + volatile uint32_t *: __rte_bit_v_assign32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #312: FILE: lib/eal/include/rte_bitops.h:213: + volatile uint32_t *: __rte_bit_v_assign32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #313: FILE: lib/eal/include/rte_bitops.h:214: + uint64_t *: __rte_bit_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #313: FILE: lib/eal/include/rte_bitops.h:214: + uint64_t *: __rte_bit_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #314: 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) #314: 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) #323: FILE: lib/eal/include/rte_bitops.h:238: + volatile uint32_t *: __rte_bit_v_flip32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #323: FILE: lib/eal/include/rte_bitops.h:238: + volatile uint32_t *: __rte_bit_v_flip32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #324: FILE: lib/eal/include/rte_bitops.h:239: + uint64_t *: __rte_bit_flip64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #324: FILE: lib/eal/include/rte_bitops.h:239: + uint64_t *: __rte_bit_flip64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #325: 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) #325: 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) #333: FILE: lib/eal/include/rte_bitops.h:265: + volatile uint32_t *: __rte_bit_atomic_v_test32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #333: FILE: lib/eal/include/rte_bitops.h:265: + volatile uint32_t *: __rte_bit_atomic_v_test32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #334: 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) #334: 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) #338: FILE: lib/eal/include/rte_bitops.h:268: + const uint64_t *: __rte_bit_atomic_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #338: FILE: lib/eal/include/rte_bitops.h:268: + const uint64_t *: __rte_bit_atomic_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #339: FILE: lib/eal/include/rte_bitops.h:269: + volatile uint64_t *: __rte_bit_atomic_v_test64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #339: FILE: lib/eal/include/rte_bitops.h:269: + volatile uint64_t *: __rte_bit_atomic_v_test64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #340: 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) #340: 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) #350: FILE: lib/eal/include/rte_bitops.h:293: + volatile uint32_t *: __rte_bit_atomic_v_set32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #350: FILE: lib/eal/include/rte_bitops.h:293: + volatile uint32_t *: __rte_bit_atomic_v_set32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #351: FILE: lib/eal/include/rte_bitops.h:294: + uint64_t *: __rte_bit_atomic_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #351: FILE: lib/eal/include/rte_bitops.h:294: + uint64_t *: __rte_bit_atomic_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #352: 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) #352: 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) #362: FILE: lib/eal/include/rte_bitops.h:318: + volatile uint32_t *: __rte_bit_atomic_v_clear32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #362: FILE: lib/eal/include/rte_bitops.h:318: + volatile uint32_t *: __rte_bit_atomic_v_clear32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #363: FILE: lib/eal/include/rte_bitops.h:319: + uint64_t *: __rte_bit_atomic_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #363: FILE: lib/eal/include/rte_bitops.h:319: + uint64_t *: __rte_bit_atomic_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #364: 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) #364: 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) #375: FILE: lib/eal/include/rte_bitops.h:345: + volatile uint32_t *: __rte_bit_atomic_v_assign32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #375: FILE: lib/eal/include/rte_bitops.h:345: + volatile uint32_t *: __rte_bit_atomic_v_assign32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #376: FILE: lib/eal/include/rte_bitops.h:346: + uint64_t *: __rte_bit_atomic_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #376: FILE: lib/eal/include/rte_bitops.h:346: + uint64_t *: __rte_bit_atomic_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #377: 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) #377: 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) #388: FILE: lib/eal/include/rte_bitops.h:372: + volatile uint32_t *: __rte_bit_atomic_v_flip32, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #388: FILE: lib/eal/include/rte_bitops.h:372: + volatile uint32_t *: __rte_bit_atomic_v_flip32, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #389: FILE: lib/eal/include/rte_bitops.h:373: + uint64_t *: __rte_bit_atomic_flip64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #389: FILE: lib/eal/include/rte_bitops.h:373: + uint64_t *: __rte_bit_atomic_flip64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #390: 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) #390: 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) #401: 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) #401: 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) #402: FILE: lib/eal/include/rte_bitops.h:400: + uint64_t *: __rte_bit_atomic_test_and_set64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #402: FILE: lib/eal/include/rte_bitops.h:400: + uint64_t *: __rte_bit_atomic_test_and_set64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #403: 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) #403: 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) #414: 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) #414: 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) #415: FILE: lib/eal/include/rte_bitops.h:427: + uint64_t *: __rte_bit_atomic_test_and_clear64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #415: FILE: lib/eal/include/rte_bitops.h:427: + uint64_t *: __rte_bit_atomic_test_and_clear64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #416: 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) #416: 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) #428: 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) #428: 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) #429: FILE: lib/eal/include/rte_bitops.h:457: + uint64_t *: __rte_bit_atomic_test_and_assign64, \ ^ ERROR:SPACING: spaces required around that ':' (ctx:OxW) #429: FILE: lib/eal/include/rte_bitops.h:457: + uint64_t *: __rte_bit_atomic_test_and_assign64, \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxO) #430: 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) #430: 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) #440: FILE: lib/eal/include/rte_bitops.h:529: + __RTE_GEN_BIT_OPS(,, size) \ ^ ERROR:SPACING: space required after that ',' (ctx:BxO) #450: FILE: lib/eal/include/rte_bitops.h:670: + __RTE_GEN_BIT_ATOMIC_OPS(,, size) \ ^ ERROR:SPACING: need consistent spacing around '*' (ctx:WxV) #464: 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 #474: 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) #475: 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 #482: 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 #486: 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) #490: 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 #500: 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) #502: 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 #507: 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) #521: 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 #530: 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) #532: 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 #537: 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 #548: 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) #553: 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 #565: 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) #569: 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 #577: 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) #589: 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 #603: 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) #611: 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 #618: 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 #627: 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) #633: 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 #648: 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) #662: 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 #669: 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) #679: FILE: lib/eal/include/rte_bitops.h:1535: +__RTE_BIT_OVERLOAD_2(, set,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #680: FILE: lib/eal/include/rte_bitops.h:1536: +__RTE_BIT_OVERLOAD_2(, clear,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #681: 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) #682: FILE: lib/eal/include/rte_bitops.h:1538: +__RTE_BIT_OVERLOAD_2(, flip,, unsigned int, nr) ^ ERROR:SPACING: space required after that ',' (ctx:VxO) #689: 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) #690: 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) #691: 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) #695: 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) #696: 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) #699: 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) #702: 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