2016-07-21 15:28, Olivier Matz: > Hi Santosh, > > On 07/21/2016 01:49 PM, Santosh Shukla wrote: > > test_mempool func not using pointer variable 'mp_ext' and incorrectly > > freed. So > > removing ptr var. Now freeing mp_stack var. > > > > Signed-off-by: Santosh Shukla <santosh.shukla at caviumnetworks.com> > > --- > > app/test/test_mempool.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/app/test/test_mempool.c b/app/test/test_mempool.c > > index 46ad670..3b21cf7 100644 > > --- a/app/test/test_mempool.c > > +++ b/app/test/test_mempool.c > > @@ -506,7 +506,6 @@ test_mempool(void) > > { > > struct rte_mempool *mp_cache = NULL; > > struct rte_mempool *mp_nocache = NULL; > > - struct rte_mempool *mp_ext = NULL; > > struct rte_mempool *mp_stack = NULL; > > > > rte_atomic32_init(&synchro); > > @@ -605,7 +604,7 @@ test_mempool(void) > > err: > > rte_mempool_free(mp_nocache); > > rte_mempool_free(mp_cache); > > - rte_mempool_free(mp_ext); > > + rte_mempool_free(mp_stack); > > return -1; > > } > > > > > > Strange, it seems these modifications were present in latest patch from > David Hunt (v6). Maybe a bad manipulation during the push?
Yes sorry, it seems to be a wrong manipulation, the v5 was pushed. This patch seems to fix my error, thanks. > The "Fixes:" line should be added though. > > Acked-by: Olivier Matz <olivier.matz at 6wind.com>