Re: [ovs-dev] [PATCH v2] datapath-windows: Add support to configure ct zone limits

2018-09-12 Thread Anand Kumar
Hi Shashank, Thanks for catching this. I have addressed this and send out a v3 of the patch. Regards, Anand Kumar On 8/31/18, 10:12 AM, "Shashank Ram" wrote: Hi Anand, one thing looks off to me here. In the signature of OvsCreateNlMsgFromCtLimit(), numAttrs is UINT32, but the

Re: [ovs-dev] [PATCH v2] datapath-windows: Add support to configure ct zone limits

2018-08-31 Thread Shashank Ram
Hi Anand, one thing looks off to me here. In the signature of OvsCreateNlMsgFromCtLimit(), numAttrs is UINT32, but the loop’s index variable is an UINT16: for (UINT16 i = 0; i < numAttrs; i++) Please address this discrepancy. -- Thanks, Shashank On 8/28/18, 10:37 AM,

[ovs-dev] [PATCH v2] datapath-windows: Add support to configure ct zone limits

2018-08-28 Thread Anand Kumar
This patch implements limiting conntrack entries per zone using dpctl commands. Example: ovs-appctl dpctl/ct-set-limits default=5 zone=1,limit=2 zone=1,limit=3 ovs-appctl dpct/ct-del-limits zone=4 ovs-appctl dpct/ct-get-limits zone=1,2,3 - Also update the netlink-socket.c to support netlink