On 4/6/20 3:03 PM, Vishal Deep Ajmera wrote:
> Hi Matteo,
> 
> Thanks for the patch. But I fail to understand why there is a memory leak.
> In fact when I tested in my setup, the test ran without any leak.

The main part is:
'_REFILL_' loop removes packets from the original batch and  
dp_execute_output_action()
doesn't free them if 'should_steal' equals to false.  Since, 
dp_execute_output_action()
returns success, '_REFILL_' doesn't put packet back to original batch leaking 
it.

You need lb_output to not be the last action to reproduce.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to