Re: [ovs-dev] [PATCH] datapath-windows: Correct endianness for deleting zone.

2017-12-11 Thread aserdean
vs-dev- > boun...@openvswitch.org] On Behalf Of Sairam Venugopal > Sent: Friday, December 8, 2017 1:26 AM > To: aserd...@ovn.org; 'Justin Pettit' ; > d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH] datapath-windows: Correct endianness for > deleting zone. > &

Re: [ovs-dev] [PATCH] datapath-windows: Correct endianness for deleting zone.

2017-12-07 Thread Sairam Venugopal
in Gabriel Serdean ; >> Sairam Venugopal >> Subject: [ovs-dev] [PATCH] datapath-windows: Correct endianness for >> deleting zone. >> >> The zone Netlink attribute is supposed to be in network-byte order, but >the >> Windows code for deleting conntrack entries was t

Re: [ovs-dev] [PATCH] datapath-windows: Correct endianness for deleting zone.

2017-12-07 Thread Justin Pettit
> On Dec 7, 2017, at 2:57 PM, Sairam Venugopal wrote: > > Thanks for the patch. Discussed about the patch offline. > > This is required since we have now introduced a newer api for deleting > 5-tuples > and this breaks the assumption of zone-id not being in the right byte order. > > Acked-b

Re: [ovs-dev] [PATCH] datapath-windows: Correct endianness for deleting zone.

2017-12-07 Thread Sairam Venugopal
Thanks for the patch. Discussed about the patch offline. This is required since we have now introduced a newer api for deleting 5-tuples and this breaks the assumption of zone-id not being in the right byte order. Acked-by: Sairam Venugopal On 12/4/17, 11:30 PM, "Justin Pettit" wrote: >T

Re: [ovs-dev] [PATCH] datapath-windows: Correct endianness for deleting zone.

2017-12-06 Thread aserdean
> -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of Justin Pettit > Sent: Tuesday, December 5, 2017 9:30 AM > To: d...@openvswitch.org; Alin Gabriel Serdean ; > Sairam Venugopal > Subject: [ovs-dev] [PA

[ovs-dev] [PATCH] datapath-windows: Correct endianness for deleting zone.

2017-12-04 Thread Justin Pettit
The zone Netlink attribute is supposed to be in network-byte order, but the Windows code for deleting conntrack entries was treating it as host-byte order. Found by inspection. Signed-off-by: Justin Pettit --- datapath-windows/ovsext/Conntrack.c | 2 +- lib/netlink-conntrack.c | 2 +