Thanks, send v3 with the suggested changes.

From: Rao, Nikhil
Sent: Monday, May 21, 2018 5:09 PM
To: Varghese, Vipin <[email protected]>; [email protected]; Jain, Deepak K 
<[email protected]>
Cc: Rao, Nikhil <[email protected]>
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 <[email protected]><mailto:[email protected]>

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

Reply via email to