diff mbox series

[67/71] app/test-pmd: replace use of fixed size rte_memcpy

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

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Stephen Hemminger Feb. 29, 2024, 10:58 p.m. UTC

  Automatically generated by devtools/cocci/rte_memcpy.cocci

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 app/test-pmd/cmdline.c      | 48 ++++++++++++++++++-------------------
 app/test-pmd/cmdline_flow.c | 24 +++++++++----------
 app/test-pmd/config.c       |  8 +++----
 3 files changed, 40 insertions(+), 40 deletions(-)

diff mbox series

Patch

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
index 566057caa523..898fb69a2f4d 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -9529,7 +9529,7 @@  static void cmd_set_vxlan_parsed(void *parsed_result,
 		vxlan_encap_conf.select_ipv4 = 0;
 	else
 		return;
-	rte_memcpy(vxlan_encap_conf.vni, &id.vni[1], 3);
+	memcpy(vxlan_encap_conf.vni, &id.vni[1], 3);
 	vxlan_encap_conf.udp_src = rte_cpu_to_be_16(res->udp_src);
 	vxlan_encap_conf.udp_dst = rte_cpu_to_be_16(res->udp_dst);
 	vxlan_encap_conf.ip_tos = res->tos;
@@ -9543,10 +9543,10 @@  static void cmd_set_vxlan_parsed(void *parsed_result,
 	}
 	if (vxlan_encap_conf.select_vlan)
 		vxlan_encap_conf.vlan_tci = rte_cpu_to_be_16(res->tci);
-	rte_memcpy(vxlan_encap_conf.eth_src, res->eth_src.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
-	rte_memcpy(vxlan_encap_conf.eth_dst, res->eth_dst.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
+	memcpy(vxlan_encap_conf.eth_src, res->eth_src.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
+	memcpy(vxlan_encap_conf.eth_dst, res->eth_dst.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
 }
 
 static cmdline_parse_inst_t cmd_set_vxlan = {
@@ -9724,7 +9724,7 @@  static void cmd_set_nvgre_parsed(void *parsed_result,
 		nvgre_encap_conf.select_ipv4 = 0;
 	else
 		return;
-	rte_memcpy(nvgre_encap_conf.tni, &id.tni[1], 3);
+	memcpy(nvgre_encap_conf.tni, &id.tni[1], 3);
 	if (nvgre_encap_conf.select_ipv4) {
 		IPV4_ADDR_TO_UINT(res->ip_src, nvgre_encap_conf.ipv4_src);
 		IPV4_ADDR_TO_UINT(res->ip_dst, nvgre_encap_conf.ipv4_dst);
@@ -9734,10 +9734,10 @@  static void cmd_set_nvgre_parsed(void *parsed_result,
 	}
 	if (nvgre_encap_conf.select_vlan)
 		nvgre_encap_conf.vlan_tci = rte_cpu_to_be_16(res->tci);
-	rte_memcpy(nvgre_encap_conf.eth_src, res->eth_src.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
-	rte_memcpy(nvgre_encap_conf.eth_dst, res->eth_dst.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
+	memcpy(nvgre_encap_conf.eth_src, res->eth_src.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
+	memcpy(nvgre_encap_conf.eth_dst, res->eth_dst.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
 }
 
 static cmdline_parse_inst_t cmd_set_nvgre = {
@@ -9851,10 +9851,10 @@  static void cmd_set_l2_encap_parsed(void *parsed_result,
 		return;
 	if (l2_encap_conf.select_vlan)
 		l2_encap_conf.vlan_tci = rte_cpu_to_be_16(res->tci);
-	rte_memcpy(l2_encap_conf.eth_src, res->eth_src.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
-	rte_memcpy(l2_encap_conf.eth_dst, res->eth_dst.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
+	memcpy(l2_encap_conf.eth_src, res->eth_src.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
+	memcpy(l2_encap_conf.eth_dst, res->eth_dst.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
 }
 
 static cmdline_parse_inst_t cmd_set_l2_encap = {
@@ -10033,7 +10033,7 @@  static void cmd_set_mplsogre_encap_parsed(void *parsed_result,
 		mplsogre_encap_conf.select_ipv4 = 0;
 	else
 		return;
-	rte_memcpy(mplsogre_encap_conf.label, &id.label, 3);
+	memcpy(mplsogre_encap_conf.label, &id.label, 3);
 	if (mplsogre_encap_conf.select_ipv4) {
 		IPV4_ADDR_TO_UINT(res->ip_src, mplsogre_encap_conf.ipv4_src);
 		IPV4_ADDR_TO_UINT(res->ip_dst, mplsogre_encap_conf.ipv4_dst);
@@ -10043,10 +10043,10 @@  static void cmd_set_mplsogre_encap_parsed(void *parsed_result,
 	}
 	if (mplsogre_encap_conf.select_vlan)
 		mplsogre_encap_conf.vlan_tci = rte_cpu_to_be_16(res->tci);
-	rte_memcpy(mplsogre_encap_conf.eth_src, res->eth_src.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
-	rte_memcpy(mplsogre_encap_conf.eth_dst, res->eth_dst.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
+	memcpy(mplsogre_encap_conf.eth_src, res->eth_src.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
+	memcpy(mplsogre_encap_conf.eth_dst, res->eth_dst.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
 }
 
 static cmdline_parse_inst_t cmd_set_mplsogre_encap = {
@@ -10269,7 +10269,7 @@  static void cmd_set_mplsoudp_encap_parsed(void *parsed_result,
 		mplsoudp_encap_conf.select_ipv4 = 0;
 	else
 		return;
-	rte_memcpy(mplsoudp_encap_conf.label, &id.label, 3);
+	memcpy(mplsoudp_encap_conf.label, &id.label, 3);
 	mplsoudp_encap_conf.udp_src = rte_cpu_to_be_16(res->udp_src);
 	mplsoudp_encap_conf.udp_dst = rte_cpu_to_be_16(res->udp_dst);
 	if (mplsoudp_encap_conf.select_ipv4) {
@@ -10281,10 +10281,10 @@  static void cmd_set_mplsoudp_encap_parsed(void *parsed_result,
 	}
 	if (mplsoudp_encap_conf.select_vlan)
 		mplsoudp_encap_conf.vlan_tci = rte_cpu_to_be_16(res->tci);
-	rte_memcpy(mplsoudp_encap_conf.eth_src, res->eth_src.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
-	rte_memcpy(mplsoudp_encap_conf.eth_dst, res->eth_dst.addr_bytes,
-		   RTE_ETHER_ADDR_LEN);
+	memcpy(mplsoudp_encap_conf.eth_src, res->eth_src.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
+	memcpy(mplsoudp_encap_conf.eth_dst, res->eth_dst.addr_bytes,
+	       RTE_ETHER_ADDR_LEN);
 }
 
 static cmdline_parse_inst_t cmd_set_mplsoudp_encap = {
diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
index b4389e51506b..17b73d2407c4 100644
--- a/app/test-pmd/cmdline_flow.c
+++ b/app/test-pmd/cmdline_flow.c
@@ -13732,7 +13732,7 @@  cmd_set_raw_parsed_sample(const struct buffer *in)
 			fprintf(stderr, "Error - Not supported action\n");
 			return;
 		}
-		rte_memcpy(data, action, sizeof(struct rte_flow_action));
+		memcpy(data, action, sizeof(struct rte_flow_action));
 		data++;
 	}
 }
@@ -13893,8 +13893,8 @@  cmd_set_raw_parsed(const struct buffer *in)
 
 				/* We have to add GTP header extra word. */
 				*total_size += sizeof(ext_word);
-				rte_memcpy(data_tail - (*total_size),
-					   &ext_word, sizeof(ext_word));
+				memcpy(data_tail - (*total_size), &ext_word,
+				       sizeof(ext_word));
 			}
 			size = sizeof(struct rte_gtp_hdr);
 			break;
@@ -13939,21 +13939,21 @@  cmd_set_raw_parsed(const struct buffer *in)
 				if (opt->checksum_rsvd.checksum) {
 					*total_size +=
 						sizeof(opt->checksum_rsvd);
-					rte_memcpy(data_tail - (*total_size),
-						   &opt->checksum_rsvd,
-						   sizeof(opt->checksum_rsvd));
+					memcpy(data_tail - (*total_size),
+					       &opt->checksum_rsvd,
+					       sizeof(opt->checksum_rsvd));
 				}
 				if (opt->key.key) {
 					*total_size += sizeof(opt->key.key);
-					rte_memcpy(data_tail - (*total_size),
-						   &opt->key.key,
-						   sizeof(opt->key.key));
+					memcpy(data_tail - (*total_size),
+					       &opt->key.key,
+					       sizeof(opt->key.key));
 				}
 				if (opt->sequence.sequence) {
 					*total_size += sizeof(opt->sequence.sequence);
-					rte_memcpy(data_tail - (*total_size),
-						   &opt->sequence.sequence,
-						   sizeof(opt->sequence.sequence));
+					memcpy(data_tail - (*total_size),
+					       &opt->sequence.sequence,
+					       sizeof(opt->sequence.sequence));
 				}
 			}
 			proto = 0x2F;
diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
index 08d7d0e2f87c..2b81f3ef3d2d 100644
--- a/app/test-pmd/config.c
+++ b/app/test-pmd/config.c
@@ -2639,8 +2639,8 @@  port_flow_template_table_create(portid_t port_id, uint32_t id,
 	}
 	pt->nb_pattern_templates = nb_pattern_templates;
 	pt->nb_actions_templates = nb_actions_templates;
-	rte_memcpy(&pt->flow_attr, &table_attr->flow_attr,
-		   sizeof(struct rte_flow_attr));
+	memcpy(&pt->flow_attr, &table_attr->flow_attr,
+	       sizeof(struct rte_flow_attr));
 	printf("Template table #%u created\n", pt->id);
 	return 0;
 }
@@ -3261,8 +3261,8 @@  port_queue_action_handle_update(portid_t port_id,
 		update = action->conf;
 		break;
 	case RTE_FLOW_ACTION_TYPE_METER_MARK:
-		rte_memcpy(&mtr_update.meter_mark, action->conf,
-			sizeof(struct rte_flow_action_meter_mark));
+		memcpy(&mtr_update.meter_mark, action->conf,
+		       sizeof(struct rte_flow_action_meter_mark));
 		if (mtr_update.meter_mark.profile)
 			mtr_update.profile_valid = 1;
 		if (mtr_update.meter_mark.policy)