Commit 49f17de7 authored by Veaceslav Falico's avatar Veaceslav Falico Committed by David S. Miller

bonding: rename last_arp_rx to last_rx

To reflect the new meaning.

CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: default avatarVeaceslav Falico <vfalico@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 8e603460
...@@ -1397,10 +1397,10 @@ int bond_enslave(struct net_device *bond_dev, struct net_device *slave_dev) ...@@ -1397,10 +1397,10 @@ int bond_enslave(struct net_device *bond_dev, struct net_device *slave_dev)
bond_update_speed_duplex(new_slave); bond_update_speed_duplex(new_slave);
new_slave->last_arp_rx = jiffies - new_slave->last_rx = jiffies -
(msecs_to_jiffies(bond->params.arp_interval) + 1); (msecs_to_jiffies(bond->params.arp_interval) + 1);
for (i = 0; i < BOND_MAX_ARP_TARGETS; i++) for (i = 0; i < BOND_MAX_ARP_TARGETS; i++)
new_slave->target_last_arp_rx[i] = new_slave->last_arp_rx; new_slave->target_last_arp_rx[i] = new_slave->last_rx;
if (bond->params.miimon && !bond->params.use_carrier) { if (bond->params.miimon && !bond->params.use_carrier) {
link_reporting = bond_check_dev_link(bond, slave_dev, 1); link_reporting = bond_check_dev_link(bond, slave_dev, 1);
...@@ -2242,7 +2242,7 @@ static void bond_validate_arp(struct bonding *bond, struct slave *slave, __be32 ...@@ -2242,7 +2242,7 @@ static void bond_validate_arp(struct bonding *bond, struct slave *slave, __be32
pr_debug("bva: sip %pI4 not found in targets\n", &sip); pr_debug("bva: sip %pI4 not found in targets\n", &sip);
return; return;
} }
slave->last_arp_rx = jiffies; slave->last_rx = jiffies;
slave->target_last_arp_rx[i] = jiffies; slave->target_last_arp_rx[i] = jiffies;
} }
...@@ -2257,7 +2257,7 @@ int bond_arp_rcv(const struct sk_buff *skb, struct bonding *bond, ...@@ -2257,7 +2257,7 @@ int bond_arp_rcv(const struct sk_buff *skb, struct bonding *bond,
if (!slave_do_arp_validate(bond, slave)) { if (!slave_do_arp_validate(bond, slave)) {
if ((slave_do_arp_validate_only(bond, slave) && is_arp) || if ((slave_do_arp_validate_only(bond, slave) && is_arp) ||
!slave_do_arp_validate_only(bond, slave)) !slave_do_arp_validate_only(bond, slave))
slave->last_arp_rx = jiffies; slave->last_rx = jiffies;
return RX_HANDLER_ANOTHER; return RX_HANDLER_ANOTHER;
} else if (!is_arp) { } else if (!is_arp) {
return RX_HANDLER_ANOTHER; return RX_HANDLER_ANOTHER;
...@@ -2369,7 +2369,7 @@ static void bond_loadbalance_arp_mon(struct work_struct *work) ...@@ -2369,7 +2369,7 @@ static void bond_loadbalance_arp_mon(struct work_struct *work)
if (slave->link != BOND_LINK_UP) { if (slave->link != BOND_LINK_UP) {
if (bond_time_in_interval(bond, trans_start, 1) && if (bond_time_in_interval(bond, trans_start, 1) &&
bond_time_in_interval(bond, slave->last_arp_rx, 1)) { bond_time_in_interval(bond, slave->last_rx, 1)) {
slave->link = BOND_LINK_UP; slave->link = BOND_LINK_UP;
slave_state_changed = 1; slave_state_changed = 1;
...@@ -2398,7 +2398,7 @@ static void bond_loadbalance_arp_mon(struct work_struct *work) ...@@ -2398,7 +2398,7 @@ static void bond_loadbalance_arp_mon(struct work_struct *work)
* if we don't know our ip yet * if we don't know our ip yet
*/ */
if (!bond_time_in_interval(bond, trans_start, 2) || if (!bond_time_in_interval(bond, trans_start, 2) ||
!bond_time_in_interval(bond, slave->last_arp_rx, 2)) { !bond_time_in_interval(bond, slave->last_rx, 2)) {
slave->link = BOND_LINK_DOWN; slave->link = BOND_LINK_DOWN;
slave_state_changed = 1; slave_state_changed = 1;
......
...@@ -190,7 +190,7 @@ struct slave { ...@@ -190,7 +190,7 @@ struct slave {
int delay; int delay;
/* all three in jiffies */ /* all three in jiffies */
unsigned long last_link_up; unsigned long last_link_up;
unsigned long last_arp_rx; unsigned long last_rx;
unsigned long target_last_arp_rx[BOND_MAX_ARP_TARGETS]; unsigned long target_last_arp_rx[BOND_MAX_ARP_TARGETS];
s8 link; /* one of BOND_LINK_XXXX */ s8 link; /* one of BOND_LINK_XXXX */
s8 new_link; s8 new_link;
...@@ -383,7 +383,7 @@ static inline unsigned long slave_last_rx(struct bonding *bond, ...@@ -383,7 +383,7 @@ static inline unsigned long slave_last_rx(struct bonding *bond,
if (bond->params.arp_all_targets == BOND_ARP_TARGETS_ALL) if (bond->params.arp_all_targets == BOND_ARP_TARGETS_ALL)
return slave_oldest_target_arp_rx(bond, slave); return slave_oldest_target_arp_rx(bond, slave);
return slave->last_arp_rx; return slave->last_rx;
} }
#ifdef CONFIG_NET_POLL_CONTROLLER #ifdef CONFIG_NET_POLL_CONTROLLER
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment