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 2213245778 for ; Fri, 9 Aug 2024 12:09:14 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1D7BF42EE4; Fri, 9 Aug 2024 12:09:14 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 297A842D78 for ; Fri, 9 Aug 2024 12:09:11 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 14F2F123EF6; Fri, 9 Aug 2024 12:09:11 +0200 (CEST) Subject: |WARNING| pw143048 [PATCH v2 5/5] eal: extend bitops to handle volatile pointers In-Reply-To: <20240809095829.589396-6-mattias.ronnblom@ericsson.com> References: <20240809095829.589396-6-mattias.ronnblom@ericsson.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: =?utf-8?q?Mattias_R=C3=B6nnblom?= Message-Id: <20240809100911.14F2F123EF6@dpdk.org> Date: Fri, 9 Aug 2024 12:09:11 +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/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