aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net/bonding/bond_main.c
diff options
context:
space:
mode:
authorStephen Hemminger <shemminger@vyatta.com>2009-06-12 15:02:44 -0400
committerDavid S. Miller <davem@davemloft.net>2009-06-14 02:28:51 -0400
commitd2991f75354941a4cdf61ce7443d21804b978f89 (patch)
treeeb52cc7a67ba92c7f1eaa16dd77b6b46296126aa /drivers/net/bonding/bond_main.c
parent9d794a0c1dbb9551d870683bca0744624db353ed (diff)
bonding: bond_create always called with default parameters
bond_create() is always called with same parameters so move the argument down. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding/bond_main.c')
-rw-r--r--drivers/net/bonding/bond_main.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c
index 2f4329e91a4c..a1fa8deb254e 100644
--- a/drivers/net/bonding/bond_main.c
+++ b/drivers/net/bonding/bond_main.c
@@ -101,7 +101,7 @@ static int arp_interval = BOND_LINK_ARP_INTERV;
101static char *arp_ip_target[BOND_MAX_ARP_TARGETS] = { NULL, }; 101static char *arp_ip_target[BOND_MAX_ARP_TARGETS] = { NULL, };
102static char *arp_validate = NULL; 102static char *arp_validate = NULL;
103static char *fail_over_mac = NULL; 103static char *fail_over_mac = NULL;
104struct bond_params bonding_defaults; 104static struct bond_params bonding_defaults;
105 105
106module_param(max_bonds, int, 0); 106module_param(max_bonds, int, 0);
107MODULE_PARM_DESC(max_bonds, "Max number of bonded devices"); 107MODULE_PARM_DESC(max_bonds, "Max number of bonded devices");
@@ -4592,7 +4592,7 @@ static const struct net_device_ops bond_netdev_ops = {
4592 * Does not allocate but creates a /proc entry. 4592 * Does not allocate but creates a /proc entry.
4593 * Allowed to fail. 4593 * Allowed to fail.
4594 */ 4594 */
4595static int bond_init(struct net_device *bond_dev, struct bond_params *params) 4595static int bond_init(struct net_device *bond_dev)
4596{ 4596{
4597 struct bonding *bond = netdev_priv(bond_dev); 4597 struct bonding *bond = netdev_priv(bond_dev);
4598 4598
@@ -4602,7 +4602,7 @@ static int bond_init(struct net_device *bond_dev, struct bond_params *params)
4602 rwlock_init(&bond->lock); 4602 rwlock_init(&bond->lock);
4603 rwlock_init(&bond->curr_slave_lock); 4603 rwlock_init(&bond->curr_slave_lock);
4604 4604
4605 bond->params = *params; /* copy params struct */ 4605 bond->params = bonding_defaults;
4606 4606
4607 bond->wq = create_singlethread_workqueue(bond_dev->name); 4607 bond->wq = create_singlethread_workqueue(bond_dev->name);
4608 if (!bond->wq) 4608 if (!bond->wq)
@@ -5116,7 +5116,7 @@ static void bond_set_lockdep_class(struct net_device *dev)
5116 * Caller must NOT hold rtnl_lock; we need to release it here before we 5116 * Caller must NOT hold rtnl_lock; we need to release it here before we
5117 * set up our sysfs entries. 5117 * set up our sysfs entries.
5118 */ 5118 */
5119int bond_create(char *name, struct bond_params *params) 5119int bond_create(const char *name)
5120{ 5120{
5121 struct net_device *bond_dev; 5121 struct net_device *bond_dev;
5122 struct bonding *bond; 5122 struct bonding *bond;
@@ -5159,7 +5159,7 @@ int bond_create(char *name, struct bond_params *params)
5159 * need to set function pointers. 5159 * need to set function pointers.
5160 */ 5160 */
5161 5161
5162 res = bond_init(bond_dev, params); 5162 res = bond_init(bond_dev);
5163 if (res < 0) { 5163 if (res < 0) {
5164 goto out_netdev; 5164 goto out_netdev;
5165 } 5165 }
@@ -5212,7 +5212,7 @@ static int __init bonding_init(void)
5212 init_rwsem(&bonding_rwsem); 5212 init_rwsem(&bonding_rwsem);
5213 5213
5214 for (i = 0; i < max_bonds; i++) { 5214 for (i = 0; i < max_bonds; i++) {
5215 res = bond_create(NULL, &bonding_defaults); 5215 res = bond_create(NULL);
5216 if (res) 5216 if (res)
5217 goto err; 5217 goto err;
5218 } 5218 }