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 906D746CC7 for ; Fri, 8 Aug 2025 09:23:17 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 744B540150; Fri, 8 Aug 2025 09:23:17 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 55BB740150 for ; Fri, 8 Aug 2025 09:23:16 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 03B6712522F; Fri, 8 Aug 2025 09:21:10 +0200 (CEST) Subject: |WARNING| pw155741 [PATCH v4 2/2] net/zxdh: add support flow director ops In-Reply-To: <20250808071048.1906724-3-wang.junlong1@zte.com.cn> References: <20250808071048.1906724-3-wang.junlong1@zte.com.cn> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Junlong Wang Message-Id: <20250808072111.03B6712522F@dpdk.org> Date: Fri, 8 Aug 2025 09:21:10 +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/155741 _coding style issues_ WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'? #336: FILE: drivers/net/zxdh/zxdh_flow.c:6: +#include CHECK:MACRO_ARG_REUSE: Macro argument reuse 'buf_size' - possible side-effects? #377: 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? #377: 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: #455: 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 #455: 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: #459: FILE: drivers/net/zxdh/zxdh_flow.c:129: + IPv6_BYTES(key->dst_ip)); CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #461: FILE: drivers/net/zxdh/zxdh_flow.c:131: + " srcip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT") ", CHECK:CAMELCASE: Avoid CamelCase: #2368: FILE: drivers/net/zxdh/zxdh_flow.h:32: +#ifndef IPv4_BYTES CHECK:CAMELCASE: Avoid CamelCase: #2369: FILE: drivers/net/zxdh/zxdh_flow.h:33: +#define IPv4_BYTES_FMT "%" PRIu8 ".%" PRIu8 ".%" PRIu8 ".%" PRIu8 CHECK:CAMELCASE: Avoid CamelCase: #2369: 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 #2370: 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? #2370: 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 #2380: 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? #2380: 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, 3005 lines checked