From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 10EB4A04C0 for ; Tue, 29 Sep 2020 04:04:13 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1AEAF1D659; Tue, 29 Sep 2020 04:04:10 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 1017) id 976C51D5A5; Tue, 29 Sep 2020 04:04:09 +0200 (CEST) In-Reply-To: <20200929015632.109364-4-simei.su@intel.com> References: <20200929015632.109364-4-simei.su@intel.com> To: test-report@dpdk.org Cc: Simei Su Message-Id: <20200929020409.976C51D5A5@dpdk.org> Date: Tue, 29 Sep 2020 04:04:09 +0200 (CEST) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw79127 [PATCH v2 3/4] net/ice: support ACL filter in DCF X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.15 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 Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/79127 _coding style issues_ CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_mask->hdr.src_addr == UINT32_MAX' #845: FILE: drivers/net/ice/ice_acl_filter.c:745: + if ((ipv4_mask->hdr.src_addr == UINT32_MAX) || + (ipv4_mask->hdr.dst_addr == UINT32_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_mask->hdr.dst_addr == UINT32_MAX' #845: FILE: drivers/net/ice/ice_acl_filter.c:745: + if ((ipv4_mask->hdr.src_addr == UINT32_MAX) || + (ipv4_mask->hdr.dst_addr == UINT32_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_spec->hdr.src_addr != 0' #854: FILE: drivers/net/ice/ice_acl_filter.c:754: + if ((ipv4_spec->hdr.src_addr != 0) && + (ipv4_mask->hdr.src_addr != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_mask->hdr.src_addr != 0' #854: FILE: drivers/net/ice/ice_acl_filter.c:754: + if ((ipv4_spec->hdr.src_addr != 0) && + (ipv4_mask->hdr.src_addr != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_spec->hdr.dst_addr != 0' #864: FILE: drivers/net/ice/ice_acl_filter.c:764: + if ((ipv4_spec->hdr.dst_addr != 0) && + (ipv4_mask->hdr.dst_addr != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'ipv4_mask->hdr.dst_addr != 0' #864: FILE: drivers/net/ice/ice_acl_filter.c:764: + if ((ipv4_spec->hdr.dst_addr != 0) && + (ipv4_mask->hdr.dst_addr != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_mask->hdr.src_port == UINT16_MAX' #897: FILE: drivers/net/ice/ice_acl_filter.c:797: + if ((tcp_mask->hdr.src_port == UINT16_MAX) || + (tcp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_mask->hdr.dst_port == UINT16_MAX' #897: FILE: drivers/net/ice/ice_acl_filter.c:797: + if ((tcp_mask->hdr.src_port == UINT16_MAX) || + (tcp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_mask->hdr.src_port == UINT16_MAX' #906: FILE: drivers/net/ice/ice_acl_filter.c:806: + if ((tcp_mask->hdr.src_port == UINT16_MAX) || + (tcp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_mask->hdr.dst_port == UINT16_MAX' #906: FILE: drivers/net/ice/ice_acl_filter.c:806: + if ((tcp_mask->hdr.src_port == UINT16_MAX) || + (tcp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #915: FILE: drivers/net/ice/ice_acl_filter.c:815: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (tcp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_spec->hdr.src_port != 0' #915: FILE: drivers/net/ice/ice_acl_filter.c:815: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (tcp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #924: FILE: drivers/net/ice/ice_acl_filter.c:824: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (tcp_spec->hdr.dst_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'tcp_spec->hdr.dst_port != 0' #924: FILE: drivers/net/ice/ice_acl_filter.c:824: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (tcp_spec->hdr.dst_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'udp_mask->hdr.src_port == UINT16_MAX' #953: FILE: drivers/net/ice/ice_acl_filter.c:853: + if ((udp_mask->hdr.src_port == UINT16_MAX) || + (udp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'udp_mask->hdr.dst_port == UINT16_MAX' #953: FILE: drivers/net/ice/ice_acl_filter.c:853: + if ((udp_mask->hdr.src_port == UINT16_MAX) || + (udp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #962: FILE: drivers/net/ice/ice_acl_filter.c:862: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (udp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'udp_spec->hdr.src_port != 0' #962: FILE: drivers/net/ice/ice_acl_filter.c:862: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (udp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #971: FILE: drivers/net/ice/ice_acl_filter.c:871: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (udp_spec->hdr.dst_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'udp_spec->hdr.dst_port != 0' #971: FILE: drivers/net/ice/ice_acl_filter.c:871: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (udp_spec->hdr.dst_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'sctp_mask->hdr.src_port == UINT16_MAX' #990: FILE: drivers/net/ice/ice_acl_filter.c:890: + if ((sctp_mask->hdr.src_port == UINT16_MAX) || + (sctp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'sctp_mask->hdr.dst_port == UINT16_MAX' #990: FILE: drivers/net/ice/ice_acl_filter.c:890: + if ((sctp_mask->hdr.src_port == UINT16_MAX) || + (sctp_mask->hdr.dst_port == UINT16_MAX)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #999: FILE: drivers/net/ice/ice_acl_filter.c:899: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (sctp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'sctp_spec->hdr.src_port != 0' #999: FILE: drivers/net/ice/ice_acl_filter.c:899: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (sctp_spec->hdr.src_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'l3 == RTE_FLOW_ITEM_TYPE_IPV4' #1008: FILE: drivers/net/ice/ice_acl_filter.c:908: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (sctp_spec->hdr.dst_port != 0)) { CHECK:UNNECESSARY_PARENTHESES: Unnecessary parentheses around 'sctp_spec->hdr.dst_port != 0' #1008: FILE: drivers/net/ice/ice_acl_filter.c:908: + if ((l3 == RTE_FLOW_ITEM_TYPE_IPV4) && + (sctp_spec->hdr.dst_port != 0)) { total: 0 errors, 0 warnings, 26 checks, 1157 lines checked