On Wed, Nov 25, 2015 at 11:43 PM, Jesse Gross <je...@kernel.org> wrote:
> On Wed, Nov 25, 2015 at 6:42 AM, Pravin B Shelar <pshe...@nicira.com> wrote:
>> diff --git a/datapath/linux/compat/geneve.c b/datapath/linux/compat/geneve.c
>> index 85cf95f..ee61537 100644
>> --- a/datapath/linux/compat/geneve.c
>> +++ b/datapath/linux/compat/geneve.c
>> +static int geneve_build_skb(struct rtable *rt, struct sk_buff *skb,
>> + __be16 tun_flags, u8 vni[3], u8 opt_len, u8 *opt,
>> + bool csum)
> [...]
>> + skb = udp_tunnel_handle_offloads(skb, csum, 0, false);
>> + if (IS_ERR(skb)) {
>> + err = PTR_ERR(skb);
>> + goto free_rt;
>> + }
>> +
>> + skb = vlan_hwaccel_push_inside(skb);
>> + if (!skb) {
>> + err = -ENOMEM;
>> + goto free_rt;
>> + }
>
> I think that vlan_hwaccel_push_inside() needs to go before
> udp_tunnel_handle_offloads(). Otherwise, we will set the inner offsets
> and then change at least the MAC header location.
ok, I have sent out updated patch.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
