Thread (16 messages) 16 messages, 5 authors, 2016-07-01

Re: [PATCH v2] mempool: rename functions with confusing names

From: Bruce Richardson <hidden>
Date: 2016-06-30 12:09:55

On Thu, Jun 30, 2016 at 03:02:22PM +0300, Panu Matilainen wrote:
On 06/30/2016 03:00 PM, Thomas Monjalon wrote:
quoted
2016-06-29 17:27, Bruce Richardson:
quoted
Fix this by introducing two new functions to replace the old ones,
* rte_mempool_avail_count to replace rte_mempool_count
* rte_mempool_in_use_count to replace rte_mempool_free_count
This patch needs to be rebased please.
quoted
--- a/lib/librte_mempool/rte_mempool_version.map
+++ b/lib/librte_mempool/rte_mempool_version.map
@@ -32,5 +32,6 @@ DPDK_16.07 {
	rte_mempool_populate_virt;
	rte_mempool_register_ops;
	rte_mempool_set_ops_byname;
+	rte_mempool_avail_count;
The "in_use_count" function is missing.
Its missing because the function is static inline. If you ask me, this would
be as fine time as any to remove that inlining ;)

	- Panu -
Yep, and yep.
V3 needed, I'll see what I can do.

/Bruce
quoted
Please keep alphabetical order.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help