diff mbox series

[v2,65/71] net/hinic: replace use of fixed size rte_memcpy

Message ID	20240301171707.95242-66-stephen@networkplumber.org (mailing list archive)
State	Superseded
Delegated to:	Thomas Monjalon
Headers	From: Stephen Hemminger <stephen@networkplumber.org> To: dev@dpdk.org Cc: Stephen Hemminger <stephen@networkplumber.org>, Ziyang Xuan <xuanziyang2@huawei.com>, Xiaoyun Wang <cloud.wangxiaoyun@huawei.com>, Guoyang Zhou <zhouguoyang@huawei.com> Subject: [PATCH v2 65/71] net/hinic: replace use of fixed size rte_memcpy Date: Fri, 1 Mar 2024 09:16:01 -0800 Message-ID: <20240301171707.95242-66-stephen@networkplumber.org> In-Reply-To: <20240301171707.95242-1-stephen@networkplumber.org> References: <20240229225936.483472-1-stephen@networkplumber.org> <20240301171707.95242-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: dev-bounces@dpdk.org
Series	replace use of fixed size rte_mempcy \| [v2,00/71] replace use of fixed size rte_mempcy [v2,01/71] cocci/rte_memcpy: add script to eliminate fixed size rte_memcpy [v2,02/71] eal: replace use of fixed size rte_memcpy [v2,03/71] ethdev: replace use of fixed size rte_memcpy [v2,04/71] eventdev: replace use of fixed size rte_memcpy [v2,05/71] cryptodev: replace use of fixed size rte_memcpy [v2,06/71] ip_frag: replace use of fixed size rte_memcpy [v2,07/71] net: replace use of fixed size rte_memcpy [v2,08/71] lpm: replace use of fixed size rte_memcpy [v2,09/71] node: replace use of fixed size rte_memcpy [v2,10/71] pdcp: replace use of fixed size rte_memcpy [v2,11/71] pipeline: replace use of fixed size rte_memcpy [v2,12/71] rib: replace use of fixed size rte_memcpy [v2,13/71] security: replace use of fixed size rte_memcpy [v2,14/71] net/mlx5: replace use of fixed size rte_memcpy [v2,15/71] net/nfp: replace use of fixed size rte_memcpy [v2,16/71] net/ngbe: replace use of fixed size rte_memcpy [v2,17/71] net/null: replace use of fixed size rte_memcpy [v2,18/71] net/pcap: replace use of fixed size rte_memcpy [v2,19/71] net/sfc: replace use of fixed size rte_memcpy [v2,20/71] net/tap: replace use of fixed size rte_memcpy [v2,21/71] net/txgbe: replace use of fixed size rte_memcpy [v2,22/71] raw/ifpga: replace use of fixed size rte_memcpy [v2,23/71] raw/skeleton: replace use of fixed size rte_memcpy [v2,24/71] net/hns3: replace use of fixed size rte_memcpy [v2,25/71] net/i40e: replace use of fixed size rte_memcpy [v2,26/71] net/iavf: replace use of fixed size rte_memcpy [v2,27/71] net/ice: replace use of fixed size rte_memcpy [v2,28/71] net/idpf: replace use of fixed size rte_memcpy [v2,29/71] net/ipn3ke: replace use of fixed size rte_memcpy [v2,30/71] net/ixgbe: replace use of fixed size rte_memcpy [v2,31/71] net/memif: replace use of fixed size rte_memcpy [v2,32/71] net/qede: replace use of fixed size rte_memcpy [v2,33/71] baseband/acc: replace use of fixed size rte_memcpy [v2,34/71] baseband/la12xx: replace use of fixed size rte_memcpy [v2,35/71] common/idpf: replace use of fixed size rte_memcpy [v2,36/71] common/qat: replace use of fixed size rte_memcpy [v2,37/71] compress/qat: replace use of fixed size rte_memcpy [v2,38/71] crypto/ccp: replace use of fixed size rte_memcpy [v2,39/71] crypto/cnxk: replace use of fixed size rte_memcpy [v2,40/71] crypto/dpaa_sec: replace use of fixed size rte_memcpy [v2,41/71] crypto/ipsec_mb: replace use of fixed size rte_memcpy [v2,42/71] crypto/qat: replace use of fixed size rte_memcpy [v2,43/71] crypto/scheduler: replace use of fixed size rte_memcpy [v2,44/71] event/cnxk: replace use of fixed size rte_memcpy [v2,45/71] event/dlb2: replace use of fixed size rte_memcpy [v2,46/71] event/dpaa2: replace use of fixed size rte_memcpy [v2,47/71] event/octeontx: replace use of fixed size rte_memcpy [v2,48/71] mempool/dpaa: replace use of fixed size rte_memcpy [v2,49/71] mempool/dpaa2: replace use of fixed size rte_memcpy [v2,50/71] ml/cnxk: replace use of fixed size rte_memcpy [v2,51/71] net/af_xdp: replace use of fixed size rte_memcpy [v2,52/71] net/avp: replace use of fixed size rte_memcpy [v2,53/71] net/axgbe: replace use of fixed size rte_memcpy [v2,54/71] net/bnx2x: replace use of fixed size rte_memcpy [v2,55/71] net/bnxt: replace use of fixed size rte_memcpy [v2,56/71] net/bonding: replace use of fixed size rte_memcpy [v2,57/71] net/cnxk: replace use of fixed size rte_memcpy [v2,58/71] net/cpfl: replace use of fixed size rte_memcpy [v2,59/71] net/cxgbe: replace use of fixed size rte_memcpy [v2,60/71] net/dpaa2: replace use of fixed size rte_memcpy [v2,61/71] net/e1000: replace use of fixed size rte_memcpy [v2,62/71] net/enic: replace use of fixed size rte_memcpy [v2,63/71] net/failsafe: replace use of fixed size rte_memcpy [v2,64/71] net/gve/base: replace use of fixed size rte_memcpy [v2,65/71] net/hinic: replace use of fixed size rte_memcpy [v2,66/71] net/mvpp2: replace use of fixed size rte_memcpy [v2,67/71] app/test-pmd: replace use of fixed size rte_memcpy [v2,68/71] app/graph: replace use of fixed size rte_memcpy [v2,69/71] app/test-eventdev: replace use of fixed size rte_memcpy [v2,70/71] app/test: replace use of fixed size rte_memcpy [v2,71/71] examples: replace use of fixed size rte_memcpy

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Stephen Hemminger March 1, 2024, 5:16 p.m. UTC

  Automatically generated by devtools/cocci/rte_memcpy.cocci

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/hinic/hinic_pmd_flow.c | 40 ++++++++++++++----------------
 1 file changed, 18 insertions(+), 22 deletions(-)

diff mbox series

Patch

diff --git a/drivers/net/hinic/hinic_pmd_flow.c b/drivers/net/hinic/hinic_pmd_flow.c
index d1a564a16303..cb61c989fd36 100644
--- a/drivers/net/hinic/hinic_pmd_flow.c
+++ b/drivers/net/hinic/hinic_pmd_flow.c
@@ -983,8 +983,7 @@  static int hinic_normal_item_check_ip(const struct rte_flow_item **in_out_item,
 		}
 
 		ipv6_spec = (const struct rte_flow_item_ipv6 *)item->spec;
-		rte_memcpy(rule->hinic_fdir.dst_ipv6,
-			   ipv6_spec->hdr.dst_addr, 16);
+		memcpy(rule->hinic_fdir.dst_ipv6, ipv6_spec->hdr.dst_addr, 16);
 
 		/*
 		 * Check if the next not void item is TCP or UDP or ICMP.
@@ -2193,8 +2192,8 @@  static int hinic_add_del_ntuple_filter(struct rte_eth_dev *dev,
 				sizeof(struct hinic_5tuple_filter), 0);
 		if (filter == NULL)
 			return -ENOMEM;
-		rte_memcpy(&filter->filter_info, &filter_5tuple,
-				sizeof(struct hinic_5tuple_filter_info));
+		memcpy(&filter->filter_info, &filter_5tuple,
+		       sizeof(struct hinic_5tuple_filter_info));
 		filter->queue = ntuple_filter->queue;
 
 		filter_info->qid = ntuple_filter->queue;
@@ -2912,8 +2911,7 @@  static int hinic_add_del_tcam_fdir_filter(struct rte_eth_dev *dev,
 				sizeof(struct hinic_tcam_filter), 0);
 		if (tcam_filter == NULL)
 			return -ENOMEM;
-		(void)rte_memcpy(&tcam_filter->tcam_key,
-				 &tcam_key, sizeof(struct tag_tcam_key));
+		memcpy(&tcam_filter->tcam_key, &tcam_key, sizeof(struct tag_tcam_key));
 		tcam_filter->queue = fdir_tcam_rule.data.qid;
 
 		ret = hinic_add_tcam_filter(dev, tcam_filter, &fdir_tcam_rule);
@@ -2990,9 +2988,9 @@  static struct rte_flow *hinic_flow_create(struct rte_eth_dev *dev,
 							&ntuple_filter, FALSE);
 				goto out;
 			}
-			rte_memcpy(&ntuple_filter_ptr->filter_info,
-				   &ntuple_filter,
-				   sizeof(struct rte_eth_ntuple_filter));
+			memcpy(&ntuple_filter_ptr->filter_info,
+			       &ntuple_filter,
+			       sizeof(struct rte_eth_ntuple_filter));
 			TAILQ_INSERT_TAIL(&nic_dev->filter_ntuple_list,
 			ntuple_filter_ptr, entries);
 			flow->rule = ntuple_filter_ptr;
@@ -3022,9 +3020,9 @@  static struct rte_flow *hinic_flow_create(struct rte_eth_dev *dev,
 						&ethertype_filter, FALSE);
 				goto out;
 			}
-			rte_memcpy(&ethertype_filter_ptr->filter_info,
-				&ethertype_filter,
-				sizeof(struct rte_eth_ethertype_filter));
+			memcpy(&ethertype_filter_ptr->filter_info,
+			       &ethertype_filter,
+			       sizeof(struct rte_eth_ethertype_filter));
 			TAILQ_INSERT_TAIL(&nic_dev->filter_ethertype_list,
 				ethertype_filter_ptr, entries);
 			flow->rule = ethertype_filter_ptr;
@@ -3065,8 +3063,8 @@  static struct rte_flow *hinic_flow_create(struct rte_eth_dev *dev,
 
 				goto out;
 			}
-			rte_memcpy(&fdir_rule_ptr->filter_info, &fdir_rule,
-				sizeof(struct hinic_fdir_rule));
+			memcpy(&fdir_rule_ptr->filter_info, &fdir_rule,
+			       sizeof(struct hinic_fdir_rule));
 			TAILQ_INSERT_TAIL(&nic_dev->filter_fdir_rule_list,
 				fdir_rule_ptr, entries);
 			flow->rule = fdir_rule_ptr;
@@ -3109,8 +3107,8 @@  static int hinic_flow_destroy(struct rte_eth_dev *dev, struct rte_flow *flow,
 	case RTE_ETH_FILTER_NTUPLE:
 		ntuple_filter_ptr = (struct hinic_ntuple_filter_ele *)
 					pmd_flow->rule;
-		rte_memcpy(&ntuple_filter, &ntuple_filter_ptr->filter_info,
-			sizeof(struct rte_eth_ntuple_filter));
+		memcpy(&ntuple_filter, &ntuple_filter_ptr->filter_info,
+		       sizeof(struct rte_eth_ntuple_filter));
 		ret = hinic_add_del_ntuple_filter(dev, &ntuple_filter, FALSE);
 		if (!ret) {
 			TAILQ_REMOVE(&nic_dev->filter_ntuple_list,
@@ -3121,9 +3119,8 @@  static int hinic_flow_destroy(struct rte_eth_dev *dev, struct rte_flow *flow,
 	case RTE_ETH_FILTER_ETHERTYPE:
 		ethertype_filter_ptr = (struct hinic_ethertype_filter_ele *)
 					pmd_flow->rule;
-		rte_memcpy(&ethertype_filter,
-			&ethertype_filter_ptr->filter_info,
-			sizeof(struct rte_eth_ethertype_filter));
+		memcpy(&ethertype_filter, &ethertype_filter_ptr->filter_info,
+		       sizeof(struct rte_eth_ethertype_filter));
 		ret = hinic_add_del_ethertype_filter(dev,
 				&ethertype_filter, FALSE);
 		if (!ret) {
@@ -3134,9 +3131,8 @@  static int hinic_flow_destroy(struct rte_eth_dev *dev, struct rte_flow *flow,
 		break;
 	case RTE_ETH_FILTER_FDIR:
 		fdir_rule_ptr = (struct hinic_fdir_rule_ele *)pmd_flow->rule;
-		rte_memcpy(&fdir_rule,
-			&fdir_rule_ptr->filter_info,
-			sizeof(struct hinic_fdir_rule));
+		memcpy(&fdir_rule, &fdir_rule_ptr->filter_info,
+		       sizeof(struct hinic_fdir_rule));
 		if (fdir_rule.mode == HINIC_FDIR_MODE_NORMAL) {
 			ret = hinic_add_del_fdir_filter(dev, &fdir_rule, FALSE);
 		} else if (fdir_rule.mode == HINIC_FDIR_MODE_TCAM) {