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 41CA446ACD for ; Wed, 2 Jul 2025 09:43:43 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3AE6240269; Wed, 2 Jul 2025 09:43:43 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 2CF5A40269 for ; Wed, 2 Jul 2025 09:43:42 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id EEEE81252FD; Wed, 2 Jul 2025 09:42:20 +0200 (CEST) Subject: |WARNING| pw154951 [PATCH v3 2/2] net/zxdh: add support flow director ops In-Reply-To: <20250702073453.113779-3-chen.bingbin@zte.com.cn> References: <20250702073453.113779-3-chen.bingbin@zte.com.cn> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Bingbin Chen Message-Id: <20250702074220.EEEE81252FD@dpdk.org> Date: Wed, 2 Jul 2025 09:42:20 +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/154951 _coding style issues_ WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'? #374: FILE: drivers/net/zxdh/zxdh_flow.c:6: +#include CHECK:MACRO_ARG_REUSE: Macro argument reuse 'buf_size' - possible side-effects? #415: FILE: drivers/net/zxdh/zxdh_flow.c:47: +#define MKDUMPSTR(buf, buf_size, cur_len, ...) \ +do { \ + if ((cur_len) >= (buf_size)) \ + break; \ + (cur_len) += snprintf((buf) + (cur_len), (buf_size) - (cur_len), __VA_ARGS__); \ +} while (0) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'cur_len' - possible side-effects? #415: FILE: drivers/net/zxdh/zxdh_flow.c:47: +#define MKDUMPSTR(buf, buf_size, cur_len, ...) \ +do { \ + if ((cur_len) >= (buf_size)) \ + break; \ + (cur_len) += snprintf((buf) + (cur_len), (buf_size) - (cur_len), __VA_ARGS__); \ +} while (0) CHECK:CAMELCASE: Avoid CamelCase: #493: FILE: drivers/net/zxdh/zxdh_flow.c:125: + "L3 dstip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT") ", CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #493: FILE: drivers/net/zxdh/zxdh_flow.c:125: + "L3 dstip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT") ", CHECK:CAMELCASE: Avoid CamelCase: #497: FILE: drivers/net/zxdh/zxdh_flow.c:129: + IPv6_BYTES(key->dst_ip)); CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #499: FILE: drivers/net/zxdh/zxdh_flow.c:131: + " srcip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT") ", CHECK:CAMELCASE: Avoid CamelCase: #2409: FILE: drivers/net/zxdh/zxdh_flow.h:32: +#ifndef IPv4_BYTES CHECK:CAMELCASE: Avoid CamelCase: #2410: FILE: drivers/net/zxdh/zxdh_flow.h:33: +#define IPv4_BYTES_FMT "%" PRIu8 ".%" PRIu8 ".%" PRIu8 ".%" PRIu8 CHECK:CAMELCASE: Avoid CamelCase: #2410: FILE: drivers/net/zxdh/zxdh_flow.h:33: +#define IPv4_BYTES_FMT "%" PRIu8 ".%" PRIu8 ".%" PRIu8 ".%" PRIu8 ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #2411: FILE: drivers/net/zxdh/zxdh_flow.h:34: +#define IPv4_BYTES(addr) \ + (uint8_t)(((addr) >> 24) & 0xFF),\ + (uint8_t)(((addr) >> 16) & 0xFF),\ + (uint8_t)(((addr) >> 8) & 0xFF),\ + (uint8_t)((addr) & 0xFF) CHECK:MACRO_ARG_REUSE: Macro argument reuse 'addr' - possible side-effects? #2411: FILE: drivers/net/zxdh/zxdh_flow.h:34: +#define IPv4_BYTES(addr) \ + (uint8_t)(((addr) >> 24) & 0xFF),\ + (uint8_t)(((addr) >> 16) & 0xFF),\ + (uint8_t)(((addr) >> 8) & 0xFF),\ + (uint8_t)((addr) & 0xFF) ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in parentheses #2421: FILE: drivers/net/zxdh/zxdh_flow.h:44: +#define IPv6_BYTES(addr) \ + addr[0], addr[1], addr[2], addr[3], addr[4], addr[5], addr[6], addr[7], \ + addr[8], addr[9], addr[10], addr[11], addr[12], addr[13], addr[14], addr[15] CHECK:MACRO_ARG_REUSE: Macro argument reuse 'addr' - possible side-effects? #2421: FILE: drivers/net/zxdh/zxdh_flow.h:44: +#define IPv6_BYTES(addr) \ + addr[0], addr[1], addr[2], addr[3], addr[4], addr[5], addr[6], addr[7], \ + addr[8], addr[9], addr[10], addr[11], addr[12], addr[13], addr[14], addr[15] total: 2 errors, 1 warnings, 11 checks, 3045 lines checked