Re: [ovs-dev] [PATCH v7 1/4] datapath-windows: Add support for NAT in conntrack

2017-05-16 Thread Sairam Venugopal
Looks like outlook messed up my formatting. I had 2 comments in this patch. The default case statement is missing and 2 other cases aren't handled: case OVS_NAT_ATTR_UNSPEC: break; case __OVS_NAT_ATTR_MAX: break; default: ; OvsCtExecute_ - parameters exceed

Re: [ovs-dev] [PATCH v7 1/4] datapath-windows: Add support for NAT in conntrack

2017-05-16 Thread Sairam Venugopal
Not sure why the "From: Anand Kumar ” shows up in the commit description. Since they are already listed as co-author, it should be safe to remove this. Acked-by: Sairam Venugopal On 5/9/17, 3:59 PM, "ovs-dev-boun...@openvswitch.org on behalf of