01/01/2023 12:28, Morten Brørup: > > From: wushao...@chinatelecom.cn [mailto:wushao...@chinatelecom.cn] > > Sent: Sunday, 1 January 2023 06.18 > > > > From: Shaohua Wu <wushao...@chinatelecom.cn> > > > > rte_mempool_dump add dump for monitoring available and > > used memory blocks > > > > Signed-off-by: Shaohua Wu <wushao...@chinatelecom.cn> > > --- > > lib/mempool/rte_mempool.c | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/lib/mempool/rte_mempool.c b/lib/mempool/rte_mempool.c > > index f33f455790..09d512a604 100644 > > --- a/lib/mempool/rte_mempool.c > > +++ b/lib/mempool/rte_mempool.c > > @@ -1265,9 +1265,14 @@ rte_mempool_dump(FILE *f, struct rte_mempool > > *mp) > > > > cache_count = rte_mempool_dump_cache(f, mp); > > common_count = rte_mempool_ops_get_count(mp); > > + > > if ((cache_count + common_count) > mp->size) > > common_count = mp->size - cache_count; > > fprintf(f, " common_pool_count=%u\n", common_count); > > + fprintf(f, " common_pool_avail_count=%u\n", > > + rte_mempool_avail_count(mp)); > > The number of available objects in the common pool is already shown as > "common_pool_count". > > If you want to show the total number of available objects in both the common > pool and the caches, you can calculate is as common_count + cache_count, > instead of calling rte_mempool_avail_count(). And "total_count" would be a > better name than "common_pool_avail_count". > > > + fprintf(f, " common_pool_used_count=%u\n", > > + rte_mempool_in_use_count(mp)); > > common_pool_used_count can be calculated as mp->size - common_count, instead > of calling rte_mempool_in_use_count(mp).
No reply... I assume it is a rejected patch.
