On Thu, Mar 16, 2023 at 06:19:44AM +0000, Naveen Yerramneni wrote:

The following text seems to have gone missing since v2:

  Add OpenFlow extn to set conntrack entries limit per zone.

I think it would be nice to expand on why this change is being made.
But it is not a hard requirement from me.

The code looks good to me, thanks for addressing by review of v2.



> Signed-off-by: Naveen Yerramneni <[email protected]>
> Reviewed-by: Simon Horman <[email protected]>
> ---
> Notes:
>   v1 -> v2
>   - Fix memory leak and added logs
>   v2 -> v3
>   - Addressed nits

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

Reply via email to