bonding: remove bond_next_slave()
authorVeaceslav Falico <vfalico@redhat.com>
Fri, 27 Sep 2013 14:12:05 +0000 (16:12 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sat, 28 Sep 2013 22:28:07 +0000 (15:28 -0700)
There are no users left, so it's safe to remove.

CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bonding.h

index 5b71601..713b6af 100644 (file)
@@ -89,9 +89,6 @@
 #define bond_is_first_slave(bond, pos) (pos == bond_first_slave(bond))
 #define bond_is_last_slave(bond, pos) (pos == bond_last_slave(bond))
 
-/* Since bond_first/last_slave can return NULL, these can return NULL too */
-#define bond_next_slave(bond, pos) __bond_next_slave(bond, pos)
-
 /**
  * bond_for_each_slave - iterate over all slaves
  * @bond:      the bond holding this list
@@ -244,34 +241,6 @@ struct bonding {
        ((struct slave *) rtnl_dereference(dev->rx_handler_data))
 
 /**
- * __bond_next_slave - get the next slave after the one provided
- * @bond - bonding struct
- * @slave - the slave provided
- *
- * Returns the next slave after the slave provided, first slave if the
- * slave provided is the last slave and NULL if slave is not found
- */
-static inline struct slave *__bond_next_slave(struct bonding *bond,
-                                             struct slave *slave)
-{
-       struct slave *slave_iter;
-       struct list_head *iter;
-       bool found = false;
-
-       netdev_for_each_lower_private(bond->dev, slave_iter, iter) {
-               if (found)
-                       return slave_iter;
-               if (slave_iter == slave)
-                       found = true;
-       }
-
-       if (found)
-               return bond_first_slave(bond);
-
-       return NULL;
-}
-
-/**
  * Returns NULL if the net_device does not belong to any of the bond's slaves
  *
  * Caller must hold bond lock for read