Hello. On 20-06-2013 11:26, Pravin B Shelar wrote:
Following patch allows transmit side vlan offload for vxlan devices.
Signed-off-by: Pravin B Shelar <pshe...@nicira.com> --- drivers/net/vxlan.c | 15 ++++++++++++++- 1 files changed, 14 insertions(+), 1 deletions(-)
diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c index e53c947..a7feba4 100644 --- a/drivers/net/vxlan.c +++ b/drivers/net/vxlan.c
[...]
@@ -1049,13 +1050,23 @@ int vxlan_xmit_skb(struct net *net, struct vxlan_handler *vh,
[...]
+ if (vlan_tx_tag_present(skb)) { + if (unlikely(!__vlan_put_tag(skb, + skb->vlan_proto, + vlan_tx_tag_get(skb)))) { + return -ENOMEM; + }
You haven't run this patch thru scripts/checkpatch.pl, have you? It should have warned you about using {} on single statement branch.
+ skb->vlan_tci = 0; + } +
WBR, Sergei _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev