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 2180C46D1F for ; Thu, 14 Aug 2025 05:04:50 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1A5EA4026D; Thu, 14 Aug 2025 05:04:50 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 446DC4026D for ; Thu, 14 Aug 2025 05:04:49 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id F2AF2123FC6; Thu, 14 Aug 2025 05:02:36 +0200 (CEST) Subject: |WARNING| pw155853 [PATCH v5 2/2] net/zxdh: add support flow director ops In-Reply-To: <20250814025248.2194951-3-chen.bingbin@zte.com.cn> References: <20250814025248.2194951-3-chen.bingbin@zte.com.cn> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Bingbin Chen Message-Id: <20250814030236.F2AF2123FC6@dpdk.org> Date: Thu, 14 Aug 2025 05:02:36 +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/155853 _coding style issues_ WARNING:TYPO_SPELLING: 'stdio' may be misspelled - perhaps 'studio'? #333: FILE: drivers/net/zxdh/zxdh_flow.c:6: +#include CHECK:MACRO_ARG_REUSE: Macro argument reuse 'buf_size' - possible side-effects? #375: FILE: drivers/net/zxdh/zxdh_flow.c:48: +#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? #375: FILE: drivers/net/zxdh/zxdh_flow.c:48: +#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: #453: FILE: drivers/net/zxdh/zxdh_flow.c:126: + "L3 dstip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT") ", CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #453: FILE: drivers/net/zxdh/zxdh_flow.c:126: + "L3 dstip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT") ", CHECK:CAMELCASE: Avoid CamelCase: #457: FILE: drivers/net/zxdh/zxdh_flow.c:130: + IPv6_BYTES(key->dst_ip)); CHECK:CONCATENATED_STRING: Concatenated strings should use spaces between elements #459: FILE: drivers/net/zxdh/zxdh_flow.c:132: + " srcip=0x%08x 0x%08x 0x%08x 0x%08x("IPv6_BYTES_FMT") ", CHECK:CAMELCASE: Avoid CamelCase: #2382: FILE: drivers/net/zxdh/zxdh_flow.h:32: +#ifndef IPv4_BYTES CHECK:CAMELCASE: Avoid CamelCase: #2383: FILE: drivers/net/zxdh/zxdh_flow.h:33: +#define IPv4_BYTES_FMT "%" PRIu8 ".%" PRIu8 ".%" PRIu8 ".%" PRIu8 CHECK:CAMELCASE: Avoid CamelCase: #2383: 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 #2384: 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? #2384: 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 #2394: 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? #2394: 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, 3025 lines checked