diff options
author | Nikolay Aleksandrov <nikolay@cumulusnetworks.com> | 2015-10-12 15:47:04 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-10-13 07:57:56 -0400 |
commit | b8d02c3cace37393bf9ff0a9eaa1ee39cda1d259 (patch) | |
tree | 1027b4d4f71e3c7173b547453e5b560ca11c1da5 /net | |
parent | e9c953eff7f0ec69a52cfa87b912ab48902a0314 (diff) |
bridge: vlan: drop unnecessary flush code
As Ido Schimmel pointed out the vlan_vid_del() code in nbp_vlan_flush is
unnecessary (and is actually a remnant of the old vlan code) so we can
remove it.
Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Reviewed-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/bridge/br_vlan.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/net/bridge/br_vlan.c b/net/bridge/br_vlan.c index ffaa6d986505..85e67568092e 100644 --- a/net/bridge/br_vlan.c +++ b/net/bridge/br_vlan.c | |||
@@ -959,14 +959,7 @@ int nbp_vlan_delete(struct net_bridge_port *port, u16 vid) | |||
959 | 959 | ||
960 | void nbp_vlan_flush(struct net_bridge_port *port) | 960 | void nbp_vlan_flush(struct net_bridge_port *port) |
961 | { | 961 | { |
962 | struct net_bridge_vlan_group *vg; | ||
963 | struct net_bridge_vlan *vlan; | ||
964 | |||
965 | ASSERT_RTNL(); | 962 | ASSERT_RTNL(); |
966 | 963 | ||
967 | vg = nbp_vlan_group(port); | 964 | __vlan_flush(nbp_vlan_group(port)); |
968 | list_for_each_entry(vlan, &vg->vlan_list, vlist) | ||
969 | vlan_vid_del(port->dev, port->br->vlan_proto, vlan->vid); | ||
970 | |||
971 | __vlan_flush(vg); | ||
972 | } | 965 | } |