DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] ring: add function to free a ring
@ 2015-08-18 14:00 Pablo de Lara
  2015-09-07  8:45 ` Olivier MATZ
  2015-09-22 13:55 ` [dpdk-dev] [PATCH v2] " Pablo de Lara
  0 siblings, 2 replies; 10+ messages in thread
From: Pablo de Lara @ 2015-08-18 14:00 UTC (permalink / raw)
  To: dev

When creating a ring, a memzone is created to allocate it in memory,
but the ring could not be freed, as memzones could not be.

Since memzones can be freed now, then rings can be as well,
taking into account if they were initialized using pre-allocated memory
(in which case, memory should be freed externally) or using rte_memzone_reserve
(with rte_ring_create), freeing the memory with rte_memzone_free.

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
---
 lib/librte_ring/rte_ring.c           | 43 ++++++++++++++++++++++++++++++++++++
 lib/librte_ring/rte_ring.h           |  7 ++++++
 lib/librte_ring/rte_ring_version.map |  7 ++++++
 3 files changed, 57 insertions(+)

diff --git a/lib/librte_ring/rte_ring.c b/lib/librte_ring/rte_ring.c
index c9e59d4..83ce6d3 100644
--- a/lib/librte_ring/rte_ring.c
+++ b/lib/librte_ring/rte_ring.c
@@ -208,6 +208,49 @@ rte_ring_create(const char *name, unsigned count, int socket_id,
 	return r;
 }
 
+/* free the ring */
+void
+rte_ring_free(struct rte_ring *r)
+{
+	struct rte_ring_list *ring_list = NULL;
+	char mz_name[RTE_MEMZONE_NAMESIZE];
+	struct rte_tailq_entry *te;
+	const struct rte_memzone *mz;
+
+	if (r == NULL)
+		return;
+
+	snprintf(mz_name, sizeof(mz_name), "%s%s", RTE_RING_MZ_PREFIX, r->name);
+	mz = rte_memzone_lookup(mz_name);
+
+	/*
+	 * Free ring memory if it was allocated with rte_memzone_reserve,
+	 * otherwise it should be freed externally
+	 */
+	if (rte_memzone_free(mz) != 0)
+		return;
+
+	ring_list = RTE_TAILQ_CAST(rte_ring_tailq.head, rte_ring_list);
+	rte_rwlock_write_lock(RTE_EAL_TAILQ_RWLOCK);
+
+	/* find out tailq entry */
+	TAILQ_FOREACH(te, ring_list, next) {
+		if (te->data == (void *) r)
+			break;
+	}
+
+	if (te == NULL) {
+		rte_rwlock_write_unlock(RTE_EAL_TAILQ_RWLOCK);
+		return;
+	}
+
+	TAILQ_REMOVE(ring_list, te, next);
+
+	rte_rwlock_write_unlock(RTE_EAL_TAILQ_RWLOCK);
+
+	rte_free(te);
+}
+
 /*
  * change the high water mark. If *count* is 0, water marking is
  * disabled
diff --git a/lib/librte_ring/rte_ring.h b/lib/librte_ring/rte_ring.h
index af68888..e75566f 100644
--- a/lib/librte_ring/rte_ring.h
+++ b/lib/librte_ring/rte_ring.h
@@ -300,6 +300,13 @@ int rte_ring_init(struct rte_ring *r, const char *name, unsigned count,
  */
 struct rte_ring *rte_ring_create(const char *name, unsigned count,
 				 int socket_id, unsigned flags);
+/**
+ * De-allocate all memory used by the ring.
+ *
+ * @param r
+ *   Ring to free
+ */
+void rte_ring_free(struct rte_ring *r);
 
 /**
  * Change the high water mark.
diff --git a/lib/librte_ring/rte_ring_version.map b/lib/librte_ring/rte_ring_version.map
index 982fdd1..5474b98 100644
--- a/lib/librte_ring/rte_ring_version.map
+++ b/lib/librte_ring/rte_ring_version.map
@@ -11,3 +11,10 @@ DPDK_2.0 {
 
 	local: *;
 };
+
+DPDK_2.2 {
+	global:
+
+	rte_ring_free;
+
+} DPDK_2.0;
-- 
2.4.2

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2015-11-03 23:52 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-18 14:00 [dpdk-dev] [PATCH] ring: add function to free a ring Pablo de Lara
2015-09-07  8:45 ` Olivier MATZ
2015-09-09  7:48   ` De Lara Guarch, Pablo
2015-09-22 13:55 ` [dpdk-dev] [PATCH v2] " Pablo de Lara
2015-10-02 14:01   ` [dpdk-dev] [PATCH v3] " Pablo de Lara
2015-10-02 14:09     ` Bruce Richardson
2015-10-02 14:27       ` De Lara Guarch, Pablo
2015-10-02 15:53     ` [dpdk-dev] [PATCH v4] " Pablo de Lara
2015-10-13 14:01       ` Olivier MATZ
2015-11-03 23:50         ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).