Hi Hemant, On 11/11/2016 04:47 PM, Hemant Agrawal wrote: > From: Nipun Gupta <nipun.gupta at nxp.com> > > This patch fixes the issue of memzone not being freed incase the > rte_mempool_populate_phys fails in the rte_mempool_populate_default > > This issue was identified when testing with OVS ~2.6 > - configure the system with low memory (e.g. < 500 MB) > - add bridge and dpdk interfaces > - delete brigde > - keep on repeating the above sequence. > > Fixes: d1d914ebbc25 ("mempool: allocate in several memory chunks by default") > > Signed-off-by: Nipun Gupta <nipun.gupta at nxp.com> > --- > lib/librte_mempool/rte_mempool.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/lib/librte_mempool/rte_mempool.c > b/lib/librte_mempool/rte_mempool.c > index e94e56f..aa513b9 100644 > --- a/lib/librte_mempool/rte_mempool.c > +++ b/lib/librte_mempool/rte_mempool.c > @@ -578,8 +578,10 @@ rte_mempool_populate_default(struct rte_mempool *mp) > mz->len, pg_sz, > rte_mempool_memchunk_mz_free, > (void *)(uintptr_t)mz); > - if (ret < 0) > + if (ret < 0) { > + rte_memzone_free(mz); > goto fail; > + } > } > > return mp->size; >
Acked-by: Olivier Matz <olivier.matz at 6wind.com> Thanks Olivier