bonding: change the bond's vlan syncing functions with the standard ones
authornikolay@redhat.com <nikolay@redhat.com>
Tue, 6 Aug 2013 10:40:15 +0000 (12:40 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 9 Aug 2013 05:31:21 +0000 (22:31 -0700)
commit1ff412ad7714f6952f76ffd77f0a7f2f563288a1
tree01ce83a0d0cf057a0683fde6a645f1f2727b1f60
parent1f07d03e2069df2ecd82301936b598a3b257c6d6
bonding: change the bond's vlan syncing functions with the standard ones

Now we have vlan_vids_add/del_by_dev() which serve the same purpose as
bond's bond_add/del_vlans_on_slave() with the good side effect of
reverting the changes if one of the additions fails.
There's only 1 change in the behaviour of enslave: if adding of the
vlans to the slave fails, we'll fail the enslaving because otherwise we
might delete some vlan that wasn't added by the bonding.
The only way this may happen is with ENOMEM currently, so we're in trouble
anyway.

Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>
Acked-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c