From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: "Mattias Rönnblom" <mattias.ronnblom@ericsson.com>
Subject: |WARNING| pw143048 [PATCH v2 5/5] eal: extend bitops to handle volatile pointers
Date: Fri, 9 Aug 2024 12:09:11 +0200 (CEST) [thread overview]
Message-ID: <20240809100911.14F2F123EF6@dpdk.org> (raw)
In-Reply-To: <20240809095829.589396-6-mattias.ronnblom@ericsson.com>
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/143048
_coding style issues_
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#179: FILE: app/test/test_bitops.c:44:
+ const mod uint ## size ## _t *const_ptr = &word; \
^
ERROR:SPACING: space required after that ',' (ctx:VxB)
#188: FILE: app/test/test_bitops.c:62:
+ rte_bit_assign, rte_bit_flip, rte_bit_test, 32,)
^
ERROR:SPACING: space required after that ',' (ctx:VxB)
#192: FILE: app/test/test_bitops.c:65:
+ rte_bit_assign, rte_bit_flip, rte_bit_test, 64,)
^
ERROR:SPACING: space required after that ',' (ctx:VxB)
#207: FILE: app/test/test_bitops.c:90:
+ bit_atomic_flip, bit_atomic_test, 32,)
^
ERROR:SPACING: space required after that ',' (ctx:VxB)
#212: FILE: app/test/test_bitops.c:94:
+ bit_atomic_flip, bit_atomic_test, 64,)
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#253: FILE: lib/eal/include/rte_bitops.h:132:
+ uint32_t *: __rte_bit_test32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#253: FILE: lib/eal/include/rte_bitops.h:132:
+ uint32_t *: __rte_bit_test32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#254: FILE: lib/eal/include/rte_bitops.h:133:
+ const uint32_t *: __rte_bit_test32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#254: FILE: lib/eal/include/rte_bitops.h:133:
+ const uint32_t *: __rte_bit_test32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#255: FILE: lib/eal/include/rte_bitops.h:134:
+ volatile uint32_t *: __rte_bit_v_test32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#255: FILE: lib/eal/include/rte_bitops.h:134:
+ volatile uint32_t *: __rte_bit_v_test32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#256: FILE: lib/eal/include/rte_bitops.h:135:
+ const volatile uint32_t *: __rte_bit_v_test32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#256: FILE: lib/eal/include/rte_bitops.h:135:
+ const volatile uint32_t *: __rte_bit_v_test32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#257: FILE: lib/eal/include/rte_bitops.h:136:
+ uint64_t *: __rte_bit_test64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#257: FILE: lib/eal/include/rte_bitops.h:136:
+ uint64_t *: __rte_bit_test64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#258: FILE: lib/eal/include/rte_bitops.h:137:
+ const uint64_t *: __rte_bit_test64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#258: FILE: lib/eal/include/rte_bitops.h:137:
+ const uint64_t *: __rte_bit_test64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#259: FILE: lib/eal/include/rte_bitops.h:138:
+ volatile uint64_t *: __rte_bit_v_test64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#259: FILE: lib/eal/include/rte_bitops.h:138:
+ volatile uint64_t *: __rte_bit_v_test64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#260: FILE: lib/eal/include/rte_bitops.h:139:
+ const volatile uint64_t *: __rte_bit_v_test64)(addr, nr)
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#260: FILE: lib/eal/include/rte_bitops.h:139:
+ const volatile uint64_t *: __rte_bit_v_test64)(addr, nr)
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#274: FILE: lib/eal/include/rte_bitops.h:161:
+ uint32_t *: __rte_bit_set32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#274: FILE: lib/eal/include/rte_bitops.h:161:
+ uint32_t *: __rte_bit_set32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#275: FILE: lib/eal/include/rte_bitops.h:162:
+ volatile uint32_t *: __rte_bit_v_set32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#275: FILE: lib/eal/include/rte_bitops.h:162:
+ volatile uint32_t *: __rte_bit_v_set32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#276: FILE: lib/eal/include/rte_bitops.h:163:
+ uint64_t *: __rte_bit_set64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#276: FILE: lib/eal/include/rte_bitops.h:163:
+ uint64_t *: __rte_bit_set64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#277: FILE: lib/eal/include/rte_bitops.h:164:
+ volatile uint64_t *: __rte_bit_v_set64)(addr, nr)
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#277: FILE: lib/eal/include/rte_bitops.h:164:
+ volatile uint64_t *: __rte_bit_v_set64)(addr, nr)
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#291: FILE: lib/eal/include/rte_bitops.h:186:
+ uint32_t *: __rte_bit_clear32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#291: FILE: lib/eal/include/rte_bitops.h:186:
+ uint32_t *: __rte_bit_clear32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#292: FILE: lib/eal/include/rte_bitops.h:187:
+ volatile uint32_t *: __rte_bit_v_clear32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#292: FILE: lib/eal/include/rte_bitops.h:187:
+ volatile uint32_t *: __rte_bit_v_clear32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#293: FILE: lib/eal/include/rte_bitops.h:188:
+ uint64_t *: __rte_bit_clear64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#293: FILE: lib/eal/include/rte_bitops.h:188:
+ uint64_t *: __rte_bit_clear64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#294: FILE: lib/eal/include/rte_bitops.h:189:
+ volatile uint64_t *: __rte_bit_v_clear64)(addr, nr)
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#294: FILE: lib/eal/include/rte_bitops.h:189:
+ volatile uint64_t *: __rte_bit_v_clear64)(addr, nr)
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#303: FILE: lib/eal/include/rte_bitops.h:213:
+ volatile uint32_t *: __rte_bit_v_assign32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#303: FILE: lib/eal/include/rte_bitops.h:213:
+ volatile uint32_t *: __rte_bit_v_assign32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#304: FILE: lib/eal/include/rte_bitops.h:214:
+ uint64_t *: __rte_bit_assign64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#304: FILE: lib/eal/include/rte_bitops.h:214:
+ uint64_t *: __rte_bit_assign64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#305: FILE: lib/eal/include/rte_bitops.h:215:
+ volatile uint64_t *: __rte_bit_v_assign64)(addr, nr, value)
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#305: FILE: lib/eal/include/rte_bitops.h:215:
+ volatile uint64_t *: __rte_bit_v_assign64)(addr, nr, value)
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#314: FILE: lib/eal/include/rte_bitops.h:238:
+ volatile uint32_t *: __rte_bit_v_flip32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#314: FILE: lib/eal/include/rte_bitops.h:238:
+ volatile uint32_t *: __rte_bit_v_flip32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#315: FILE: lib/eal/include/rte_bitops.h:239:
+ uint64_t *: __rte_bit_flip64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#315: FILE: lib/eal/include/rte_bitops.h:239:
+ uint64_t *: __rte_bit_flip64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#316: FILE: lib/eal/include/rte_bitops.h:240:
+ volatile uint64_t *: __rte_bit_v_flip64)(addr, nr)
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#316: FILE: lib/eal/include/rte_bitops.h:240:
+ volatile uint64_t *: __rte_bit_v_flip64)(addr, nr)
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#324: FILE: lib/eal/include/rte_bitops.h:265:
+ volatile uint32_t *: __rte_bit_atomic_v_test32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#324: FILE: lib/eal/include/rte_bitops.h:265:
+ volatile uint32_t *: __rte_bit_atomic_v_test32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#325: FILE: lib/eal/include/rte_bitops.h:266:
+ const volatile uint32_t *: __rte_bit_atomic_v_test32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#325: FILE: lib/eal/include/rte_bitops.h:266:
+ const volatile uint32_t *: __rte_bit_atomic_v_test32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#329: FILE: lib/eal/include/rte_bitops.h:268:
+ const uint64_t *: __rte_bit_atomic_test64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#329: FILE: lib/eal/include/rte_bitops.h:268:
+ const uint64_t *: __rte_bit_atomic_test64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#330: FILE: lib/eal/include/rte_bitops.h:269:
+ volatile uint64_t *: __rte_bit_atomic_v_test64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#330: FILE: lib/eal/include/rte_bitops.h:269:
+ volatile uint64_t *: __rte_bit_atomic_v_test64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#331: FILE: lib/eal/include/rte_bitops.h:270:
+ const volatile uint64_t *: __rte_bit_atomic_v_test64) \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#331: FILE: lib/eal/include/rte_bitops.h:270:
+ const volatile uint64_t *: __rte_bit_atomic_v_test64) \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#341: FILE: lib/eal/include/rte_bitops.h:293:
+ volatile uint32_t *: __rte_bit_atomic_v_set32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#341: FILE: lib/eal/include/rte_bitops.h:293:
+ volatile uint32_t *: __rte_bit_atomic_v_set32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#342: FILE: lib/eal/include/rte_bitops.h:294:
+ uint64_t *: __rte_bit_atomic_set64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#342: FILE: lib/eal/include/rte_bitops.h:294:
+ uint64_t *: __rte_bit_atomic_set64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#343: FILE: lib/eal/include/rte_bitops.h:295:
+ volatile uint64_t *: __rte_bit_atomic_v_set64)(addr, nr, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#343: FILE: lib/eal/include/rte_bitops.h:295:
+ volatile uint64_t *: __rte_bit_atomic_v_set64)(addr, nr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#353: FILE: lib/eal/include/rte_bitops.h:318:
+ volatile uint32_t *: __rte_bit_atomic_v_clear32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#353: FILE: lib/eal/include/rte_bitops.h:318:
+ volatile uint32_t *: __rte_bit_atomic_v_clear32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#354: FILE: lib/eal/include/rte_bitops.h:319:
+ uint64_t *: __rte_bit_atomic_clear64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#354: FILE: lib/eal/include/rte_bitops.h:319:
+ uint64_t *: __rte_bit_atomic_clear64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#355: FILE: lib/eal/include/rte_bitops.h:320:
+ volatile uint64_t *: __rte_bit_atomic_v_clear64)(addr, nr, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#355: FILE: lib/eal/include/rte_bitops.h:320:
+ volatile uint64_t *: __rte_bit_atomic_v_clear64)(addr, nr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#366: FILE: lib/eal/include/rte_bitops.h:345:
+ volatile uint32_t *: __rte_bit_atomic_v_assign32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#366: FILE: lib/eal/include/rte_bitops.h:345:
+ volatile uint32_t *: __rte_bit_atomic_v_assign32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#367: FILE: lib/eal/include/rte_bitops.h:346:
+ uint64_t *: __rte_bit_atomic_assign64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#367: FILE: lib/eal/include/rte_bitops.h:346:
+ uint64_t *: __rte_bit_atomic_assign64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#368: FILE: lib/eal/include/rte_bitops.h:347:
+ volatile uint64_t *: __rte_bit_atomic_v_assign64)(addr, nr, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#368: FILE: lib/eal/include/rte_bitops.h:347:
+ volatile uint64_t *: __rte_bit_atomic_v_assign64)(addr, nr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#379: FILE: lib/eal/include/rte_bitops.h:372:
+ volatile uint32_t *: __rte_bit_atomic_v_flip32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#379: FILE: lib/eal/include/rte_bitops.h:372:
+ volatile uint32_t *: __rte_bit_atomic_v_flip32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#380: FILE: lib/eal/include/rte_bitops.h:373:
+ uint64_t *: __rte_bit_atomic_flip64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#380: FILE: lib/eal/include/rte_bitops.h:373:
+ uint64_t *: __rte_bit_atomic_flip64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#381: FILE: lib/eal/include/rte_bitops.h:374:
+ volatile uint64_t *: __rte_bit_atomic_v_flip64)(addr, nr, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#381: FILE: lib/eal/include/rte_bitops.h:374:
+ volatile uint64_t *: __rte_bit_atomic_v_flip64)(addr, nr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#392: FILE: lib/eal/include/rte_bitops.h:399:
+ volatile uint32_t *: __rte_bit_atomic_v_test_and_set32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#392: FILE: lib/eal/include/rte_bitops.h:399:
+ volatile uint32_t *: __rte_bit_atomic_v_test_and_set32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#393: FILE: lib/eal/include/rte_bitops.h:400:
+ uint64_t *: __rte_bit_atomic_test_and_set64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#393: FILE: lib/eal/include/rte_bitops.h:400:
+ uint64_t *: __rte_bit_atomic_test_and_set64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#394: FILE: lib/eal/include/rte_bitops.h:401:
+ volatile uint64_t *: __rte_bit_atomic_v_test_and_set64) \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#394: FILE: lib/eal/include/rte_bitops.h:401:
+ volatile uint64_t *: __rte_bit_atomic_v_test_and_set64) \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#405: FILE: lib/eal/include/rte_bitops.h:426:
+ volatile uint32_t *: __rte_bit_atomic_v_test_and_clear32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#405: FILE: lib/eal/include/rte_bitops.h:426:
+ volatile uint32_t *: __rte_bit_atomic_v_test_and_clear32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#406: FILE: lib/eal/include/rte_bitops.h:427:
+ uint64_t *: __rte_bit_atomic_test_and_clear64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#406: FILE: lib/eal/include/rte_bitops.h:427:
+ uint64_t *: __rte_bit_atomic_test_and_clear64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#407: FILE: lib/eal/include/rte_bitops.h:428:
+ volatile uint64_t *: __rte_bit_atomic_v_test_and_clear64) \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#407: FILE: lib/eal/include/rte_bitops.h:428:
+ volatile uint64_t *: __rte_bit_atomic_v_test_and_clear64) \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#419: FILE: lib/eal/include/rte_bitops.h:456:
+ volatile uint32_t *: __rte_bit_atomic_v_test_and_assign32, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#419: FILE: lib/eal/include/rte_bitops.h:456:
+ volatile uint32_t *: __rte_bit_atomic_v_test_and_assign32, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#420: FILE: lib/eal/include/rte_bitops.h:457:
+ uint64_t *: __rte_bit_atomic_test_and_assign64, \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#420: FILE: lib/eal/include/rte_bitops.h:457:
+ uint64_t *: __rte_bit_atomic_test_and_assign64, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxO)
#421: FILE: lib/eal/include/rte_bitops.h:458:
+ volatile uint64_t *: __rte_bit_atomic_v_test_and_assign64) \
^
ERROR:SPACING: spaces required around that ':' (ctx:OxW)
#421: FILE: lib/eal/include/rte_bitops.h:458:
+ volatile uint64_t *: __rte_bit_atomic_v_test_and_assign64) \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#454: FILE: lib/eal/include/rte_bitops.h:548:
+ __rte_bit_atomic_ ## v ## test ## size(const qualifier uint ## size ## _t *addr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#461: FILE: lib/eal/include/rte_bitops.h:553:
+ const qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr = \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#473: FILE: lib/eal/include/rte_bitops.h:562:
+ __rte_bit_atomic_ ## v ## set ## size(qualifier uint ## size ## _t *addr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#480: FILE: lib/eal/include/rte_bitops.h:567:
+ qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr = \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#492: FILE: lib/eal/include/rte_bitops.h:576:
+ __rte_bit_atomic_ ## v ## clear ## size(qualifier uint ## size ## _t *addr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#499: FILE: lib/eal/include/rte_bitops.h:581:
+ qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr = \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#511: FILE: lib/eal/include/rte_bitops.h:590:
+ __rte_bit_atomic_ ## v ## flip ## size(qualifier uint ## size ## _t *addr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#518: FILE: lib/eal/include/rte_bitops.h:595:
+ qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr = \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#531: FILE: lib/eal/include/rte_bitops.h:604:
+ __rte_bit_atomic_## v ## assign ## size(qualifier uint ## size ## _t *addr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#552: FILE: lib/eal/include/rte_bitops.h:618:
+ __rte_bit_atomic_ ## v ## test_and_set ## size(qualifier uint ## size ## _t *addr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#560: FILE: lib/eal/include/rte_bitops.h:624:
+ qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr = \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#576: FILE: lib/eal/include/rte_bitops.h:638:
+ __rte_bit_atomic_ ## v ## test_and_clear ## size(qualifier uint ## size ## _t *addr, \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#584: FILE: lib/eal/include/rte_bitops.h:644:
+ qualifier RTE_ATOMIC(uint ## size ## _t) *a_addr = \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#601: FILE: lib/eal/include/rte_bitops.h:658:
+ __rte_bit_atomic_ ## v ## test_and_assign ## size(qualifier uint ## size ## _t *addr, \
^
WARNING:LONG_LINE: line length of 104 exceeds 100 columns
#609: FILE: lib/eal/include/rte_bitops.h:664:
+ return __rte_bit_atomic_ ## v ## test_and_set ## size(addr, nr, memory_order); \
WARNING:LONG_LINE: line length of 106 exceeds 100 columns
#613: FILE: lib/eal/include/rte_bitops.h:666:
+ return __rte_bit_atomic_ ## v ## test_and_clear ## size(addr, nr, memory_order); \
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#625: FILE: lib/eal/include/rte_bitops.h:669:
+#define __RTE_GEN_BIT_ATOMIC_OPS(v, qualifier, size) \
+ __RTE_GEN_BIT_ATOMIC_TEST(v, qualifier, size) \
+ __RTE_GEN_BIT_ATOMIC_SET(v, qualifier, size) \
+ __RTE_GEN_BIT_ATOMIC_CLEAR(v, qualifier, size) \
+ __RTE_GEN_BIT_ATOMIC_ASSIGN(v, qualifier, size) \
+ __RTE_GEN_BIT_ATOMIC_TEST_AND_SET(v, qualifier, size) \
+ __RTE_GEN_BIT_ATOMIC_TEST_AND_CLEAR(v, qualifier, size) \
+ __RTE_GEN_BIT_ATOMIC_TEST_AND_ASSIGN(v, qualifier, size) \
+ __RTE_GEN_BIT_ATOMIC_FLIP(v, qualifier, size)
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#637: FILE: lib/eal/include/rte_bitops.h:679:
+#define __RTE_GEN_BIT_ATOMIC_OPS_SIZE(size) \
+ __RTE_GEN_BIT_ATOMIC_OPS(,, size) \
+ __RTE_GEN_BIT_ATOMIC_OPS(v_, volatile, size)
ERROR:SPACING: space required after that ',' (ctx:BxO)
#638: FILE: lib/eal/include/rte_bitops.h:680:
+ __RTE_GEN_BIT_ATOMIC_OPS(,, size) \
^
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#655: FILE: lib/eal/include/rte_bitops.h:1392:
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#665: FILE: lib/eal/include/rte_bitops.h:1398:
+#define __RTE_BIT_OVERLOAD_SZ_2(family, fun, c, size, arg1_type, arg1_name) \
+ __RTE_BIT_OVERLOAD_V_2(family,, fun, c, size, arg1_type, \
+ arg1_name) \
+ __RTE_BIT_OVERLOAD_V_2(family, v_, fun, c volatile, size, \
+ arg1_type, arg1_name)
ERROR:SPACING: space required after that ',' (ctx:VxO)
#666: FILE: lib/eal/include/rte_bitops.h:1399:
+ __RTE_BIT_OVERLOAD_V_2(family,, fun, c, size, arg1_type, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#673: FILE: lib/eal/include/rte_bitops.h:1404:
+#define __RTE_BIT_OVERLOAD_2(family, fun, c, arg1_type, arg1_name) \
+ __RTE_BIT_OVERLOAD_SZ_2(family, fun, c, 32, arg1_type, arg1_name) \
+ __RTE_BIT_OVERLOAD_SZ_2(family, fun, c, 64, arg1_type, arg1_name)
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#677: FILE: lib/eal/include/rte_bitops.h:1408:
+#define __RTE_BIT_OVERLOAD_V_2R(family, v, fun, c, size, ret_type, arg1_type, \
+ arg1_name) \
static inline ret_type \
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
arg1_type arg1_name) \
{ \
+ return __rte_bit_ ## family ## v ## fun ## size(addr, \
+ arg1_name); \
}
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#681: FILE: lib/eal/include/rte_bitops.h:1411:
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#691: FILE: lib/eal/include/rte_bitops.h:1418:
+#define __RTE_BIT_OVERLOAD_SZ_2R(family, fun, c, size, ret_type, arg1_type, \
+ arg1_name) \
+ __RTE_BIT_OVERLOAD_V_2R(family,, fun, c, size, ret_type, arg1_type, \
+ arg1_name) \
+ __RTE_BIT_OVERLOAD_V_2R(family, v_, fun, c volatile, \
+ size, ret_type, arg1_type, arg1_name)
ERROR:SPACING: space required after that ',' (ctx:VxO)
#693: FILE: lib/eal/include/rte_bitops.h:1420:
+ __RTE_BIT_OVERLOAD_V_2R(family,, fun, c, size, ret_type, arg1_type, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#698: FILE: lib/eal/include/rte_bitops.h:1425:
+#define __RTE_BIT_OVERLOAD_2R(family, fun, c, ret_type, arg1_type, arg1_name) \
+ __RTE_BIT_OVERLOAD_SZ_2R(family, fun, c, 32, ret_type, arg1_type, \
arg1_name) \
+ __RTE_BIT_OVERLOAD_SZ_2R(family, fun, c, 64, ret_type, arg1_type, \
arg1_name)
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#712: FILE: lib/eal/include/rte_bitops.h:1434:
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#721: FILE: lib/eal/include/rte_bitops.h:1441:
+#define __RTE_BIT_OVERLOAD_SZ_3(family, fun, c, size, arg1_type, arg1_name, \
+ arg2_type, arg2_name) \
+ __RTE_BIT_OVERLOAD_V_3(family,, fun, c, size, arg1_type, arg1_name, \
+ arg2_type, arg2_name) \
+ __RTE_BIT_OVERLOAD_V_3(family, v_, fun, c volatile, size, arg1_type, \
+ arg1_name, arg2_type, arg2_name)
ERROR:SPACING: space required after that ',' (ctx:VxO)
#723: FILE: lib/eal/include/rte_bitops.h:1443:
+ __RTE_BIT_OVERLOAD_V_3(family,, fun, c, size, arg1_type, arg1_name, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#728: FILE: lib/eal/include/rte_bitops.h:1448:
+#define __RTE_BIT_OVERLOAD_3(family, fun, c, arg1_type, arg1_name, arg2_type, \
arg2_name) \
+ __RTE_BIT_OVERLOAD_SZ_3(family, fun, c, 32, arg1_type, arg1_name, \
arg2_type, arg2_name) \
+ __RTE_BIT_OVERLOAD_SZ_3(family, fun, c, 64, arg1_type, arg1_name, \
arg2_type, arg2_name)
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#739: FILE: lib/eal/include/rte_bitops.h:1455:
+#define __RTE_BIT_OVERLOAD_V_3R(family, v, fun, c, size, ret_type, arg1_type, \
+ arg1_name, arg2_type, arg2_name) \
static inline ret_type \
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
+ arg1_type arg1_name, arg2_type arg2_name) \
{ \
+ return __rte_bit_ ## family ## v ## fun ## size(addr, \
+ arg1_name, \
+ arg2_name); \
}
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#744: FILE: lib/eal/include/rte_bitops.h:1458:
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#756: FILE: lib/eal/include/rte_bitops.h:1466:
+#define __RTE_BIT_OVERLOAD_SZ_3R(family, fun, c, size, ret_type, arg1_type, \
arg1_name, arg2_type, arg2_name) \
+ __RTE_BIT_OVERLOAD_V_3R(family,, fun, c, size, ret_type, \
+ arg1_type, arg1_name, arg2_type, arg2_name) \
+ __RTE_BIT_OVERLOAD_V_3R(family, v_, fun, c volatile, size, \
+ ret_type, arg1_type, arg1_name, arg2_type, \
+ arg2_name)
ERROR:SPACING: space required after that ',' (ctx:VxO)
#760: FILE: lib/eal/include/rte_bitops.h:1468:
+ __RTE_BIT_OVERLOAD_V_3R(family,, fun, c, size, ret_type, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#768: FILE: lib/eal/include/rte_bitops.h:1474:
+#define __RTE_BIT_OVERLOAD_3R(family, fun, c, ret_type, arg1_type, arg1_name, \
+ arg2_type, arg2_name) \
+ __RTE_BIT_OVERLOAD_SZ_3R(family, fun, c, 32, ret_type, \
+ arg1_type, arg1_name, arg2_type, arg2_name) \
+ __RTE_BIT_OVERLOAD_SZ_3R(family, fun, c, 64, ret_type, \
+ arg1_type, arg1_name, arg2_type, arg2_name)
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#780: FILE: lib/eal/include/rte_bitops.h:1484:
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#794: FILE: lib/eal/include/rte_bitops.h:1493:
+#define __RTE_BIT_OVERLOAD_SZ_4(family, fun, c, size, arg1_type, arg1_name, \
arg2_type, arg2_name, arg3_type, arg3_name) \
+ __RTE_BIT_OVERLOAD_V_4(family,, fun, c, size, arg1_type, \
+ arg1_name, arg2_type, arg2_name, arg3_type, \
+ arg3_name) \
+ __RTE_BIT_OVERLOAD_V_4(family, v_, fun, c volatile, size, \
+ arg1_type, arg1_name, arg2_type, arg2_name, \
+ arg3_type, arg3_name)
ERROR:SPACING: space required after that ',' (ctx:VxO)
#802: FILE: lib/eal/include/rte_bitops.h:1495:
+ __RTE_BIT_OVERLOAD_V_4(family,, fun, c, size, arg1_type, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#809: FILE: lib/eal/include/rte_bitops.h:1502:
+#define __RTE_BIT_OVERLOAD_4(family, fun, c, arg1_type, arg1_name, arg2_type, \
+ arg2_name, arg3_type, arg3_name) \
+ __RTE_BIT_OVERLOAD_SZ_4(family, fun, c, 32, arg1_type, \
+ arg1_name, arg2_type, arg2_name, arg3_type, \
+ arg3_name) \
+ __RTE_BIT_OVERLOAD_SZ_4(family, fun, c, 64, arg1_type, \
+ arg1_name, arg2_type, arg2_name, arg3_type, \
+ arg3_name)
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#818: FILE: lib/eal/include/rte_bitops.h:1511:
+#define __RTE_BIT_OVERLOAD_V_4R(family, v, fun, c, size, ret_type, arg1_type, \
+ arg1_name, arg2_type, arg2_name, arg3_type, \
+ arg3_name) \
static inline ret_type \
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
+ arg1_type arg1_name, arg2_type arg2_name, \
+ arg3_type arg3_name) \
{ \
+ return __rte_bit_ ## family ## v ## fun ## size(addr, \
+ arg1_name, \
+ arg2_name, \
+ arg3_name); \
}
ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#824: FILE: lib/eal/include/rte_bitops.h:1515:
+ rte_bit_ ## family ## fun(c uint ## size ## _t *addr, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#839: FILE: lib/eal/include/rte_bitops.h:1525:
+#define __RTE_BIT_OVERLOAD_SZ_4R(family, fun, c, size, ret_type, arg1_type, \
arg1_name, arg2_type, arg2_name, arg3_type, \
arg3_name) \
+ __RTE_BIT_OVERLOAD_V_4R(family,, fun, c, size, ret_type, arg1_type, \
+ arg1_name, arg2_type, arg2_name, arg3_type, \
+ arg3_name) \
+ __RTE_BIT_OVERLOAD_V_4R(family, v_, fun, c volatile, size, \
+ ret_type, arg1_type, arg1_name, arg2_type, \
+ arg2_name, arg3_type, arg3_name)
ERROR:SPACING: space required after that ',' (ctx:VxO)
#853: FILE: lib/eal/include/rte_bitops.h:1528:
+ __RTE_BIT_OVERLOAD_V_4R(family,, fun, c, size, ret_type, arg1_type, \
^
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses
#860: FILE: lib/eal/include/rte_bitops.h:1535:
+#define __RTE_BIT_OVERLOAD_4R(family, fun, c, ret_type, arg1_type, arg1_name, \
+ arg2_type, arg2_name, arg3_type, arg3_name) \
+ __RTE_BIT_OVERLOAD_SZ_4R(family, fun, c, 32, ret_type, \
+ arg1_type, arg1_name, arg2_type, arg2_name, \
+ arg3_type, arg3_name) \
+ __RTE_BIT_OVERLOAD_SZ_4R(family, fun, c, 64, ret_type, \
+ arg1_type, arg1_name, arg2_type, arg2_name, \
+ arg3_type, arg3_name)
ERROR:SPACING: space required after that ',' (ctx:VxO)
#870: FILE: lib/eal/include/rte_bitops.h:1545:
+__RTE_BIT_OVERLOAD_2(, set,, unsigned int, nr)
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#871: FILE: lib/eal/include/rte_bitops.h:1546:
+__RTE_BIT_OVERLOAD_2(, clear,, unsigned int, nr)
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#872: FILE: lib/eal/include/rte_bitops.h:1547:
+__RTE_BIT_OVERLOAD_3(, assign,, unsigned int, nr, bool, value)
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#873: FILE: lib/eal/include/rte_bitops.h:1548:
+__RTE_BIT_OVERLOAD_2(, flip,, unsigned int, nr)
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#880: FILE: lib/eal/include/rte_bitops.h:1552:
+__RTE_BIT_OVERLOAD_3(atomic_, set,, unsigned int, nr, int, memory_order)
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#881: FILE: lib/eal/include/rte_bitops.h:1553:
+__RTE_BIT_OVERLOAD_3(atomic_, clear,, unsigned int, nr, int, memory_order)
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#882: FILE: lib/eal/include/rte_bitops.h:1554:
+__RTE_BIT_OVERLOAD_4(atomic_, assign,, unsigned int, nr, bool, value,
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#886: FILE: lib/eal/include/rte_bitops.h:1556:
+__RTE_BIT_OVERLOAD_3(atomic_, flip,, unsigned int, nr, int, memory_order)
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#887: FILE: lib/eal/include/rte_bitops.h:1557:
+__RTE_BIT_OVERLOAD_3R(atomic_, test_and_set,, bool, unsigned int, nr,
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#890: FILE: lib/eal/include/rte_bitops.h:1559:
+__RTE_BIT_OVERLOAD_3R(atomic_, test_and_clear,, bool, unsigned int, nr,
^
ERROR:SPACING: space required after that ',' (ctx:VxO)
#893: FILE: lib/eal/include/rte_bitops.h:1561:
+__RTE_BIT_OVERLOAD_4R(atomic_, test_and_assign,, bool, unsigned int, nr,
^
total: 156 errors, 2 warnings, 713 lines checked
next prev parent reply other threads:[~2024-08-09 10:09 UTC|newest]
Thread overview: 104+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20240809095829.589396-6-mattias.ronnblom@ericsson.com>
2024-08-09 9:43 ` |SUCCESS| pw143046-143048 " qemudev
2024-08-09 9:47 ` qemudev
2024-08-09 10:09 ` checkpatch [this message]
2024-08-09 11:02 ` |FAILURE| pw143048 " 0-day Robot
2024-08-09 13:22 ` |SUCCESS| pw143046-143048 [PATCH] [v2,5/5] eal: extend bitops to han dpdklab
2024-08-09 13:22 ` dpdklab
2024-08-09 13:25 ` dpdklab
2024-08-09 13:26 ` |PENDING| " dpdklab
2024-08-09 13:30 ` |SUCCESS| " dpdklab
2024-08-09 13:31 ` dpdklab
2024-08-09 13:34 ` dpdklab
2024-08-09 13:38 ` dpdklab
2024-08-09 13:42 ` dpdklab
2024-08-09 13:45 ` dpdklab
2024-08-09 13:53 ` dpdklab
2024-08-09 14:09 ` dpdklab
2024-08-09 14:29 ` dpdklab
2024-08-09 14:33 ` dpdklab
2024-08-09 14:37 ` dpdklab
2024-08-09 14:54 ` |WARNING| " dpdklab
2024-08-09 14:57 ` |SUCCESS| " dpdklab
2024-08-09 15:00 ` |WARNING| " dpdklab
2024-08-09 15:05 ` dpdklab
2024-08-09 15:05 ` dpdklab
2024-08-09 15:05 ` dpdklab
2024-08-09 15:06 ` dpdklab
2024-08-09 15:12 ` dpdklab
2024-08-09 15:20 ` dpdklab
2024-08-09 15:22 ` dpdklab
2024-08-09 15:22 ` dpdklab
2024-08-09 15:29 ` dpdklab
2024-08-09 15:37 ` dpdklab
2024-08-09 15:43 ` dpdklab
2024-08-09 15:50 ` dpdklab
2024-08-09 18:34 ` |PENDING| " dpdklab
2024-08-09 18:38 ` dpdklab
2024-08-09 18:52 ` dpdklab
2024-08-09 19:04 ` |SUCCESS| " dpdklab
2024-08-09 19:11 ` |PENDING| " dpdklab
2024-08-09 19:54 ` dpdklab
2024-08-09 19:57 ` dpdklab
2024-08-09 20:06 ` dpdklab
2024-08-09 20:19 ` dpdklab
2024-08-09 20:24 ` dpdklab
2024-08-09 20:26 ` dpdklab
2024-08-09 20:28 ` dpdklab
2024-08-09 20:28 ` dpdklab
2024-08-09 20:31 ` dpdklab
2024-08-09 20:33 ` dpdklab
2024-08-09 20:34 ` dpdklab
2024-08-09 20:34 ` dpdklab
2024-08-09 20:35 ` |FAILURE| " dpdklab
2024-08-09 20:37 ` |PENDING| " dpdklab
2024-08-09 20:37 ` dpdklab
2024-08-09 20:38 ` |FAILURE| " dpdklab
2024-08-09 20:39 ` |PENDING| " dpdklab
2024-08-09 20:42 ` dpdklab
2024-08-09 20:43 ` dpdklab
2024-08-09 20:45 ` |FAILURE| " dpdklab
2024-08-09 20:49 ` |PENDING| " dpdklab
2024-08-09 20:54 ` |FAILURE| " dpdklab
2024-08-09 20:56 ` dpdklab
2024-08-09 20:57 ` |PENDING| " dpdklab
2024-08-09 20:57 ` |FAILURE| " dpdklab
2024-08-09 20:58 ` dpdklab
2024-08-09 21:00 ` |PENDING| " dpdklab
2024-08-09 21:01 ` dpdklab
2024-08-09 21:01 ` dpdklab
2024-08-09 21:05 ` dpdklab
2024-08-09 21:05 ` |FAILURE| " dpdklab
2024-08-09 21:05 ` |PENDING| " dpdklab
2024-08-09 21:07 ` |FAILURE| " dpdklab
2024-08-09 21:09 ` |PENDING| " dpdklab
2024-08-09 21:10 ` dpdklab
2024-08-09 21:10 ` dpdklab
2024-08-09 21:13 ` dpdklab
2024-08-09 21:15 ` dpdklab
2024-08-09 21:15 ` dpdklab
2024-08-09 21:16 ` dpdklab
2024-08-09 21:16 ` |FAILURE| " dpdklab
2024-08-09 21:16 ` |PENDING| " dpdklab
2024-08-09 21:17 ` |FAILURE| " dpdklab
2024-08-09 21:17 ` |PENDING| " dpdklab
2024-08-09 21:17 ` dpdklab
2024-08-09 21:18 ` dpdklab
2024-08-09 21:19 ` dpdklab
2024-08-09 21:19 ` dpdklab
2024-08-09 21:23 ` |FAILURE| " dpdklab
2024-08-09 21:25 ` |PENDING| " dpdklab
2024-08-09 21:25 ` |FAILURE| " dpdklab
2024-08-09 21:26 ` dpdklab
2024-08-09 21:26 ` |PENDING| " dpdklab
2024-08-09 21:35 ` dpdklab
2024-08-09 21:37 ` dpdklab
2024-08-09 21:43 ` dpdklab
2024-08-09 21:46 ` dpdklab
2024-08-09 21:46 ` dpdklab
2024-08-09 21:53 ` |SUCCESS| " dpdklab
2024-08-09 22:10 ` |PENDING| " dpdklab
2024-08-09 22:13 ` dpdklab
2024-08-09 22:17 ` dpdklab
2024-08-09 23:32 ` dpdklab
2024-08-10 3:04 ` |SUCCESS| " dpdklab
2024-08-10 16:16 ` |FAILURE| " 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=20240809100911.14F2F123EF6@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).