Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in tunnel Decap

2017-11-28 Thread Shashank Ram
, Alin. > -Original Message- > From: Shashank Ram [mailto:r...@vmware.com] > Sent: Tuesday, November 28, 2017 3:33 PM > To: d...@openvswitch.org; Alin Serdean > > Subject: Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in >

Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in tunnel Decap

2017-11-28 Thread Alin Serdean
vember 28, 2017 3:33 PM > To: d...@openvswitch.org; Alin Serdean > > Subject: Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in > tunnel Decap > > Alin, do you think it makes sense to also apply this on 2.8? > > -- > Thanks, > Shashank &g

Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in tunnel Decap

2017-11-28 Thread Shashank Ram
Alin, do you think it makes sense to also apply this on 2.8? -- Thanks, Shashank On 11/28/17, 11:47 AM, "ovs-dev-boun...@openvswitch.org on behalf of Shashank Ram" wrote: Alin, can we apply this patch? It’s been sitting around in the mailing list for a week without any reviews after an A

Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in tunnel Decap

2017-11-28 Thread aserdean
Thanks Anand and Shashank, I applied this on master. > -Original Message- > From: Shashank Ram [mailto:r...@vmware.com] > Sent: Tuesday, November 28, 2017 8:18 AM > To: Anand Kumar ; d...@openvswitch.org; Alin > Serdean > Subject: Re: [ovs-dev] [PATCH] datapath-windows

Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in tunnel Decap

2017-11-27 Thread Shashank Ram
Alin, can we apply this patch? It’s been sitting around in the mailing list for a week without any reviews after an ACK. -- Thanks, Shashank On 11/21/17, 11:18 PM, "Anand Kumar" wrote: Acked-by: Anand Kumar Thanks, Anand Kumar On 11/20/17, 3:06 PM, "ovs-dev-boun...@

Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in tunnel Decap

2017-11-21 Thread Anand Kumar
Acked-by: Anand Kumar Thanks, Anand Kumar On 11/20/17, 3:06 PM, "ovs-dev-boun...@openvswitch.org on behalf of Shashank Ram" wrote: Decap functions for tunneling protocols do not compute the packet header offsets correctly when there is a VLAN tag in the L2 header. This results in

[ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in tunnel Decap

2017-11-20 Thread Shashank Ram
Decap functions for tunneling protocols do not compute the packet header offsets correctly when there is a VLAN tag in the L2 header. This results in incorrect checksum computation causing the packet to be dropped. This patch adds support to account for the VLAN tag in the packet if its present, a