diff options
| author | Konstantin Khlebnikov <khlebnikov@yandex-team.ru> | 2015-07-14 09:35:50 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-07-16 00:33:39 -0400 |
| commit | 515866f8185b92fb18a782408c53839f003c7669 (patch) | |
| tree | 1a3a1efb934a21e0327e79521338bd63ab34c193 /drivers/net/ipvlan | |
| parent | 20b953edf9cfbba6b7c8cfbd33a30f95f6878f7b (diff) | |
ipvlan: remove counters of ipv4 and ipv6 addresses
They are unused after commit f631c44bbe15 ("ipvlan: Always set broadcast bit in
multicast filter").
Signed-off-by: Konstantin Khlebnikov <khlebnikov@yandex-team.ru>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ipvlan')
| -rw-r--r-- | drivers/net/ipvlan/ipvlan.h | 2 | ||||
| -rw-r--r-- | drivers/net/ipvlan/ipvlan_main.c | 33 |
2 files changed, 12 insertions, 23 deletions
diff --git a/drivers/net/ipvlan/ipvlan.h b/drivers/net/ipvlan/ipvlan.h index 953a97492fab..68e2549c28c6 100644 --- a/drivers/net/ipvlan/ipvlan.h +++ b/drivers/net/ipvlan/ipvlan.h | |||
| @@ -67,8 +67,6 @@ struct ipvl_dev { | |||
| 67 | struct ipvl_port *port; | 67 | struct ipvl_port *port; |
| 68 | struct net_device *phy_dev; | 68 | struct net_device *phy_dev; |
| 69 | struct list_head addrs; | 69 | struct list_head addrs; |
| 70 | int ipv4cnt; | ||
| 71 | int ipv6cnt; | ||
| 72 | struct ipvl_pcpu_stats __percpu *pcpu_stats; | 70 | struct ipvl_pcpu_stats __percpu *pcpu_stats; |
| 73 | DECLARE_BITMAP(mac_filters, IPVLAN_MAC_FILTER_SIZE); | 71 | DECLARE_BITMAP(mac_filters, IPVLAN_MAC_FILTER_SIZE); |
| 74 | netdev_features_t sfeatures; | 72 | netdev_features_t sfeatures; |
diff --git a/drivers/net/ipvlan/ipvlan_main.c b/drivers/net/ipvlan/ipvlan_main.c index 1acc283160d9..048ecf0c76fb 100644 --- a/drivers/net/ipvlan/ipvlan_main.c +++ b/drivers/net/ipvlan/ipvlan_main.c | |||
| @@ -153,10 +153,9 @@ static int ipvlan_open(struct net_device *dev) | |||
| 153 | else | 153 | else |
| 154 | dev->flags &= ~IFF_NOARP; | 154 | dev->flags &= ~IFF_NOARP; |
| 155 | 155 | ||
| 156 | if (ipvlan->ipv6cnt > 0 || ipvlan->ipv4cnt > 0) { | 156 | list_for_each_entry(addr, &ipvlan->addrs, anode) |
| 157 | list_for_each_entry(addr, &ipvlan->addrs, anode) | 157 | ipvlan_ht_addr_add(ipvlan, addr); |
| 158 | ipvlan_ht_addr_add(ipvlan, addr); | 158 | |
| 159 | } | ||
| 160 | return dev_uc_add(phy_dev, phy_dev->dev_addr); | 159 | return dev_uc_add(phy_dev, phy_dev->dev_addr); |
| 161 | } | 160 | } |
| 162 | 161 | ||
| @@ -171,10 +170,9 @@ static int ipvlan_stop(struct net_device *dev) | |||
| 171 | 170 | ||
| 172 | dev_uc_del(phy_dev, phy_dev->dev_addr); | 171 | dev_uc_del(phy_dev, phy_dev->dev_addr); |
| 173 | 172 | ||
| 174 | if (ipvlan->ipv6cnt > 0 || ipvlan->ipv4cnt > 0) { | 173 | list_for_each_entry(addr, &ipvlan->addrs, anode) |
| 175 | list_for_each_entry(addr, &ipvlan->addrs, anode) | 174 | ipvlan_ht_addr_del(addr, !dev->dismantle); |
| 176 | ipvlan_ht_addr_del(addr, !dev->dismantle); | 175 | |
| 177 | } | ||
| 178 | return 0; | 176 | return 0; |
| 179 | } | 177 | } |
| 180 | 178 | ||
| @@ -471,8 +469,6 @@ static int ipvlan_link_new(struct net *src_net, struct net_device *dev, | |||
| 471 | ipvlan->port = port; | 469 | ipvlan->port = port; |
| 472 | ipvlan->sfeatures = IPVLAN_FEATURES; | 470 | ipvlan->sfeatures = IPVLAN_FEATURES; |
| 473 | INIT_LIST_HEAD(&ipvlan->addrs); | 471 | INIT_LIST_HEAD(&ipvlan->addrs); |
| 474 | ipvlan->ipv4cnt = 0; | ||
| 475 | ipvlan->ipv6cnt = 0; | ||
| 476 | 472 | ||
| 477 | /* TODO Probably put random address here to be presented to the | 473 | /* TODO Probably put random address here to be presented to the |
| 478 | * world but keep using the physical-dev address for the outgoing | 474 | * world but keep using the physical-dev address for the outgoing |
| @@ -508,12 +504,11 @@ static void ipvlan_link_delete(struct net_device *dev, struct list_head *head) | |||
| 508 | struct ipvl_dev *ipvlan = netdev_priv(dev); | 504 | struct ipvl_dev *ipvlan = netdev_priv(dev); |
| 509 | struct ipvl_addr *addr, *next; | 505 | struct ipvl_addr *addr, *next; |
| 510 | 506 | ||
| 511 | if (ipvlan->ipv6cnt > 0 || ipvlan->ipv4cnt > 0) { | 507 | list_for_each_entry_safe(addr, next, &ipvlan->addrs, anode) { |
| 512 | list_for_each_entry_safe(addr, next, &ipvlan->addrs, anode) { | 508 | ipvlan_ht_addr_del(addr, !dev->dismantle); |
| 513 | ipvlan_ht_addr_del(addr, !dev->dismantle); | 509 | list_del(&addr->anode); |
| 514 | list_del(&addr->anode); | ||
| 515 | } | ||
| 516 | } | 510 | } |
| 511 | |||
| 517 | list_del_rcu(&ipvlan->pnode); | 512 | list_del_rcu(&ipvlan->pnode); |
| 518 | unregister_netdevice_queue(dev, head); | 513 | unregister_netdevice_queue(dev, head); |
| 519 | netdev_upper_dev_unlink(ipvlan->phy_dev, dev); | 514 | netdev_upper_dev_unlink(ipvlan->phy_dev, dev); |
| @@ -627,7 +622,7 @@ static int ipvlan_add_addr6(struct ipvl_dev *ipvlan, struct in6_addr *ip6_addr) | |||
| 627 | memcpy(&addr->ip6addr, ip6_addr, sizeof(struct in6_addr)); | 622 | memcpy(&addr->ip6addr, ip6_addr, sizeof(struct in6_addr)); |
| 628 | addr->atype = IPVL_IPV6; | 623 | addr->atype = IPVL_IPV6; |
| 629 | list_add_tail(&addr->anode, &ipvlan->addrs); | 624 | list_add_tail(&addr->anode, &ipvlan->addrs); |
| 630 | ipvlan->ipv6cnt++; | 625 | |
| 631 | /* If the interface is not up, the address will be added to the hash | 626 | /* If the interface is not up, the address will be added to the hash |
| 632 | * list by ipvlan_open. | 627 | * list by ipvlan_open. |
| 633 | */ | 628 | */ |
| @@ -647,8 +642,6 @@ static void ipvlan_del_addr6(struct ipvl_dev *ipvlan, struct in6_addr *ip6_addr) | |||
| 647 | 642 | ||
| 648 | ipvlan_ht_addr_del(addr, true); | 643 | ipvlan_ht_addr_del(addr, true); |
| 649 | list_del(&addr->anode); | 644 | list_del(&addr->anode); |
| 650 | ipvlan->ipv6cnt--; | ||
| 651 | WARN_ON(ipvlan->ipv6cnt < 0); | ||
| 652 | kfree_rcu(addr, rcu); | 645 | kfree_rcu(addr, rcu); |
| 653 | 646 | ||
| 654 | return; | 647 | return; |
| @@ -699,7 +692,7 @@ static int ipvlan_add_addr4(struct ipvl_dev *ipvlan, struct in_addr *ip4_addr) | |||
| 699 | memcpy(&addr->ip4addr, ip4_addr, sizeof(struct in_addr)); | 692 | memcpy(&addr->ip4addr, ip4_addr, sizeof(struct in_addr)); |
| 700 | addr->atype = IPVL_IPV4; | 693 | addr->atype = IPVL_IPV4; |
| 701 | list_add_tail(&addr->anode, &ipvlan->addrs); | 694 | list_add_tail(&addr->anode, &ipvlan->addrs); |
| 702 | ipvlan->ipv4cnt++; | 695 | |
| 703 | /* If the interface is not up, the address will be added to the hash | 696 | /* If the interface is not up, the address will be added to the hash |
| 704 | * list by ipvlan_open. | 697 | * list by ipvlan_open. |
| 705 | */ | 698 | */ |
| @@ -719,8 +712,6 @@ static void ipvlan_del_addr4(struct ipvl_dev *ipvlan, struct in_addr *ip4_addr) | |||
| 719 | 712 | ||
| 720 | ipvlan_ht_addr_del(addr, true); | 713 | ipvlan_ht_addr_del(addr, true); |
| 721 | list_del(&addr->anode); | 714 | list_del(&addr->anode); |
| 722 | ipvlan->ipv4cnt--; | ||
| 723 | WARN_ON(ipvlan->ipv4cnt < 0); | ||
| 724 | kfree_rcu(addr, rcu); | 715 | kfree_rcu(addr, rcu); |
| 725 | 716 | ||
| 726 | return; | 717 | return; |
