geneve: Remove redundant socket checks.
authorpravin shelar <pshelar@ovn.org>
Mon, 21 Nov 2016 19:03:00 +0000 (11:03 -0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 21 Nov 2016 19:05:49 +0000 (14:05 -0500)
Geneve already has check for device socket in route
lookup function. So no need to check it in xmit
function.

Signed-off-by: Pravin B Shelar <pshelar@ovn.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/geneve.c

index 95dd19a3cafef35e79232fe2295a85563c0f918f..ab9fea6936a206b7a4c9becf5d0c0622d182da4f 100644 (file)
@@ -785,14 +785,11 @@ static int geneve_xmit_skb(struct sk_buff *skb, struct net_device *dev,
        struct geneve_sock *gs4 = rcu_dereference(geneve->sock4);
        const struct ip_tunnel_key *key = &info->key;
        struct rtable *rt;
-       int err = -EINVAL;
        struct flowi4 fl4;
        __u8 tos, ttl;
        __be16 sport;
        __be16 df;
-
-       if (!gs4)
-               return err;
+       int err;
 
        rt = geneve_get_v4_rt(skb, dev, &fl4, info);
        if (IS_ERR(rt))
@@ -828,13 +825,10 @@ static int geneve6_xmit_skb(struct sk_buff *skb, struct net_device *dev,
        struct geneve_sock *gs6 = rcu_dereference(geneve->sock6);
        const struct ip_tunnel_key *key = &info->key;
        struct dst_entry *dst = NULL;
-       int err = -EINVAL;
        struct flowi6 fl6;
        __u8 prio, ttl;
        __be16 sport;
-
-       if (!gs6)
-               return err;
+       int err;
 
        dst = geneve_get_v6_dst(skb, dev, &fl6, info);
        if (IS_ERR(dst))