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

Reply via email to