Thanks, send v3 with the suggested changes. From: Rao, Nikhil Sent: Monday, May 21, 2018 5:09 PM To: Varghese, Vipin <vipin.vargh...@intel.com>; dev@dpdk.org; Jain, Deepak K <deepak.k.j...@intel.com> Cc: Rao, Nikhil <nikhil....@intel.com> Subject: Re: [PATCH] test/eventdev: add multi port test to suite
Hi Vipin, Thanks for posting V2. On 5/21/2018 4:30 PM, Varghese, Vipin wrote: Hi Nikhil, Thanks for the inputs, I have shared the V2 with suggested changes. As for <snipped> + adapter_free(); Is it necessary to call adapter_free() above ? <Snipped> With the changes suggested above Acked-by: Nikhil Rao <nikhil....@intel.com><mailto:nikhil....@intel.com> Yes it not required by current test sequence. This is added for safety, in case in previous free is incomplete or interrupted. I don't see a case where the previous free can be incomplete or interrupted. Nikhil