ipvlan: unhash addresses without synchronize_rcu
authorKonstantin Khlebnikov <khlebnikov@yandex-team.ru>
Tue, 14 Jul 2015 13:35:53 +0000 (16:35 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 16 Jul 2015 04:33:39 +0000 (21:33 -0700)
All structures used in traffic forwarding are rcu-protected:
ipvl_addr, ipvl_dev and ipvl_port. Thus we can unhash addresses
without synchronization. We'll anyway hash it back into the same
bucket: in worst case lockless lookup will scan hash once again.

Signed-off-by: Konstantin Khlebnikov <khlebnikov@yandex-team.ru>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ipvlan/ipvlan.h
drivers/net/ipvlan/ipvlan_core.c
drivers/net/ipvlan/ipvlan_main.c

index 68e2549c28c6813826b2e49543514f4505ce2141..40f9d7e4a0ea7e0d42cb4ce675fcfb668d7a770c 100644 (file)
@@ -122,5 +122,5 @@ struct ipvl_addr *ipvlan_find_addr(const struct ipvl_dev *ipvlan,
 bool ipvlan_addr_busy(struct ipvl_port *port, void *iaddr, bool is_v6);
 struct ipvl_addr *ipvlan_ht_addr_lookup(const struct ipvl_port *port,
                                        const void *iaddr, bool is_v6);
-void ipvlan_ht_addr_del(struct ipvl_addr *addr, bool sync);
+void ipvlan_ht_addr_del(struct ipvl_addr *addr);
 #endif /* __IPVLAN_H */
index 8afbedad620d9ed27576dc6426878ae858979a51..8f8628a0adba3a837796df5a0e7c6284293481bd 100644 (file)
@@ -85,11 +85,9 @@ void ipvlan_ht_addr_add(struct ipvl_dev *ipvlan, struct ipvl_addr *addr)
                hlist_add_head_rcu(&addr->hlnode, &port->hlhead[hash]);
 }
 
-void ipvlan_ht_addr_del(struct ipvl_addr *addr, bool sync)
+void ipvlan_ht_addr_del(struct ipvl_addr *addr)
 {
        hlist_del_init_rcu(&addr->hlnode);
-       if (sync)
-               synchronize_rcu();
 }
 
 struct ipvl_addr *ipvlan_find_addr(const struct ipvl_dev *ipvlan,
index 7d81e37c3f76dc3a2af5ba73e957cf5eeab0c7db..e995bc501ee6ce08ca337532a356839a837b75d9 100644 (file)
@@ -171,7 +171,7 @@ static int ipvlan_stop(struct net_device *dev)
        dev_uc_del(phy_dev, phy_dev->dev_addr);
 
        list_for_each_entry(addr, &ipvlan->addrs, anode)
-               ipvlan_ht_addr_del(addr, !dev->dismantle);
+               ipvlan_ht_addr_del(addr);
 
        return 0;
 }
@@ -505,7 +505,7 @@ static void ipvlan_link_delete(struct net_device *dev, struct list_head *head)
        struct ipvl_addr *addr, *next;
 
        list_for_each_entry_safe(addr, next, &ipvlan->addrs, anode) {
-               ipvlan_ht_addr_del(addr, !dev->dismantle);
+               ipvlan_ht_addr_del(addr);
                list_del(&addr->anode);
                kfree_rcu(addr, rcu);
        }
@@ -641,7 +641,7 @@ static void ipvlan_del_addr6(struct ipvl_dev *ipvlan, struct in6_addr *ip6_addr)
        if (!addr)
                return;
 
-       ipvlan_ht_addr_del(addr, true);
+       ipvlan_ht_addr_del(addr);
        list_del(&addr->anode);
        kfree_rcu(addr, rcu);
 
@@ -711,7 +711,7 @@ static void ipvlan_del_addr4(struct ipvl_dev *ipvlan, struct in_addr *ip4_addr)
        if (!addr)
                return;
 
-       ipvlan_ht_addr_del(addr, true);
+       ipvlan_ht_addr_del(addr);
        list_del(&addr->anode);
        kfree_rcu(addr, rcu);