Hi Santosh, On Thu, 11 May 2017 16:00:43 +0530, Santosh Shukla <santosh.shu...@caviumnetworks.com> wrote: > Cc: sta...@dpdk.org > Signed-off-by: Santosh Shukla <santosh.shu...@caviumnetworks.com> > --- > test/test/test_mbuf.c | 100 > +++++++++++++++++++++++++------------------------- > 1 file changed, 51 insertions(+), 49 deletions(-) >
Thank you for doing this fix. My only comment is that commit introduces a memory leak, which is fixed by your second patch. The behavior before your patch was to allocate the pools once but never free them. The behavior after this patch is to allocate the pools each time the test is started and never free them. After your 2 patches, the behavior is good. So I suggest to merge them into one commit. What do you think? Olivier
