From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: "Mattias Rönnblom" <mattias.ronnblom@ericsson.com>
Subject: |WARNING| pw144109 [PATCH v7 6/6] eal: extend bitops to handle volatile pointers
Date: Tue, 17 Sep 2024 11:48:01 +0200 (CEST) [thread overview]
Message-ID: <20240917094801.DEE26121D10@dpdk.org> (raw)
In-Reply-To: <20240917093646.723777-7-mattias.ronnblom@ericsson.com>
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
next prev parent reply other threads:[~2024-09-17 9:48 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20240917093646.723777-7-mattias.ronnblom@ericsson.com>
2024-09-17 9:21 ` |SUCCESS| pw144108-144109 " qemudev
2024-09-17 9:25 ` qemudev
2024-09-17 9:48 ` checkpatch [this message]
2024-09-17 10:44 ` |FAILURE| pw144109 " 0-day Robot
2024-09-17 12:51 ` |SUCCESS| pw144108-144109 [PATCH] [v7,6/6] eal: extend bitops to han dpdklab
2024-09-17 12:53 ` dpdklab
2024-09-17 13:01 ` dpdklab
2024-09-17 13:03 ` dpdklab
2024-09-17 13:11 ` |PENDING| " dpdklab
2024-09-17 13:39 ` dpdklab
2024-09-17 13:43 ` |SUCCESS| " dpdklab
2024-09-17 13:56 ` |PENDING| " dpdklab
2024-09-17 13:57 ` |SUCCESS| " dpdklab
2024-09-17 14:15 ` |FAILURE| " dpdklab
2024-09-17 14:16 ` dpdklab
2024-09-17 14:30 ` |SUCCESS| " dpdklab
2024-09-17 14:32 ` dpdklab
2024-09-17 17:03 ` dpdklab
2024-09-17 18:39 ` dpdklab
2024-09-17 20:13 ` dpdklab
2024-09-17 22:26 ` |FAILURE| " dpdklab
2024-09-22 3:34 ` |SUCCESS| " dpdklab
2024-09-22 3:45 ` dpdklab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240917094801.DEE26121D10@dpdk.org \
--to=checkpatch@dpdk.org \
--cc=mattias.ronnblom@ericsson.com \
--cc=test-report@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).