diff mbox series

[71/71] examples: replace use of fixed size rte_memcpy

Message ID	20240229225936.483472-72-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>, Nicolas Chautru <nicolas.chautru@intel.com>, Tomasz Kantecki <tomasz.kantecki@intel.com>, Kirill Rybalchenko <kirill.rybalchenko@intel.com>, Maxime Coquelin <maxime.coquelin@redhat.com>, Chenbo Xia <chenbox@nvidia.com> Subject: [PATCH 71/71] examples: replace use of fixed size rte_memcpy Date: Thu, 29 Feb 2024 14:58:58 -0800 Message-ID: <20240229225936.483472-72-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	warning	coding style issues
ci/loongarch-compilation	success	Compilation OK
ci/loongarch-unit-testing	success	Unit Testing PASS
ci/Intel-compilation	success	Compilation OK
ci/intel-Testing	success	Testing PASS
ci/github-robot: build	fail	github build: failed
ci/intel-Functional	success	Functional PASS
ci/iol-intel-Performance	success	Performance Testing PASS
ci/iol-mellanox-Performance	success	Performance Testing PASS
ci/iol-abi-testing	success	Testing PASS
ci/iol-unit-arm64-testing	success	Testing PASS
ci/iol-compile-arm64-testing	success	Testing PASS
ci/iol-compile-amd64-testing	success	Testing PASS
ci/iol-unit-amd64-testing	success	Testing PASS
ci/iol-intel-Functional	success	Functional Testing PASS
ci/iol-broadcom-Performance	success	Performance Testing PASS
ci/iol-broadcom-Functional	success	Functional Testing PASS
ci/iol-sample-apps-testing	success	Testing PASS

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>
---
 examples/bbdev_app/main.c      |  2 +-
 examples/l2fwd-cat/cat.c       |  3 +--
 examples/ptpclient/ptpclient.c | 11 +++++------
 examples/vhost/main.c          |  5 ++---
 examples/vmdq/main.c           |  5 ++---
 examples/vmdq_dcb/main.c       | 14 +++++---------
 6 files changed, 16 insertions(+), 24 deletions(-)

diff mbox series

Patch

diff --git a/examples/bbdev_app/main.c b/examples/bbdev_app/main.c
index 16599ae9cee3..52c8eabba778 100644
--- a/examples/bbdev_app/main.c
+++ b/examples/bbdev_app/main.c
@@ -359,7 +359,7 @@  add_ether_hdr(struct rte_mbuf *pkt_src, struct rte_mbuf *pkt_dst)
 	eth_to = rte_pktmbuf_mtod(pkt_dst, struct rte_ether_hdr *);
 
 	/* copy header */
-	rte_memcpy(eth_to, eth_from, sizeof(struct rte_ether_hdr));
+	memcpy(eth_to, eth_from, sizeof(struct rte_ether_hdr));
 }
 
 static inline void
diff --git a/examples/l2fwd-cat/cat.c b/examples/l2fwd-cat/cat.c
index 00e4cde48b48..b03fef47fdb7 100644
--- a/examples/l2fwd-cat/cat.c
+++ b/examples/l2fwd-cat/cat.c
@@ -314,8 +314,7 @@  parse_l3ca(const char *l3ca)
 		if (cmask != 0 && is_contiguous(cmask) == 0)
 			goto err;
 
-		rte_memcpy(&m_config[idx].cpumask,
-			&cpuset, sizeof(rte_cpuset_t));
+		memcpy(&m_config[idx].cpumask, &cpuset, sizeof(rte_cpuset_t));
 
 		if (cmask != 0) {
 			m_config[idx].cdp = 1;
diff --git a/examples/ptpclient/ptpclient.c b/examples/ptpclient/ptpclient.c
index 2535d848a1e9..e918bc05db34 100644
--- a/examples/ptpclient/ptpclient.c
+++ b/examples/ptpclient/ptpclient.c
@@ -359,9 +359,9 @@  parse_sync(struct ptpv2_data_slave_ordinary *ptp_data, uint16_t rx_tstamp_idx)
 	ptp_data->seqID_SYNC = rte_be_to_cpu_16(ptp_hdr->seq_id);
 
 	if (ptp_data->ptpset == 0) {
-		rte_memcpy(&ptp_data->master_clock_id,
-				&ptp_hdr->source_port_id.clock_id,
-				sizeof(struct clock_id));
+		memcpy(&ptp_data->master_clock_id,
+		       &ptp_hdr->source_port_id.clock_id,
+		       sizeof(struct clock_id));
 		ptp_data->ptpset = 1;
 	}
 
@@ -466,9 +466,8 @@  parse_fup(struct ptpv2_data_slave_ordinary *ptp_data)
 		client_clkid->id[6] = eth_hdr->src_addr.addr_bytes[4];
 		client_clkid->id[7] = eth_hdr->src_addr.addr_bytes[5];
 
-		rte_memcpy(&ptp_data->client_clock_id,
-			   client_clkid,
-			   sizeof(struct clock_id));
+		memcpy(&ptp_data->client_clock_id, client_clkid,
+		       sizeof(struct clock_id));
 
 		/* Enable flag for hardware timestamping. */
 		created_pkt->ol_flags |= RTE_MBUF_F_TX_IEEE1588_TMST;
diff --git a/examples/vhost/main.c b/examples/vhost/main.c
index 3fc1b151d182..5691ec110ac8 100644
--- a/examples/vhost/main.c
+++ b/examples/vhost/main.c
@@ -405,9 +405,8 @@  get_eth_conf(struct rte_eth_conf *eth_conf, uint32_t num_devices)
 		conf.pool_map[i].pools = (1UL << i);
 	}
 
-	(void)(rte_memcpy(eth_conf, &vmdq_conf_default, sizeof(*eth_conf)));
-	(void)(rte_memcpy(&eth_conf->rx_adv_conf.vmdq_rx_conf, &conf,
-		   sizeof(eth_conf->rx_adv_conf.vmdq_rx_conf)));
+	(void)(memcpy(eth_conf, &vmdq_conf_default, sizeof(*eth_conf)));
+	(void)(memcpy(&eth_conf->rx_adv_conf.vmdq_rx_conf, &conf, sizeof(eth_conf->rx_adv_conf.vmdq_rx_conf)));
 	return 0;
 }
 
diff --git a/examples/vmdq/main.c b/examples/vmdq/main.c
index 4a3ce6884c5c..3a9d1281d901 100644
--- a/examples/vmdq/main.c
+++ b/examples/vmdq/main.c
@@ -151,9 +151,8 @@  get_eth_conf(struct rte_eth_conf *eth_conf, uint32_t num_pools)
 		conf.pool_map[i].pools = (1UL << (i % num_pools));
 	}
 
-	(void)(rte_memcpy(eth_conf, &vmdq_conf_default, sizeof(*eth_conf)));
-	(void)(rte_memcpy(&eth_conf->rx_adv_conf.vmdq_rx_conf, &conf,
-		   sizeof(eth_conf->rx_adv_conf.vmdq_rx_conf)));
+	(void)(memcpy(eth_conf, &vmdq_conf_default, sizeof(*eth_conf)));
+	(void)(memcpy(&eth_conf->rx_adv_conf.vmdq_rx_conf, &conf, sizeof(eth_conf->rx_adv_conf.vmdq_rx_conf)));
 	if (rss_enable) {
 		eth_conf->rxmode.mq_mode = RTE_ETH_MQ_RX_VMDQ_RSS;
 		eth_conf->rx_adv_conf.rss_conf.rss_hf = RTE_ETH_RSS_IP |
diff --git a/examples/vmdq_dcb/main.c b/examples/vmdq_dcb/main.c
index 4ccc2fe4b01c..4aab67b23d67 100644
--- a/examples/vmdq_dcb/main.c
+++ b/examples/vmdq_dcb/main.c
@@ -161,15 +161,11 @@  get_eth_conf(struct rte_eth_conf *eth_conf)
 		tx_conf.dcb_tc[i] = i % num_tcs;
 	}
 	dcb_conf.nb_tcs = (enum rte_eth_nb_tcs)num_tcs;
-	(void)(rte_memcpy(eth_conf, &vmdq_dcb_conf_default, sizeof(*eth_conf)));
-	(void)(rte_memcpy(&eth_conf->rx_adv_conf.vmdq_dcb_conf, &conf,
-			  sizeof(conf)));
-	(void)(rte_memcpy(&eth_conf->rx_adv_conf.dcb_rx_conf, &dcb_conf,
-			  sizeof(dcb_conf)));
-	(void)(rte_memcpy(&eth_conf->rx_adv_conf.vmdq_rx_conf, &vmdq_conf,
-			  sizeof(vmdq_conf)));
-	(void)(rte_memcpy(&eth_conf->tx_adv_conf.vmdq_dcb_tx_conf, &tx_conf,
-			  sizeof(tx_conf)));
+	(void)(memcpy(eth_conf, &vmdq_dcb_conf_default, sizeof(*eth_conf)));
+	(void)(memcpy(&eth_conf->rx_adv_conf.vmdq_dcb_conf, &conf, sizeof(conf)));
+	(void)(memcpy(&eth_conf->rx_adv_conf.dcb_rx_conf, &dcb_conf, sizeof(dcb_conf)));
+	(void)(memcpy(&eth_conf->rx_adv_conf.vmdq_rx_conf, &vmdq_conf, sizeof(vmdq_conf)));
+	(void)(memcpy(&eth_conf->tx_adv_conf.vmdq_dcb_tx_conf, &tx_conf, sizeof(tx_conf)));
 	if (rss_enable) {
 		eth_conf->rxmode.mq_mode = RTE_ETH_MQ_RX_VMDQ_DCB_RSS;
 		eth_conf->rx_adv_conf.rss_conf.rss_hf = RTE_ETH_RSS_IP |