diff mbox series

[v4,11/11] {common,net}/cnxk: remove unused plt_bitmap_free()

Message ID 20241222125725.1532157-12-otilibil@eurecom.fr (mailing list archive)
State New
Delegated to: Thomas Monjalon
Headers
Series devtools, lib, test, net, common: remove unused rte_bitmap_free() and plt_bitmap_free() |

Checks

Context Check Description
ci/loongarch-compilation success Compilation OK
ci/checkpatch success coding style OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/github-robot: build success github build: passed
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing fail Testing issues
ci/intel-Functional success Functional PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-marvell-Functional success Functional Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-compile-amd64-testing success Testing PASS

Commit Message

Ariel Otilibili-Anieli Dec. 22, 2024, 12:49 p.m. UTC 
Depends on commits:
* 13844f8557 ("common/cnxk: remove unused plt_bitmap_free()")
* d5941e7269 ("devtools/cocci,lib/eal: remove unused rte_bitmap_free()").

Fixes: f752780f30 ("common/cnxk: add ROC MACsec initialization")
Fixes: bea5d990a9 ("net/cnxk: support outbound soft expiry notification")
Fixes: 05d727e8b1 ("common/cnxk: support NIX traffic management")
Fixes: 124ff1a4cb ("common/cnxk: support NPA device")
Fixes: 357f5ebc8a ("common/cnxk: support flow aging")
Fixes: 7eabd6c637 ("net/cnxk: support inline security setup for cn9k")
Signed-off-by: Ariel Otilibili <otilibil@eurecom.fr>
---
Cc: stable@dpdk.org
Cc: Nithin Dabilpuram <ndabilpuram@marvell.com>
Cc: Kiran Kumar K <kirankumark@marvell.com>
Cc: Sunil Kumar Kori <skori@marvell.com>
Cc: Satha Rao <skoteshwar@marvell.com>
Cc: Harman Kalra <hkalra@marvell.com>
---
 drivers/common/cnxk/roc_mcs.c         | 4 ----
 drivers/common/cnxk/roc_nix_inl_dev.c | 2 --
 drivers/common/cnxk/roc_nix_tm.c      | 5 -----
 drivers/common/cnxk/roc_npa.c         | 4 ----
 drivers/common/cnxk/roc_npc_aging.c   | 1 -
 drivers/net/cnxk/cnxk_ethdev.c        | 1 -
 6 files changed, 17 deletions(-)
diff mbox series

Patch

diff --git a/drivers/common/cnxk/roc_mcs.c b/drivers/common/cnxk/roc_mcs.c
index f823f7f478..4df3b2ff92 100644
--- a/drivers/common/cnxk/roc_mcs.c
+++ b/drivers/common/cnxk/roc_mcs.c
@@ -719,13 +719,9 @@  mcs_alloc_bmap(uint16_t entries, void **mem, struct plt_bitmap **bmap)
 static void
 rsrc_bmap_free(struct mcs_rsrc *rsrc)
 {
-	plt_bitmap_free(rsrc->tcam_bmap);
 	plt_free(rsrc->tcam_bmap_mem);
-	plt_bitmap_free(rsrc->secy_bmap);
 	plt_free(rsrc->secy_bmap_mem);
-	plt_bitmap_free(rsrc->sc_bmap);
 	plt_free(rsrc->sc_bmap_mem);
-	plt_bitmap_free(rsrc->sa_bmap);
 	plt_free(rsrc->sa_bmap_mem);
 }
 
diff --git a/drivers/common/cnxk/roc_nix_inl_dev.c b/drivers/common/cnxk/roc_nix_inl_dev.c
index ffe6eef81f..d2b5aa5bab 100644
--- a/drivers/common/cnxk/roc_nix_inl_dev.c
+++ b/drivers/common/cnxk/roc_nix_inl_dev.c
@@ -837,7 +837,6 @@  nix_inl_outb_poll_thread_setup(struct nix_inl_dev *inl_dev)
 	rc = plt_thread_create_control(&inl_dev->soft_exp_poll_thread,
 			"outb-poll", nix_inl_outb_poll_thread, inl_dev);
 	if (rc) {
-		plt_bitmap_free(inl_dev->soft_exp_ring_bmap);
 		plt_free(inl_dev->soft_exp_ring_bmap_mem);
 	}
 
@@ -1069,7 +1068,6 @@  roc_nix_inl_dev_fini(struct roc_nix_inl_dev *roc_inl_dev)
 	if (inl_dev->set_soft_exp_poll) {
 		soft_exp_poll_thread_exit = true;
 		plt_thread_join(inl_dev->soft_exp_poll_thread, NULL);
-		plt_bitmap_free(inl_dev->soft_exp_ring_bmap);
 		plt_free(inl_dev->soft_exp_ring_bmap_mem);
 		plt_free(inl_dev->sa_soft_exp_ring);
 	}
diff --git a/drivers/common/cnxk/roc_nix_tm.c b/drivers/common/cnxk/roc_nix_tm.c
index abfe80978b..f8b14acb16 100644
--- a/drivers/common/cnxk/roc_nix_tm.c
+++ b/drivers/common/cnxk/roc_nix_tm.c
@@ -2030,11 +2030,6 @@  void
 nix_tm_conf_fini(struct roc_nix *roc_nix)
 {
 	struct nix *nix = roc_nix_to_nix_priv(roc_nix);
-	uint16_t hw_lvl;
 
-	for (hw_lvl = 0; hw_lvl < NIX_TXSCH_LVL_CNT; hw_lvl++) {
-		plt_bitmap_free(nix->schq_bmp[hw_lvl]);
-		plt_bitmap_free(nix->schq_contig_bmp[hw_lvl]);
-	}
 	plt_free(nix->schq_bmp_mem);
 }
diff --git a/drivers/common/cnxk/roc_npa.c b/drivers/common/cnxk/roc_npa.c
index a33f9a8499..9ec010119b 100644
--- a/drivers/common/cnxk/roc_npa.c
+++ b/drivers/common/cnxk/roc_npa.c
@@ -1227,7 +1227,6 @@  npa_dev_init(struct npa_lf *lf, uintptr_t base, struct mbox *mbox)
 	lf->npa_qint_mem = plt_zmalloc(sizeof(struct npa_qint) * nr_pools, 0);
 	if (lf->npa_qint_mem == NULL) {
 		rc = NPA_ERR_ALLOC;
-		goto bmap_free;
 	}
 
 	/* Allocate memory for nap_aura_lim memory */
@@ -1256,8 +1255,6 @@  npa_dev_init(struct npa_lf *lf, uintptr_t base, struct mbox *mbox)
 	plt_free(lf->aura_lim);
 qint_free:
 	plt_free(lf->npa_qint_mem);
-bmap_free:
-	plt_bitmap_free(lf->npa_bmp);
 bmap_mem_free:
 	plt_free(lf->npa_bmp_mem);
 lf_free:
@@ -1274,7 +1271,6 @@  npa_dev_fini(struct npa_lf *lf)
 
 	plt_free(lf->aura_lim);
 	plt_free(lf->npa_qint_mem);
-	plt_bitmap_free(lf->npa_bmp);
 	plt_free(lf->npa_bmp_mem);
 	plt_free(lf->aura_attr);
 
diff --git a/drivers/common/cnxk/roc_npc_aging.c b/drivers/common/cnxk/roc_npc_aging.c
index 258c15e341..93b1bf2a47 100644
--- a/drivers/common/cnxk/roc_npc_aging.c
+++ b/drivers/common/cnxk/roc_npc_aging.c
@@ -43,7 +43,6 @@  npc_aged_flows_bitmap_free(struct roc_npc *roc_npc)
 	struct roc_npc_flow_age *flow_age;
 
 	flow_age = &roc_npc->flow_age;
-	plt_bitmap_free(flow_age->aged_flows);
 	if (flow_age->age_mem)
 		plt_free(roc_npc->flow_age.age_mem);
 }
diff --git a/drivers/net/cnxk/cnxk_ethdev.c b/drivers/net/cnxk/cnxk_ethdev.c
index ea980a6d5e..3d30c2939f 100644
--- a/drivers/net/cnxk/cnxk_ethdev.c
+++ b/drivers/net/cnxk/cnxk_ethdev.c
@@ -311,7 +311,6 @@  nix_security_release(struct cnxk_eth_dev *dev)
 			plt_err("Failed to cleanup nix inline outb, rc=%d", rc);
 		ret |= rc;
 
-		plt_bitmap_free(dev->outb.sa_bmap);
 		plt_free(dev->outb.sa_bmap_mem);
 		dev->outb.sa_bmap = NULL;
 		dev->outb.sa_bmap_mem = NULL;