diff options
author | Stephen Hemminger <shemminger@vyatta.com> | 2009-06-12 15:02:44 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-06-14 02:28:51 -0400 |
commit | d2991f75354941a4cdf61ce7443d21804b978f89 (patch) | |
tree | eb52cc7a67ba92c7f1eaa16dd77b6b46296126aa /drivers/net/bonding/bond_main.c | |
parent | 9d794a0c1dbb9551d870683bca0744624db353ed (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.c | 12 |
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; | |||
101 | static char *arp_ip_target[BOND_MAX_ARP_TARGETS] = { NULL, }; | 101 | static char *arp_ip_target[BOND_MAX_ARP_TARGETS] = { NULL, }; |
102 | static char *arp_validate = NULL; | 102 | static char *arp_validate = NULL; |
103 | static char *fail_over_mac = NULL; | 103 | static char *fail_over_mac = NULL; |
104 | struct bond_params bonding_defaults; | 104 | static struct bond_params bonding_defaults; |
105 | 105 | ||
106 | module_param(max_bonds, int, 0); | 106 | module_param(max_bonds, int, 0); |
107 | MODULE_PARM_DESC(max_bonds, "Max number of bonded devices"); | 107 | MODULE_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 | */ |
4595 | static int bond_init(struct net_device *bond_dev, struct bond_params *params) | 4595 | static 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 | */ |
5119 | int bond_create(char *name, struct bond_params *params) | 5119 | int 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 | } |