Re: [PATCH] test_mempool: remove unused mp_ext var

[Thomas Monjalon <thomas.monjalon@6wind.com>]

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@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@6wind.com>