On 6/18/21 9:14 PM, Dumitru Ceara wrote:
>> Should this be "need_trim = true;"? Currently, it's impossible for
>> need_trim to ever be true in this function.
>>
> Ugh, you're right, sorry about that.  I'll also add a test to exercise
> this.  Thanks for the review!

Turns out I had a test already but it was wrong. :D

I fixed it in v3:
http://patchwork.ozlabs.org/project/ovn/list/?series=249720

Regards,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to