2005-04-16 16:20:36 -06:00
/*
* Copyright ( c ) 1999 - 2004 Intel Corporation . All rights reserved .
*
* This program is free software ; you can redistribute it and / or modify it
* under the terms of the GNU General Public License as published by the
* Free Software Foundation ; either version 2 of the License , or
* ( at your option ) any later version .
*
* This program is distributed in the hope that it will be useful , but
* WITHOUT ANY WARRANTY ; without even the implied warranty of MERCHANTABILITY
* or FITNESS FOR A PARTICULAR PURPOSE . See the GNU General Public License
* for more details .
*
* You should have received a copy of the GNU General Public License along
* with this program ; if not , write to the Free Software Foundation , Inc . ,
* 59 Temple Place - Suite 330 , Boston , MA 02111 - 1307 , USA .
*
* The full GNU General Public License is included in this distribution in the
* file called LICENSE .
*
*/
2009-12-13 21:06:07 -07:00
# define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
2005-04-16 16:20:36 -06:00
# include <linux/skbuff.h>
# include <linux/netdevice.h>
# include <linux/etherdevice.h>
# include <linux/pkt_sched.h>
# include <linux/spinlock.h>
# include <linux/slab.h>
# include <linux/timer.h>
# include <linux/ip.h>
# include <linux/ipv6.h>
# include <linux/if_arp.h>
# include <linux/if_ether.h>
# include <linux/if_bonding.h>
# include <linux/if_vlan.h>
# include <linux/in.h>
# include <net/ipx.h>
# include <net/arp.h>
2008-08-28 13:38:41 -06:00
# include <net/ipv6.h>
2005-04-16 16:20:36 -06:00
# include <asm/byteorder.h>
# include "bonding.h"
# include "bond_alb.h"
2009-09-01 00:31:18 -06:00
# ifndef __long_aligned
# define __long_aligned __attribute__((aligned((sizeof(long)))))
# endif
static const u8 mac_bcast [ ETH_ALEN ] __long_aligned = {
0xff , 0xff , 0xff , 0xff , 0xff , 0xff
} ;
static const u8 mac_v6_allmcast [ ETH_ALEN ] __long_aligned = {
0x33 , 0x33 , 0x00 , 0x00 , 0x00 , 0x01
} ;
2005-04-16 16:20:36 -06:00
static const int alb_delta_in_ticks = HZ / ALB_TIMER_TICKS_PER_SEC ;
# pragma pack(1)
struct learning_pkt {
u8 mac_dst [ ETH_ALEN ] ;
u8 mac_src [ ETH_ALEN ] ;
2007-08-22 18:06:58 -06:00
__be16 type ;
2005-04-16 16:20:36 -06:00
u8 padding [ ETH_ZLEN - ETH_HLEN ] ;
} ;
struct arp_pkt {
2007-08-22 18:06:58 -06:00
__be16 hw_addr_space ;
__be16 prot_addr_space ;
2005-04-16 16:20:36 -06:00
u8 hw_addr_len ;
u8 prot_addr_len ;
2007-08-22 18:06:58 -06:00
__be16 op_code ;
2005-04-16 16:20:36 -06:00
u8 mac_src [ ETH_ALEN ] ; /* sender hardware address */
2007-08-22 18:06:58 -06:00
__be32 ip_src ; /* sender IP address */
2005-04-16 16:20:36 -06:00
u8 mac_dst [ ETH_ALEN ] ; /* target hardware address */
2007-08-22 18:06:58 -06:00
__be32 ip_dst ; /* target IP address */
2005-04-16 16:20:36 -06:00
} ;
# pragma pack()
2007-03-10 12:07:19 -07:00
static inline struct arp_pkt * arp_pkt ( const struct sk_buff * skb )
{
2007-04-10 21:50:43 -06:00
return ( struct arp_pkt * ) skb_network_header ( skb ) ;
2007-03-10 12:07:19 -07:00
}
2005-04-16 16:20:36 -06:00
/* Forward declaration */
static void alb_send_learning_packets ( struct slave * slave , u8 mac_addr [ ] ) ;
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
static void rlb_purge_src_ip ( struct bonding * bond , struct arp_pkt * arp ) ;
static void rlb_src_unlink ( struct bonding * bond , u32 index ) ;
static void rlb_src_link ( struct bonding * bond , u32 ip_src_hash ,
u32 ip_dst_hash ) ;
2005-04-16 16:20:36 -06:00
2007-04-20 23:47:35 -06:00
static inline u8 _simple_hash ( const u8 * hash_start , int hash_size )
2005-04-16 16:20:36 -06:00
{
int i ;
u8 hash = 0 ;
for ( i = 0 ; i < hash_size ; i + + ) {
hash ^ = hash_start [ i ] ;
}
return hash ;
}
/*********************** tlb specific functions ***************************/
2012-01-09 05:01:37 -07:00
static inline void _lock_tx_hashtbl_bh ( struct bonding * bond )
2005-04-16 16:20:36 -06:00
{
2007-10-17 18:37:50 -06:00
spin_lock_bh ( & ( BOND_ALB_INFO ( bond ) . tx_hashtbl_lock ) ) ;
2005-04-16 16:20:36 -06:00
}
2012-01-09 05:01:37 -07:00
static inline void _unlock_tx_hashtbl_bh ( struct bonding * bond )
2005-04-16 16:20:36 -06:00
{
2007-10-17 18:37:50 -06:00
spin_unlock_bh ( & ( BOND_ALB_INFO ( bond ) . tx_hashtbl_lock ) ) ;
2005-04-16 16:20:36 -06:00
}
2012-01-09 05:01:37 -07:00
static inline void _lock_tx_hashtbl ( struct bonding * bond )
{
spin_lock ( & ( BOND_ALB_INFO ( bond ) . tx_hashtbl_lock ) ) ;
}
static inline void _unlock_tx_hashtbl ( struct bonding * bond )
{
spin_unlock ( & ( BOND_ALB_INFO ( bond ) . tx_hashtbl_lock ) ) ;
}
2005-04-16 16:20:36 -06:00
/* Caller must hold tx_hashtbl lock */
static inline void tlb_init_table_entry ( struct tlb_client_info * entry , int save_load )
{
if ( save_load ) {
entry - > load_history = 1 + entry - > tx_bytes /
BOND_TLB_REBALANCE_INTERVAL ;
entry - > tx_bytes = 0 ;
}
entry - > tx_slave = NULL ;
entry - > next = TLB_NULL_INDEX ;
entry - > prev = TLB_NULL_INDEX ;
}
static inline void tlb_init_slave ( struct slave * slave )
{
SLAVE_TLB_INFO ( slave ) . load = 0 ;
SLAVE_TLB_INFO ( slave ) . head = TLB_NULL_INDEX ;
}
2012-01-09 05:01:37 -07:00
/* Caller must hold bond lock for read, BH disabled */
static void __tlb_clear_slave ( struct bonding * bond , struct slave * slave ,
int save_load )
2005-04-16 16:20:36 -06:00
{
struct tlb_client_info * tx_hash_table ;
u32 index ;
/* clear slave from tx_hashtbl */
tx_hash_table = BOND_ALB_INFO ( bond ) . tx_hashtbl ;
2008-10-30 18:41:16 -06:00
/* skip this if we've already freed the tx hash table */
if ( tx_hash_table ) {
index = SLAVE_TLB_INFO ( slave ) . head ;
while ( index ! = TLB_NULL_INDEX ) {
u32 next_index = tx_hash_table [ index ] . next ;
tlb_init_table_entry ( & tx_hash_table [ index ] , save_load ) ;
index = next_index ;
}
2005-04-16 16:20:36 -06:00
}
tlb_init_slave ( slave ) ;
2012-01-09 05:01:37 -07:00
}
2006-01-09 13:14:00 -07:00
2012-01-09 05:01:37 -07:00
/* Caller must hold bond lock for read */
static void tlb_clear_slave ( struct bonding * bond , struct slave * slave ,
int save_load )
{
_lock_tx_hashtbl_bh ( bond ) ;
__tlb_clear_slave ( bond , slave , save_load ) ;
_unlock_tx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
/* Must be called before starting the monitor timer */
static int tlb_initialize ( struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
int size = TLB_HASH_TABLE_SIZE * sizeof ( struct tlb_client_info ) ;
2005-11-09 11:35:30 -07:00
struct tlb_client_info * new_hashtbl ;
2005-04-16 16:20:36 -06:00
int i ;
2007-02-06 15:16:40 -07:00
new_hashtbl = kzalloc ( size , GFP_KERNEL ) ;
2012-01-29 05:56:23 -07:00
if ( ! new_hashtbl )
2005-04-16 16:20:36 -06:00
return - 1 ;
2012-01-29 05:56:23 -07:00
2012-01-09 05:01:37 -07:00
_lock_tx_hashtbl_bh ( bond ) ;
2005-11-09 11:35:30 -07:00
bond_info - > tx_hashtbl = new_hashtbl ;
2005-04-16 16:20:36 -06:00
for ( i = 0 ; i < TLB_HASH_TABLE_SIZE ; i + + ) {
2011-04-07 21:40:19 -06:00
tlb_init_table_entry ( & bond_info - > tx_hashtbl [ i ] , 0 ) ;
2005-04-16 16:20:36 -06:00
}
2012-01-09 05:01:37 -07:00
_unlock_tx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
return 0 ;
}
/* Must be called only after all slaves have been released */
static void tlb_deinitialize ( struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
2012-01-09 05:01:37 -07:00
_lock_tx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
kfree ( bond_info - > tx_hashtbl ) ;
bond_info - > tx_hashtbl = NULL ;
2012-01-09 05:01:37 -07:00
_unlock_tx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
2010-05-18 21:26:39 -06:00
static long long compute_gap ( struct slave * slave )
{
return ( s64 ) ( slave - > speed < < 20 ) - /* Convert to Megabit per sec */
( s64 ) ( SLAVE_TLB_INFO ( slave ) . load < < 3 ) ; /* Bytes to bits */
}
2005-04-16 16:20:36 -06:00
/* Caller must hold bond lock for read */
static struct slave * tlb_get_least_loaded_slave ( struct bonding * bond )
{
struct slave * slave , * least_loaded ;
2010-05-18 21:26:39 -06:00
long long max_gap ;
int i ;
2005-04-16 16:20:36 -06:00
2010-05-18 21:26:39 -06:00
least_loaded = NULL ;
max_gap = LLONG_MIN ;
2005-04-16 16:20:36 -06:00
/* Find the slave with the largest gap */
2010-05-18 21:26:39 -06:00
bond_for_each_slave ( bond , slave , i ) {
2005-04-16 16:20:36 -06:00
if ( SLAVE_IS_OK ( slave ) ) {
2010-05-18 21:26:39 -06:00
long long gap = compute_gap ( slave ) ;
2005-04-16 16:20:36 -06:00
if ( max_gap < gap ) {
least_loaded = slave ;
max_gap = gap ;
}
}
}
return least_loaded ;
}
2012-01-09 05:01:37 -07:00
static struct slave * __tlb_choose_channel ( struct bonding * bond , u32 hash_index ,
u32 skb_len )
2005-04-16 16:20:36 -06:00
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct tlb_client_info * hash_table ;
struct slave * assigned_slave ;
hash_table = bond_info - > tx_hashtbl ;
assigned_slave = hash_table [ hash_index ] . tx_slave ;
if ( ! assigned_slave ) {
assigned_slave = tlb_get_least_loaded_slave ( bond ) ;
if ( assigned_slave ) {
struct tlb_slave_info * slave_info =
& ( SLAVE_TLB_INFO ( assigned_slave ) ) ;
u32 next_index = slave_info - > head ;
hash_table [ hash_index ] . tx_slave = assigned_slave ;
hash_table [ hash_index ] . next = next_index ;
hash_table [ hash_index ] . prev = TLB_NULL_INDEX ;
if ( next_index ! = TLB_NULL_INDEX ) {
hash_table [ next_index ] . prev = hash_index ;
}
slave_info - > head = hash_index ;
slave_info - > load + =
hash_table [ hash_index ] . load_history ;
}
}
if ( assigned_slave ) {
hash_table [ hash_index ] . tx_bytes + = skb_len ;
}
return assigned_slave ;
}
2012-01-09 05:01:37 -07:00
/* Caller must hold bond lock for read */
static struct slave * tlb_choose_channel ( struct bonding * bond , u32 hash_index ,
u32 skb_len )
{
struct slave * tx_slave ;
/*
* We don ' t need to disable softirq here , becase
* tlb_choose_channel ( ) is only called by bond_alb_xmit ( )
* which already has softirq disabled .
*/
_lock_tx_hashtbl ( bond ) ;
tx_slave = __tlb_choose_channel ( bond , hash_index , skb_len ) ;
_unlock_tx_hashtbl ( bond ) ;
return tx_slave ;
}
2005-04-16 16:20:36 -06:00
/*********************** rlb specific functions ***************************/
2012-01-09 05:01:37 -07:00
static inline void _lock_rx_hashtbl_bh ( struct bonding * bond )
2005-04-16 16:20:36 -06:00
{
2007-10-17 18:37:50 -06:00
spin_lock_bh ( & ( BOND_ALB_INFO ( bond ) . rx_hashtbl_lock ) ) ;
2005-04-16 16:20:36 -06:00
}
2012-01-09 05:01:37 -07:00
static inline void _unlock_rx_hashtbl_bh ( struct bonding * bond )
2005-04-16 16:20:36 -06:00
{
2007-10-17 18:37:50 -06:00
spin_unlock_bh ( & ( BOND_ALB_INFO ( bond ) . rx_hashtbl_lock ) ) ;
2005-04-16 16:20:36 -06:00
}
2012-01-09 05:01:37 -07:00
static inline void _lock_rx_hashtbl ( struct bonding * bond )
{
spin_lock ( & ( BOND_ALB_INFO ( bond ) . rx_hashtbl_lock ) ) ;
}
static inline void _unlock_rx_hashtbl ( struct bonding * bond )
{
spin_unlock ( & ( BOND_ALB_INFO ( bond ) . rx_hashtbl_lock ) ) ;
}
2005-04-16 16:20:36 -06:00
/* when an ARP REPLY is received from a client update its info
* in the rx_hashtbl
*/
static void rlb_update_entry_from_arp ( struct bonding * bond , struct arp_pkt * arp )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct rlb_client_info * client_info ;
u32 hash_index ;
2012-01-09 05:01:37 -07:00
_lock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
hash_index = _simple_hash ( ( u8 * ) & ( arp - > ip_src ) , sizeof ( arp - > ip_src ) ) ;
client_info = & ( bond_info - > rx_hashtbl [ hash_index ] ) ;
if ( ( client_info - > assigned ) & &
( client_info - > ip_src = = arp - > ip_dst ) & &
bonding: check if clients MAC addr has changed
When two systems using bonding devices in adaptive load
balancing (ALB) communicates with each other, an endless
ping-pong of ARP replies starts between these two systems.
What happens? In the ALB mode, bonding driver keeps track
of each client connected in a hash table, so it can do the
receive load balancing (RLB). This hash table is updated
when an ARP reply is received, then it scans for the client
entry, updates its MAC address and flag it to be announced
later. Therefore, two seconds later, the alb monitor runs
and send for each updated client entry two ARP replies
updating this specific client. The same process happens on
the receiving system, causing the endless ping-pong of arp
replies.
See more information including the relevant functions below:
System 1 System 2
bond0 bond0
ping <system2>
ARP request --------->
<--------- ARP reply
+->rlb_arp_recv <---------------------+ <--- loop begins
| rlb_update_entry_from_arp |
| client_info->ntt = 1; |
| bond_info->rx_ntt = 1; |
| |
| <communication succeed> |
| |
| bond_alb_monitor |
| rlb_update_rx_clients |
| rlb_update_client |
| arp_create(ARPOP_REPLY) |
| send ARP reply --------------> V
| send ARP reply -------------->
| rlb_arp_recv
| rlb_update_entry_from_arp
| client_info->ntt = 1;
| bond_info->rx_ntt = 1;
| < snipped, same as in system 1>
+------- <-------------- send ARP reply
<-------------- send ARP reply
Besides the unneeded networking traffic, this loop breaks
a cluster because a backup system can't take over the IP
address. There is always one system sending an ARP reply
poisoning the network.
This patch fixes the problem adding a check for the MAC
address before updating it. Thus, if the MAC address didn't
change, there is no need to update neither to announce it later.
Signed-off-by: Flavio Leitner <fleitner@redhat.com>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2010-06-29 02:24:39 -06:00
( client_info - > ip_dst = = arp - > ip_src ) & &
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
( ! ether_addr_equal_64bits ( client_info - > mac_dst , arp - > mac_src ) ) ) {
2005-04-16 16:20:36 -06:00
/* update the clients MAC address */
memcpy ( client_info - > mac_dst , arp - > mac_src , ETH_ALEN ) ;
client_info - > ntt = 1 ;
bond_info - > rx_ntt = 1 ;
}
2012-01-09 05:01:37 -07:00
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
2012-06-11 13:23:07 -06:00
static int rlb_arp_recv ( const struct sk_buff * skb , struct bonding * bond ,
struct slave * slave )
2005-04-16 16:20:36 -06:00
{
2012-06-11 13:23:07 -06:00
struct arp_pkt * arp , _arp ;
2005-04-16 16:20:36 -06:00
2011-04-18 21:48:16 -06:00
if ( skb - > protocol ! = cpu_to_be16 ( ETH_P_ARP ) )
2012-05-13 13:45:13 -06:00
goto out ;
2005-04-16 16:20:36 -06:00
2012-06-11 13:23:07 -06:00
arp = skb_header_pointer ( skb , 0 , sizeof ( _arp ) , & _arp ) ;
if ( ! arp )
2012-05-13 13:45:13 -06:00
goto out ;
2005-04-16 16:20:36 -06:00
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
/* We received an ARP from arp->ip_src.
* We might have used this IP address previously ( on the bonding host
* itself or on a system that is bridged together with the bond ) .
* However , if arp - > mac_src is different than what is stored in
* rx_hashtbl , some other host is now using the IP and we must prevent
* sending out client updates with this IP address and the old MAC
* address .
* Clean up all hash table entries that have this address as ip_src but
* have a different mac_src .
*/
rlb_purge_src_ip ( bond , arp ) ;
2005-04-16 16:20:36 -06:00
if ( arp - > op_code = = htons ( ARPOP_REPLY ) ) {
/* update rx hash table for this ARP */
rlb_update_entry_from_arp ( bond , arp ) ;
2008-12-10 00:09:22 -07:00
pr_debug ( " Server received an ARP Reply from client \n " ) ;
2005-04-16 16:20:36 -06:00
}
2012-05-13 13:45:13 -06:00
out :
return RX_HANDLER_ANOTHER ;
2005-04-16 16:20:36 -06:00
}
/* Caller must hold bond lock for read */
static struct slave * rlb_next_rx_slave ( struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct slave * rx_slave , * slave , * start_at ;
int i = 0 ;
if ( bond_info - > next_rx_slave ) {
start_at = bond_info - > next_rx_slave ;
} else {
start_at = bond - > first_slave ;
}
rx_slave = NULL ;
bond_for_each_slave_from ( bond , slave , i , start_at ) {
if ( SLAVE_IS_OK ( slave ) ) {
if ( ! rx_slave ) {
rx_slave = slave ;
} else if ( slave - > speed > rx_slave - > speed ) {
rx_slave = slave ;
}
}
}
if ( rx_slave ) {
bond_info - > next_rx_slave = rx_slave - > next ;
}
return rx_slave ;
}
/* teach the switch the mac of a disabled slave
* on the primary for fault tolerance
*
* Caller must hold bond - > curr_slave_lock for write or bond lock for write
*/
static void rlb_teach_disabled_mac_on_primary ( struct bonding * bond , u8 addr [ ] )
{
if ( ! bond - > curr_active_slave ) {
return ;
}
if ( ! bond - > alb_info . primary_is_promisc ) {
2008-07-14 21:51:36 -06:00
if ( ! dev_set_promiscuity ( bond - > curr_active_slave - > dev , 1 ) )
bond - > alb_info . primary_is_promisc = 1 ;
else
bond - > alb_info . primary_is_promisc = 0 ;
2005-04-16 16:20:36 -06:00
}
bond - > alb_info . rlb_promisc_timeout_counter = 0 ;
alb_send_learning_packets ( bond - > curr_active_slave , addr ) ;
}
/* slave being removed should not be active at this point
*
* Caller must hold bond lock for read
*/
static void rlb_clear_slave ( struct bonding * bond , struct slave * slave )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct rlb_client_info * rx_hash_table ;
u32 index , next_index ;
/* clear slave from rx_hashtbl */
2012-01-09 05:01:37 -07:00
_lock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
rx_hash_table = bond_info - > rx_hashtbl ;
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
index = bond_info - > rx_hashtbl_used_head ;
2005-04-16 16:20:36 -06:00
for ( ; index ! = RLB_NULL_INDEX ; index = next_index ) {
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
next_index = rx_hash_table [ index ] . used_next ;
2005-04-16 16:20:36 -06:00
if ( rx_hash_table [ index ] . slave = = slave ) {
struct slave * assigned_slave = rlb_next_rx_slave ( bond ) ;
if ( assigned_slave ) {
rx_hash_table [ index ] . slave = assigned_slave ;
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ! ether_addr_equal_64bits ( rx_hash_table [ index ] . mac_dst ,
mac_bcast ) ) {
2005-04-16 16:20:36 -06:00
bond_info - > rx_hashtbl [ index ] . ntt = 1 ;
bond_info - > rx_ntt = 1 ;
/* A slave has been removed from the
* table because it is either disabled
* or being released . We must retry the
* update to avoid clients from not
* being updated & disconnecting when
* there is stress
*/
bond_info - > rlb_update_retry_counter =
RLB_UPDATE_RETRY ;
}
} else { /* there is no active slave */
rx_hash_table [ index ] . slave = NULL ;
}
}
}
2012-01-09 05:01:37 -07:00
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
2007-10-17 18:37:50 -06:00
write_lock_bh ( & bond - > curr_slave_lock ) ;
2005-04-16 16:20:36 -06:00
if ( slave ! = bond - > curr_active_slave ) {
rlb_teach_disabled_mac_on_primary ( bond , slave - > dev - > dev_addr ) ;
}
2007-10-17 18:37:50 -06:00
write_unlock_bh ( & bond - > curr_slave_lock ) ;
2005-04-16 16:20:36 -06:00
}
static void rlb_update_client ( struct rlb_client_info * client_info )
{
int i ;
if ( ! client_info - > slave ) {
return ;
}
for ( i = 0 ; i < RLB_ARP_BURST_SIZE ; i + + ) {
struct sk_buff * skb ;
skb = arp_create ( ARPOP_REPLY , ETH_P_ARP ,
client_info - > ip_dst ,
client_info - > slave - > dev ,
client_info - > ip_src ,
client_info - > mac_dst ,
client_info - > slave - > dev - > dev_addr ,
client_info - > mac_dst ) ;
if ( ! skb ) {
2009-12-13 21:06:07 -07:00
pr_err ( " %s: Error: failed to create an ARP packet \n " ,
2013-01-03 15:49:01 -07:00
client_info - > slave - > bond - > dev - > name ) ;
2005-04-16 16:20:36 -06:00
continue ;
}
skb - > dev = client_info - > slave - > dev ;
if ( client_info - > tag ) {
2013-04-18 20:04:30 -06:00
skb = vlan_put_tag ( skb , htons ( ETH_P_8021Q ) , client_info - > vlan_id ) ;
2005-04-16 16:20:36 -06:00
if ( ! skb ) {
2009-12-13 21:06:07 -07:00
pr_err ( " %s: Error: failed to insert VLAN tag \n " ,
2013-01-03 15:49:01 -07:00
client_info - > slave - > bond - > dev - > name ) ;
2005-04-16 16:20:36 -06:00
continue ;
}
}
arp_xmit ( skb ) ;
}
}
/* sends ARP REPLIES that update the clients that need updating */
static void rlb_update_rx_clients ( struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct rlb_client_info * client_info ;
u32 hash_index ;
2012-01-09 05:01:37 -07:00
_lock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
hash_index = bond_info - > rx_hashtbl_used_head ;
for ( ; hash_index ! = RLB_NULL_INDEX ;
hash_index = client_info - > used_next ) {
2005-04-16 16:20:36 -06:00
client_info = & ( bond_info - > rx_hashtbl [ hash_index ] ) ;
if ( client_info - > ntt ) {
rlb_update_client ( client_info ) ;
if ( bond_info - > rlb_update_retry_counter = = 0 ) {
client_info - > ntt = 0 ;
}
}
}
2009-10-16 07:20:49 -06:00
/* do not update the entries again until this counter is zero so that
2005-04-16 16:20:36 -06:00
* not to confuse the clients .
*/
bond_info - > rlb_update_delay_counter = RLB_UPDATE_DELAY ;
2012-01-09 05:01:37 -07:00
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
/* The slave was assigned a new mac address - update the clients */
static void rlb_req_update_slave_clients ( struct bonding * bond , struct slave * slave )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct rlb_client_info * client_info ;
int ntt = 0 ;
u32 hash_index ;
2012-01-09 05:01:37 -07:00
_lock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
hash_index = bond_info - > rx_hashtbl_used_head ;
for ( ; hash_index ! = RLB_NULL_INDEX ;
hash_index = client_info - > used_next ) {
2005-04-16 16:20:36 -06:00
client_info = & ( bond_info - > rx_hashtbl [ hash_index ] ) ;
if ( ( client_info - > slave = = slave ) & &
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
! ether_addr_equal_64bits ( client_info - > mac_dst , mac_bcast ) ) {
2005-04-16 16:20:36 -06:00
client_info - > ntt = 1 ;
ntt = 1 ;
}
}
// update the team's flag only after the whole iteration
if ( ntt ) {
bond_info - > rx_ntt = 1 ;
//fasten the change
bond_info - > rlb_update_retry_counter = RLB_UPDATE_RETRY ;
}
2012-01-09 05:01:37 -07:00
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
/* mark all clients using src_ip to be updated */
2007-08-22 18:06:58 -06:00
static void rlb_req_update_subnet_clients ( struct bonding * bond , __be32 src_ip )
2005-04-16 16:20:36 -06:00
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct rlb_client_info * client_info ;
u32 hash_index ;
_lock_rx_hashtbl ( bond ) ;
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
hash_index = bond_info - > rx_hashtbl_used_head ;
for ( ; hash_index ! = RLB_NULL_INDEX ;
hash_index = client_info - > used_next ) {
2005-04-16 16:20:36 -06:00
client_info = & ( bond_info - > rx_hashtbl [ hash_index ] ) ;
if ( ! client_info - > slave ) {
2009-12-13 21:06:07 -07:00
pr_err ( " %s: Error: found a client with no channel in the client's hash table \n " ,
2005-11-09 11:34:57 -07:00
bond - > dev - > name ) ;
2005-04-16 16:20:36 -06:00
continue ;
}
/*update all clients using this src_ip, that are not assigned
* to the team ' s address ( curr_active_slave ) and have a known
* unicast mac address .
*/
if ( ( client_info - > ip_src = = src_ip ) & &
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
! ether_addr_equal_64bits ( client_info - > slave - > dev - > dev_addr ,
bond - > dev - > dev_addr ) & &
! ether_addr_equal_64bits ( client_info - > mac_dst , mac_bcast ) ) {
2005-04-16 16:20:36 -06:00
client_info - > ntt = 1 ;
bond_info - > rx_ntt = 1 ;
}
}
_unlock_rx_hashtbl ( bond ) ;
}
/* Caller must hold both bond and ptr locks for read */
static struct slave * rlb_choose_channel ( struct sk_buff * skb , struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
2007-03-10 12:07:19 -07:00
struct arp_pkt * arp = arp_pkt ( skb ) ;
2005-04-16 16:20:36 -06:00
struct slave * assigned_slave ;
struct rlb_client_info * client_info ;
u32 hash_index = 0 ;
_lock_rx_hashtbl ( bond ) ;
2011-02-27 16:34:28 -07:00
hash_index = _simple_hash ( ( u8 * ) & arp - > ip_dst , sizeof ( arp - > ip_dst ) ) ;
2005-04-16 16:20:36 -06:00
client_info = & ( bond_info - > rx_hashtbl [ hash_index ] ) ;
if ( client_info - > assigned ) {
if ( ( client_info - > ip_src = = arp - > ip_src ) & &
( client_info - > ip_dst = = arp - > ip_dst ) ) {
/* the entry is already assigned to this client */
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ! ether_addr_equal_64bits ( arp - > mac_dst , mac_bcast ) ) {
2005-04-16 16:20:36 -06:00
/* update mac address from arp */
memcpy ( client_info - > mac_dst , arp - > mac_dst , ETH_ALEN ) ;
}
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
memcpy ( client_info - > mac_src , arp - > mac_src , ETH_ALEN ) ;
2005-04-16 16:20:36 -06:00
assigned_slave = client_info - > slave ;
if ( assigned_slave ) {
_unlock_rx_hashtbl ( bond ) ;
return assigned_slave ;
}
} else {
/* the entry is already assigned to some other client,
* move the old client to primary ( curr_active_slave ) so
* that the new client can be assigned to this entry .
*/
if ( bond - > curr_active_slave & &
client_info - > slave ! = bond - > curr_active_slave ) {
client_info - > slave = bond - > curr_active_slave ;
rlb_update_client ( client_info ) ;
}
}
}
/* assign a new slave */
assigned_slave = rlb_next_rx_slave ( bond ) ;
if ( assigned_slave ) {
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
if ( ! ( client_info - > assigned & &
client_info - > ip_src = = arp - > ip_src ) ) {
/* ip_src is going to be updated,
* fix the src hash list
*/
u32 hash_src = _simple_hash ( ( u8 * ) & arp - > ip_src ,
sizeof ( arp - > ip_src ) ) ;
rlb_src_unlink ( bond , hash_index ) ;
rlb_src_link ( bond , hash_src , hash_index ) ;
}
2005-04-16 16:20:36 -06:00
client_info - > ip_src = arp - > ip_src ;
client_info - > ip_dst = arp - > ip_dst ;
/* arp->mac_dst is broadcast for arp reqeusts.
* will be updated with clients actual unicast mac address
* upon receiving an arp reply .
*/
memcpy ( client_info - > mac_dst , arp - > mac_dst , ETH_ALEN ) ;
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
memcpy ( client_info - > mac_src , arp - > mac_src , ETH_ALEN ) ;
2005-04-16 16:20:36 -06:00
client_info - > slave = assigned_slave ;
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ! ether_addr_equal_64bits ( client_info - > mac_dst , mac_bcast ) ) {
2005-04-16 16:20:36 -06:00
client_info - > ntt = 1 ;
bond - > alb_info . rx_ntt = 1 ;
} else {
client_info - > ntt = 0 ;
}
2011-07-19 22:54:46 -06:00
if ( bond_vlan_used ( bond ) ) {
2008-03-21 23:29:34 -06:00
if ( ! vlan_get_tag ( skb , & client_info - > vlan_id ) )
2005-04-16 16:20:36 -06:00
client_info - > tag = 1 ;
}
if ( ! client_info - > assigned ) {
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
u32 prev_tbl_head = bond_info - > rx_hashtbl_used_head ;
bond_info - > rx_hashtbl_used_head = hash_index ;
client_info - > used_next = prev_tbl_head ;
2005-04-16 16:20:36 -06:00
if ( prev_tbl_head ! = RLB_NULL_INDEX ) {
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
bond_info - > rx_hashtbl [ prev_tbl_head ] . used_prev =
2005-04-16 16:20:36 -06:00
hash_index ;
}
client_info - > assigned = 1 ;
}
}
_unlock_rx_hashtbl ( bond ) ;
return assigned_slave ;
}
/* chooses (and returns) transmit channel for arp reply
* does not choose channel for other arp types since they are
* sent on the curr_active_slave
*/
static struct slave * rlb_arp_xmit ( struct sk_buff * skb , struct bonding * bond )
{
2007-03-10 12:07:19 -07:00
struct arp_pkt * arp = arp_pkt ( skb ) ;
2005-04-16 16:20:36 -06:00
struct slave * tx_slave = NULL ;
2012-11-27 16:57:04 -07:00
/* Don't modify or load balance ARPs that do not originate locally
* ( e . g . , arrive via a bridge ) .
*/
if ( ! bond_slave_has_mac ( bond , arp - > mac_src ) )
return NULL ;
2008-09-02 08:08:08 -06:00
if ( arp - > op_code = = htons ( ARPOP_REPLY ) ) {
2005-04-16 16:20:36 -06:00
/* the arp must be sent on the selected
* rx channel
*/
tx_slave = rlb_choose_channel ( skb , bond ) ;
if ( tx_slave ) {
memcpy ( arp - > mac_src , tx_slave - > dev - > dev_addr , ETH_ALEN ) ;
}
2008-12-10 00:09:22 -07:00
pr_debug ( " Server sent ARP Reply packet \n " ) ;
2008-09-02 08:08:08 -06:00
} else if ( arp - > op_code = = htons ( ARPOP_REQUEST ) ) {
2005-04-16 16:20:36 -06:00
/* Create an entry in the rx_hashtbl for this client as a
* place holder .
* When the arp reply is received the entry will be updated
* with the correct unicast address of the client .
*/
rlb_choose_channel ( skb , bond ) ;
2011-04-10 18:16:32 -06:00
/* The ARP reply packets must be delayed so that
2005-04-16 16:20:36 -06:00
* they can cancel out the influence of the ARP request .
*/
bond - > alb_info . rlb_update_delay_counter = RLB_UPDATE_DELAY ;
/* arp requests are broadcast and are sent on the primary
* the arp request will collapse all clients on the subnet to
* the primary slave . We must register these clients to be
* updated with their assigned mac .
*/
rlb_req_update_subnet_clients ( bond , arp - > ip_src ) ;
2008-12-10 00:09:22 -07:00
pr_debug ( " Server sent ARP Request packet \n " ) ;
2005-04-16 16:20:36 -06:00
}
return tx_slave ;
}
/* Caller must hold bond lock for read */
static void rlb_rebalance ( struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct slave * assigned_slave ;
struct rlb_client_info * client_info ;
int ntt ;
u32 hash_index ;
2012-01-09 05:01:37 -07:00
_lock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
ntt = 0 ;
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
hash_index = bond_info - > rx_hashtbl_used_head ;
for ( ; hash_index ! = RLB_NULL_INDEX ;
hash_index = client_info - > used_next ) {
2005-04-16 16:20:36 -06:00
client_info = & ( bond_info - > rx_hashtbl [ hash_index ] ) ;
assigned_slave = rlb_next_rx_slave ( bond ) ;
if ( assigned_slave & & ( client_info - > slave ! = assigned_slave ) ) {
client_info - > slave = assigned_slave ;
client_info - > ntt = 1 ;
ntt = 1 ;
}
}
/* update the team's flag only after the whole iteration */
if ( ntt ) {
bond_info - > rx_ntt = 1 ;
}
2012-01-09 05:01:37 -07:00
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
/* Caller must hold rx_hashtbl lock */
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
static void rlb_init_table_entry_dst ( struct rlb_client_info * entry )
{
entry - > used_next = RLB_NULL_INDEX ;
entry - > used_prev = RLB_NULL_INDEX ;
entry - > assigned = 0 ;
entry - > slave = NULL ;
entry - > tag = 0 ;
}
static void rlb_init_table_entry_src ( struct rlb_client_info * entry )
{
entry - > src_first = RLB_NULL_INDEX ;
entry - > src_prev = RLB_NULL_INDEX ;
entry - > src_next = RLB_NULL_INDEX ;
}
2005-04-16 16:20:36 -06:00
static void rlb_init_table_entry ( struct rlb_client_info * entry )
{
memset ( entry , 0 , sizeof ( struct rlb_client_info ) ) ;
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
rlb_init_table_entry_dst ( entry ) ;
rlb_init_table_entry_src ( entry ) ;
}
static void rlb_delete_table_entry_dst ( struct bonding * bond , u32 index )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
u32 next_index = bond_info - > rx_hashtbl [ index ] . used_next ;
u32 prev_index = bond_info - > rx_hashtbl [ index ] . used_prev ;
if ( index = = bond_info - > rx_hashtbl_used_head )
bond_info - > rx_hashtbl_used_head = next_index ;
if ( prev_index ! = RLB_NULL_INDEX )
bond_info - > rx_hashtbl [ prev_index ] . used_next = next_index ;
if ( next_index ! = RLB_NULL_INDEX )
bond_info - > rx_hashtbl [ next_index ] . used_prev = prev_index ;
}
/* unlink a rlb hash table entry from the src list */
static void rlb_src_unlink ( struct bonding * bond , u32 index )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
u32 next_index = bond_info - > rx_hashtbl [ index ] . src_next ;
u32 prev_index = bond_info - > rx_hashtbl [ index ] . src_prev ;
bond_info - > rx_hashtbl [ index ] . src_next = RLB_NULL_INDEX ;
bond_info - > rx_hashtbl [ index ] . src_prev = RLB_NULL_INDEX ;
if ( next_index ! = RLB_NULL_INDEX )
bond_info - > rx_hashtbl [ next_index ] . src_prev = prev_index ;
if ( prev_index = = RLB_NULL_INDEX )
return ;
/* is prev_index pointing to the head of this list? */
if ( bond_info - > rx_hashtbl [ prev_index ] . src_first = = index )
bond_info - > rx_hashtbl [ prev_index ] . src_first = next_index ;
else
bond_info - > rx_hashtbl [ prev_index ] . src_next = next_index ;
}
static void rlb_delete_table_entry ( struct bonding * bond , u32 index )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct rlb_client_info * entry = & ( bond_info - > rx_hashtbl [ index ] ) ;
rlb_delete_table_entry_dst ( bond , index ) ;
rlb_init_table_entry_dst ( entry ) ;
rlb_src_unlink ( bond , index ) ;
}
/* add the rx_hashtbl[ip_dst_hash] entry to the list
* of entries with identical ip_src_hash
*/
static void rlb_src_link ( struct bonding * bond , u32 ip_src_hash , u32 ip_dst_hash )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
u32 next ;
bond_info - > rx_hashtbl [ ip_dst_hash ] . src_prev = ip_src_hash ;
next = bond_info - > rx_hashtbl [ ip_src_hash ] . src_first ;
bond_info - > rx_hashtbl [ ip_dst_hash ] . src_next = next ;
if ( next ! = RLB_NULL_INDEX )
bond_info - > rx_hashtbl [ next ] . src_prev = ip_dst_hash ;
bond_info - > rx_hashtbl [ ip_src_hash ] . src_first = ip_dst_hash ;
}
/* deletes all rx_hashtbl entries with arp->ip_src if their mac_src does
* not match arp - > mac_src */
static void rlb_purge_src_ip ( struct bonding * bond , struct arp_pkt * arp )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
u32 ip_src_hash = _simple_hash ( ( u8 * ) & ( arp - > ip_src ) , sizeof ( arp - > ip_src ) ) ;
u32 index ;
_lock_rx_hashtbl_bh ( bond ) ;
index = bond_info - > rx_hashtbl [ ip_src_hash ] . src_first ;
while ( index ! = RLB_NULL_INDEX ) {
struct rlb_client_info * entry = & ( bond_info - > rx_hashtbl [ index ] ) ;
u32 next_index = entry - > src_next ;
if ( entry - > ip_src = = arp - > ip_src & &
! ether_addr_equal_64bits ( arp - > mac_src , entry - > mac_src ) )
rlb_delete_table_entry ( bond , index ) ;
index = next_index ;
}
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
static int rlb_initialize ( struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
2005-11-09 11:35:30 -07:00
struct rlb_client_info * new_hashtbl ;
2005-04-16 16:20:36 -06:00
int size = RLB_HASH_TABLE_SIZE * sizeof ( struct rlb_client_info ) ;
int i ;
2005-11-09 11:35:30 -07:00
new_hashtbl = kmalloc ( size , GFP_KERNEL ) ;
2012-01-29 05:56:23 -07:00
if ( ! new_hashtbl )
2005-04-16 16:20:36 -06:00
return - 1 ;
2012-01-29 05:56:23 -07:00
2012-01-09 05:01:37 -07:00
_lock_rx_hashtbl_bh ( bond ) ;
2005-11-09 11:35:30 -07:00
bond_info - > rx_hashtbl = new_hashtbl ;
2005-04-16 16:20:36 -06:00
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
bond_info - > rx_hashtbl_used_head = RLB_NULL_INDEX ;
2005-04-16 16:20:36 -06:00
for ( i = 0 ; i < RLB_HASH_TABLE_SIZE ; i + + ) {
rlb_init_table_entry ( bond_info - > rx_hashtbl + i ) ;
}
2012-01-09 05:01:37 -07:00
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
/* register to receive ARPs */
2011-04-18 21:48:16 -06:00
bond - > recv_probe = rlb_arp_recv ;
2005-04-16 16:20:36 -06:00
return 0 ;
}
static void rlb_deinitialize ( struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
2012-01-09 05:01:37 -07:00
_lock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
kfree ( bond_info - > rx_hashtbl ) ;
bond_info - > rx_hashtbl = NULL ;
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
bond_info - > rx_hashtbl_used_head = RLB_NULL_INDEX ;
2005-04-16 16:20:36 -06:00
2012-01-09 05:01:37 -07:00
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
static void rlb_clear_vlan ( struct bonding * bond , unsigned short vlan_id )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
u32 curr_index ;
2012-01-09 05:01:37 -07:00
_lock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
curr_index = bond_info - > rx_hashtbl_used_head ;
2005-04-16 16:20:36 -06:00
while ( curr_index ! = RLB_NULL_INDEX ) {
struct rlb_client_info * curr = & ( bond_info - > rx_hashtbl [ curr_index ] ) ;
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
u32 next_index = bond_info - > rx_hashtbl [ curr_index ] . used_next ;
2005-04-16 16:20:36 -06:00
bonding: delete migrated IP addresses from the rlb hash table
Bonding in balance-alb mode records information from ARP packets
passing through the bond in a hash table (rx_hashtbl).
At certain situations (e.g. link change of a slave),
rlb_update_rx_clients() will send out ARP packets to update ARP
caches of other hosts on the network to achieve RX load
balancing.
The problem is that once an IP address is recorded in the hash
table, it stays there indefinitely. If this IP address is
migrated to a different host in the network, bonding still sends
out ARP packets that poison other systems' ARP caches with
invalid information.
This patch solves this by looking at all incoming ARP packets,
and checking if the source IP address is one of the source
addresses stored in the rx_hashtbl. If it is, but the MAC
addresses differ, the corresponding hash table entries are
removed. Thus, when an IP address is migrated, the first ARP
broadcast by its new owner will purge the offending entries of
rx_hashtbl.
The hash table is hashed by ip_dst. To be able to do the above
check efficiently (not walking the whole hash table), we need a
reverse mapping (by ip_src).
I added three new members in struct rlb_client_info:
rx_hashtbl[x].src_first will point to the start of a list of
entries for which hash(ip_src) == x.
The list is linked with src_next and src_prev.
When an incoming ARP packet arrives at rlb_arp_recv()
rlb_purge_src_ip() can quickly walk only the entries on the
corresponding lists, i.e. the entries that are likely to contain
the offending IP address.
To avoid confusion, I renamed these existing fields of struct
rlb_client_info:
next -> used_next
prev -> used_prev
rx_hashtbl_head -> rx_hashtbl_used_head
(The current linked list is _not_ a list of hash table
entries with colliding ip_dst. It's a list of entries that are
being used; its purpose is to avoid walking the whole hash table
when looking for used entries.)
Signed-off-by: Jiri Bohac <jbohac@suse.cz>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-11-27 21:42:14 -07:00
if ( curr - > tag & & ( curr - > vlan_id = = vlan_id ) )
rlb_delete_table_entry ( bond , curr_index ) ;
2005-04-16 16:20:36 -06:00
curr_index = next_index ;
}
2012-01-09 05:01:37 -07:00
_unlock_rx_hashtbl_bh ( bond ) ;
2005-04-16 16:20:36 -06:00
}
/*********************** tlb/rlb shared functions *********************/
static void alb_send_learning_packets ( struct slave * slave , u8 mac_addr [ ] )
{
struct bonding * bond = bond_get_bond_by_slave ( slave ) ;
struct learning_pkt pkt ;
int size = sizeof ( struct learning_pkt ) ;
int i ;
memset ( & pkt , 0 , size ) ;
memcpy ( pkt . mac_dst , mac_addr , ETH_ALEN ) ;
memcpy ( pkt . mac_src , mac_addr , ETH_ALEN ) ;
2009-02-01 01:45:17 -07:00
pkt . type = cpu_to_be16 ( ETH_P_LOOP ) ;
2005-04-16 16:20:36 -06:00
for ( i = 0 ; i < MAX_LP_BURST ; i + + ) {
struct sk_buff * skb ;
char * data ;
skb = dev_alloc_skb ( size ) ;
if ( ! skb ) {
return ;
}
data = skb_put ( skb , size ) ;
memcpy ( data , & pkt , size ) ;
2007-03-19 16:30:44 -06:00
skb_reset_mac_header ( skb ) ;
2007-04-10 22:21:55 -06:00
skb - > network_header = skb - > mac_header + ETH_HLEN ;
2005-04-16 16:20:36 -06:00
skb - > protocol = pkt . type ;
skb - > priority = TC_PRIO_CONTROL ;
skb - > dev = slave - > dev ;
2011-07-19 22:54:46 -06:00
if ( bond_vlan_used ( bond ) ) {
2005-04-16 16:20:36 -06:00
struct vlan_entry * vlan ;
vlan = bond_next_vlan ( bond ,
bond - > alb_info . current_alb_vlan ) ;
bond - > alb_info . current_alb_vlan = vlan ;
if ( ! vlan ) {
kfree_skb ( skb ) ;
continue ;
}
2013-04-18 20:04:30 -06:00
skb = vlan_put_tag ( skb , htons ( ETH_P_8021Q ) , vlan - > vlan_id ) ;
2005-04-16 16:20:36 -06:00
if ( ! skb ) {
2009-12-13 21:06:07 -07:00
pr_err ( " %s: Error: failed to insert VLAN tag \n " ,
2005-11-09 11:34:57 -07:00
bond - > dev - > name ) ;
2005-04-16 16:20:36 -06:00
continue ;
}
}
dev_queue_xmit ( skb ) ;
}
}
2012-01-18 05:24:54 -07:00
static int alb_set_slave_mac_addr ( struct slave * slave , u8 addr [ ] )
2005-04-16 16:20:36 -06:00
{
struct net_device * dev = slave - > dev ;
struct sockaddr s_addr ;
2012-01-18 05:24:54 -07:00
if ( slave - > bond - > params . mode = = BOND_MODE_TLB ) {
2005-04-16 16:20:36 -06:00
memcpy ( dev - > dev_addr , addr , dev - > addr_len ) ;
return 0 ;
}
/* for rlb each slave must have a unique hw mac addresses so that */
/* each slave will receive packets destined to a different mac */
memcpy ( s_addr . sa_data , addr , dev - > addr_len ) ;
s_addr . sa_family = dev - > type ;
if ( dev_set_mac_address ( dev , & s_addr ) ) {
2009-12-13 21:06:07 -07:00
pr_err ( " %s: Error: dev_set_mac_address of dev %s failed! \n "
" ALB mode requires that the base driver support setting the hw address also when the network device's interface is open \n " ,
2013-01-03 15:49:01 -07:00
slave - > bond - > dev - > name , dev - > name ) ;
2005-04-16 16:20:36 -06:00
return - EOPNOTSUPP ;
}
return 0 ;
}
2007-10-17 18:37:49 -06:00
/*
* Swap MAC addresses between two slaves .
*
* Called with RTNL held , and no other locks .
*
*/
2013-05-27 17:14:51 -06:00
static void alb_swap_mac_addr ( struct slave * slave1 , struct slave * slave2 )
2005-04-16 16:20:36 -06:00
{
u8 tmp_mac_addr [ ETH_ALEN ] ;
memcpy ( tmp_mac_addr , slave1 - > dev - > dev_addr , ETH_ALEN ) ;
2012-01-18 05:24:54 -07:00
alb_set_slave_mac_addr ( slave1 , slave2 - > dev - > dev_addr ) ;
alb_set_slave_mac_addr ( slave2 , tmp_mac_addr ) ;
2005-04-16 16:20:36 -06:00
2007-10-17 18:37:49 -06:00
}
/*
* Send learning packets after MAC address swap .
*
2008-01-17 17:24:59 -07:00
* Called with RTNL and no other locks
2007-10-17 18:37:49 -06:00
*/
static void alb_fasten_mac_swap ( struct bonding * bond , struct slave * slave1 ,
struct slave * slave2 )
{
int slaves_state_differ = ( SLAVE_IS_OK ( slave1 ) ! = SLAVE_IS_OK ( slave2 ) ) ;
struct slave * disabled_slave = NULL ;
2008-01-17 17:24:59 -07:00
ASSERT_RTNL ( ) ;
2005-04-16 16:20:36 -06:00
/* fasten the change in the switch */
if ( SLAVE_IS_OK ( slave1 ) ) {
alb_send_learning_packets ( slave1 , slave1 - > dev - > dev_addr ) ;
if ( bond - > alb_info . rlb_enabled ) {
/* inform the clients that the mac address
* has changed
*/
rlb_req_update_slave_clients ( bond , slave1 ) ;
}
} else {
disabled_slave = slave1 ;
}
if ( SLAVE_IS_OK ( slave2 ) ) {
alb_send_learning_packets ( slave2 , slave2 - > dev - > dev_addr ) ;
if ( bond - > alb_info . rlb_enabled ) {
/* inform the clients that the mac address
* has changed
*/
rlb_req_update_slave_clients ( bond , slave2 ) ;
}
} else {
disabled_slave = slave2 ;
}
if ( bond - > alb_info . rlb_enabled & & slaves_state_differ ) {
/* A disabled slave was assigned an active mac addr */
rlb_teach_disabled_mac_on_primary ( bond ,
disabled_slave - > dev - > dev_addr ) ;
}
}
/**
* alb_change_hw_addr_on_detach
* @ bond : bonding we ' re working on
* @ slave : the slave that was just detached
*
* We assume that @ slave was already detached from the slave list .
*
* If @ slave ' s permanent hw address is different both from its current
* address and from @ bond ' s address , then somewhere in the bond there ' s
* a slave that has @ slave ' s permanet address as its current address .
* We ' ll make sure that that slave no longer uses @ slave ' s permanent address .
*
2008-01-17 17:24:59 -07:00
* Caller must hold RTNL and no other locks
2005-04-16 16:20:36 -06:00
*/
static void alb_change_hw_addr_on_detach ( struct bonding * bond , struct slave * slave )
{
int perm_curr_diff ;
int perm_bond_diff ;
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
perm_curr_diff = ! ether_addr_equal_64bits ( slave - > perm_hwaddr ,
slave - > dev - > dev_addr ) ;
perm_bond_diff = ! ether_addr_equal_64bits ( slave - > perm_hwaddr ,
bond - > dev - > dev_addr ) ;
2005-04-16 16:20:36 -06:00
if ( perm_curr_diff & & perm_bond_diff ) {
struct slave * tmp_slave ;
int i , found = 0 ;
bond_for_each_slave ( bond , tmp_slave , i ) {
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ether_addr_equal_64bits ( slave - > perm_hwaddr ,
tmp_slave - > dev - > dev_addr ) ) {
2005-04-16 16:20:36 -06:00
found = 1 ;
break ;
}
}
if ( found ) {
2007-10-17 18:37:49 -06:00
/* locking: needs RTNL and nothing else */
2013-05-27 17:14:51 -06:00
alb_swap_mac_addr ( slave , tmp_slave ) ;
2007-10-17 18:37:49 -06:00
alb_fasten_mac_swap ( bond , slave , tmp_slave ) ;
2005-04-16 16:20:36 -06:00
}
}
}
/**
* alb_handle_addr_collision_on_attach
* @ bond : bonding we ' re working on
* @ slave : the slave that was just attached
*
* checks uniqueness of slave ' s mac address and handles the case the
* new slave uses the bonds mac address .
*
* If the permanent hw address of @ slave is @ bond ' s hw address , we need to
* find a different hw address to give @ slave , that isn ' t in use by any other
2011-04-10 18:16:32 -06:00
* slave in the bond . This address must be , of course , one of the permanent
2005-04-16 16:20:36 -06:00
* addresses of the other slaves .
*
* We go over the slave list , and for each slave there we compare its
* permanent hw address with the current address of all the other slaves .
* If no match was found , then we ' ve found a slave with a permanent address
* that isn ' t used by any other slave in the bond , so we can assign it to
* @ slave .
*
* assumption : this function is called before @ slave is attached to the
2013-06-17 11:30:35 -06:00
* bond slave list .
2005-04-16 16:20:36 -06:00
*/
static int alb_handle_addr_collision_on_attach ( struct bonding * bond , struct slave * slave )
{
2013-06-17 11:30:35 -06:00
struct slave * tmp_slave1 , * free_mac_slave = NULL ;
2005-04-16 16:20:36 -06:00
struct slave * has_bond_addr = bond - > curr_active_slave ;
2013-06-17 11:30:35 -06:00
int i ;
2005-04-16 16:20:36 -06:00
if ( bond - > slave_cnt = = 0 ) {
/* this is the first slave */
return 0 ;
}
/* if slave's mac address differs from bond's mac address
* check uniqueness of slave ' s mac address against the other
* slaves in the bond .
*/
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ! ether_addr_equal_64bits ( slave - > perm_hwaddr , bond - > dev - > dev_addr ) ) {
2013-06-17 11:30:35 -06:00
if ( ! bond_slave_has_mac ( bond , slave - > dev - > dev_addr ) )
2005-07-28 13:00:15 -06:00
return 0 ;
2005-04-16 16:20:36 -06:00
2005-07-28 13:00:15 -06:00
/* Try setting slave mac to bond address and fall-through
to code handling that situation below . . . */
2012-01-18 05:24:54 -07:00
alb_set_slave_mac_addr ( slave , bond - > dev - > dev_addr ) ;
2005-04-16 16:20:36 -06:00
}
/* The slave's address is equal to the address of the bond.
* Search for a spare address in the bond for this slave .
*/
bond_for_each_slave ( bond , tmp_slave1 , i ) {
2013-06-17 11:30:35 -06:00
if ( ! bond_slave_has_mac ( bond , tmp_slave1 - > perm_hwaddr ) ) {
2005-04-16 16:20:36 -06:00
/* no slave has tmp_slave1's perm addr
* as its curr addr
*/
free_mac_slave = tmp_slave1 ;
break ;
}
if ( ! has_bond_addr ) {
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ether_addr_equal_64bits ( tmp_slave1 - > dev - > dev_addr ,
bond - > dev - > dev_addr ) ) {
2005-04-16 16:20:36 -06:00
has_bond_addr = tmp_slave1 ;
}
}
}
if ( free_mac_slave ) {
2012-01-18 05:24:54 -07:00
alb_set_slave_mac_addr ( slave , free_mac_slave - > perm_hwaddr ) ;
2005-04-16 16:20:36 -06:00
2009-12-13 21:06:07 -07:00
pr_warning ( " %s: Warning: the hw address of slave %s is in use by the bond; giving it the hw address of %s \n " ,
2009-08-12 22:11:52 -06:00
bond - > dev - > name , slave - > dev - > name ,
free_mac_slave - > dev - > name ) ;
2005-04-16 16:20:36 -06:00
} else if ( has_bond_addr ) {
2009-12-13 21:06:07 -07:00
pr_err ( " %s: Error: the hw address of slave %s is in use by the bond; couldn't find a slave with a free hw address to give it (this should not have happened) \n " ,
2005-11-09 11:34:57 -07:00
bond - > dev - > name , slave - > dev - > name ) ;
2005-04-16 16:20:36 -06:00
return - EFAULT ;
}
return 0 ;
}
/**
* alb_set_mac_address
* @ bond :
* @ addr :
*
* In TLB mode all slaves are configured to the bond ' s hw address , but set
* their dev_addr field to different addresses ( based on their permanent hw
* addresses ) .
*
* For each slave , this function sets the interface to the new address and then
* changes its dev_addr field to its previous value .
*
* Unwinding assumes bond ' s mac address has not yet changed .
*/
static int alb_set_mac_address ( struct bonding * bond , void * addr )
{
struct sockaddr sa ;
struct slave * slave , * stop_at ;
char tmp_addr [ ETH_ALEN ] ;
int res ;
int i ;
if ( bond - > alb_info . rlb_enabled ) {
return 0 ;
}
bond_for_each_slave ( bond , slave , i ) {
/* save net_device's current hw address */
memcpy ( tmp_addr , slave - > dev - > dev_addr , ETH_ALEN ) ;
res = dev_set_mac_address ( slave - > dev , addr ) ;
/* restore net_device's hw address */
memcpy ( slave - > dev - > dev_addr , tmp_addr , ETH_ALEN ) ;
2008-11-19 22:56:05 -07:00
if ( res )
2005-04-16 16:20:36 -06:00
goto unwind ;
}
return 0 ;
unwind :
memcpy ( sa . sa_data , bond - > dev - > dev_addr , bond - > dev - > addr_len ) ;
sa . sa_family = bond - > dev - > type ;
/* unwind from head to the slave that failed */
stop_at = slave ;
bond_for_each_slave_from_to ( bond , slave , i , bond - > first_slave , stop_at ) {
memcpy ( tmp_addr , slave - > dev - > dev_addr , ETH_ALEN ) ;
dev_set_mac_address ( slave - > dev , & sa ) ;
memcpy ( slave - > dev - > dev_addr , tmp_addr , ETH_ALEN ) ;
}
return res ;
}
/************************ exported alb funcions ************************/
int bond_alb_initialize ( struct bonding * bond , int rlb_enabled )
{
int res ;
res = tlb_initialize ( bond ) ;
if ( res ) {
return res ;
}
if ( rlb_enabled ) {
bond - > alb_info . rlb_enabled = 1 ;
/* initialize rlb */
res = rlb_initialize ( bond ) ;
if ( res ) {
tlb_deinitialize ( bond ) ;
return res ;
}
2005-11-09 11:35:35 -07:00
} else {
bond - > alb_info . rlb_enabled = 0 ;
2005-04-16 16:20:36 -06:00
}
return 0 ;
}
void bond_alb_deinitialize ( struct bonding * bond )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
tlb_deinitialize ( bond ) ;
if ( bond_info - > rlb_enabled ) {
rlb_deinitialize ( bond ) ;
}
}
int bond_alb_xmit ( struct sk_buff * skb , struct net_device * bond_dev )
{
2008-11-13 00:37:49 -07:00
struct bonding * bond = netdev_priv ( bond_dev ) ;
2005-04-16 16:20:36 -06:00
struct ethhdr * eth_data ;
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct slave * tx_slave = NULL ;
2007-08-22 18:06:58 -06:00
static const __be32 ip_bcast = htonl ( 0xffffffff ) ;
2005-04-16 16:20:36 -06:00
int hash_size = 0 ;
int do_tx_balance = 1 ;
u32 hash_index = 0 ;
2007-04-20 23:47:35 -06:00
const u8 * hash_start = NULL ;
2005-04-16 16:20:36 -06:00
int res = 1 ;
2008-08-28 13:38:41 -06:00
struct ipv6hdr * ip6hdr ;
2005-04-16 16:20:36 -06:00
2007-03-19 16:30:44 -06:00
skb_reset_mac_header ( skb ) ;
2005-04-16 16:20:36 -06:00
eth_data = eth_hdr ( skb ) ;
2011-05-06 19:48:02 -06:00
/* make sure that the curr_active_slave do not change during tx
2005-04-16 16:20:36 -06:00
*/
read_lock ( & bond - > curr_slave_lock ) ;
switch ( ntohs ( skb - > protocol ) ) {
2007-04-20 23:47:35 -06:00
case ETH_P_IP : {
const struct iphdr * iph = ip_hdr ( skb ) ;
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ether_addr_equal_64bits ( eth_data - > h_dest , mac_bcast ) | |
2007-04-20 23:47:35 -06:00
( iph - > daddr = = ip_bcast ) | |
( iph - > protocol = = IPPROTO_IGMP ) ) {
2005-04-16 16:20:36 -06:00
do_tx_balance = 0 ;
break ;
}
2007-04-20 23:47:35 -06:00
hash_start = ( char * ) & ( iph - > daddr ) ;
hash_size = sizeof ( iph - > daddr ) ;
}
2005-04-16 16:20:36 -06:00
break ;
case ETH_P_IPV6 :
2008-08-28 13:38:41 -06:00
/* IPv6 doesn't really use broadcast mac address, but leave
* that here just in case .
*/
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ether_addr_equal_64bits ( eth_data - > h_dest , mac_bcast ) ) {
2005-04-16 16:20:36 -06:00
do_tx_balance = 0 ;
break ;
2008-08-28 13:38:41 -06:00
}
/* IPv6 uses all-nodes multicast as an equivalent to
* broadcasts in IPv4 .
*/
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ether_addr_equal_64bits ( eth_data - > h_dest , mac_v6_allmcast ) ) {
2008-08-28 13:38:41 -06:00
do_tx_balance = 0 ;
break ;
}
/* Additianally, DAD probes should not be tx-balanced as that
* will lead to false positives for duplicate addresses and
* prevent address configuration from working .
*/
ip6hdr = ipv6_hdr ( skb ) ;
if ( ipv6_addr_any ( & ip6hdr - > saddr ) ) {
do_tx_balance = 0 ;
break ;
2005-04-16 16:20:36 -06:00
}
2007-04-25 18:54:47 -06:00
hash_start = ( char * ) & ( ipv6_hdr ( skb ) - > daddr ) ;
hash_size = sizeof ( ipv6_hdr ( skb ) - > daddr ) ;
2005-04-16 16:20:36 -06:00
break ;
case ETH_P_IPX :
2007-08-22 18:06:58 -06:00
if ( ipx_hdr ( skb ) - > ipx_checksum ! = IPX_NO_CHECKSUM ) {
2005-04-16 16:20:36 -06:00
/* something is wrong with this packet */
do_tx_balance = 0 ;
break ;
}
if ( ipx_hdr ( skb ) - > ipx_type ! = IPX_TYPE_NCP ) {
/* The only protocol worth balancing in
* this family since it has an " ARP " like
* mechanism
*/
do_tx_balance = 0 ;
break ;
}
hash_start = ( char * ) eth_data - > h_dest ;
hash_size = ETH_ALEN ;
break ;
case ETH_P_ARP :
do_tx_balance = 0 ;
if ( bond_info - > rlb_enabled ) {
tx_slave = rlb_arp_xmit ( skb , bond ) ;
}
break ;
default :
do_tx_balance = 0 ;
break ;
}
if ( do_tx_balance ) {
hash_index = _simple_hash ( hash_start , hash_size ) ;
tx_slave = tlb_choose_channel ( bond , hash_index , skb - > len ) ;
}
if ( ! tx_slave ) {
/* unbalanced or unassigned, send through primary */
tx_slave = bond - > curr_active_slave ;
bond_info - > unbalanced_load + = skb - > len ;
}
if ( tx_slave & & SLAVE_IS_OK ( tx_slave ) ) {
if ( tx_slave ! = bond - > curr_active_slave ) {
memcpy ( eth_data - > h_source ,
tx_slave - > dev - > dev_addr ,
ETH_ALEN ) ;
}
res = bond_dev_queue_xmit ( bond , skb , tx_slave - > dev ) ;
} else {
if ( tx_slave ) {
2012-01-09 05:01:37 -07:00
_lock_tx_hashtbl ( bond ) ;
__tlb_clear_slave ( bond , tx_slave , 0 ) ;
_unlock_tx_hashtbl ( bond ) ;
2005-04-16 16:20:36 -06:00
}
}
2012-06-12 23:30:07 -06:00
read_unlock ( & bond - > curr_slave_lock ) ;
2005-04-16 16:20:36 -06:00
if ( res ) {
/* no suitable interface, frame not sent */
2012-06-12 23:30:07 -06:00
kfree_skb ( skb ) ;
2005-04-16 16:20:36 -06:00
}
2009-07-05 20:23:38 -06:00
return NETDEV_TX_OK ;
2005-04-16 16:20:36 -06:00
}
2007-10-17 18:37:45 -06:00
void bond_alb_monitor ( struct work_struct * work )
2005-04-16 16:20:36 -06:00
{
2007-10-17 18:37:45 -06:00
struct bonding * bond = container_of ( work , struct bonding ,
alb_work . work ) ;
2005-04-16 16:20:36 -06:00
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
struct slave * slave ;
int i ;
read_lock ( & bond - > lock ) ;
if ( bond - > slave_cnt = = 0 ) {
bond_info - > tx_rebalance_counter = 0 ;
bond_info - > lp_counter = 0 ;
goto re_arm ;
}
bond_info - > tx_rebalance_counter + + ;
bond_info - > lp_counter + + ;
/* send learning packets */
if ( bond_info - > lp_counter > = BOND_ALB_LP_TICKS ) {
/* change of curr_active_slave involves swapping of mac addresses.
* in order to avoid this swapping from happening while
* sending the learning packets , the curr_slave_lock must be held for
* read .
*/
read_lock ( & bond - > curr_slave_lock ) ;
bond_for_each_slave ( bond , slave , i ) {
2005-11-09 11:36:50 -07:00
alb_send_learning_packets ( slave , slave - > dev - > dev_addr ) ;
2005-04-16 16:20:36 -06:00
}
read_unlock ( & bond - > curr_slave_lock ) ;
bond_info - > lp_counter = 0 ;
}
/* rebalance tx traffic */
if ( bond_info - > tx_rebalance_counter > = BOND_TLB_REBALANCE_TICKS ) {
read_lock ( & bond - > curr_slave_lock ) ;
bond_for_each_slave ( bond , slave , i ) {
tlb_clear_slave ( bond , slave , 1 ) ;
if ( slave = = bond - > curr_active_slave ) {
SLAVE_TLB_INFO ( slave ) . load =
bond_info - > unbalanced_load /
BOND_TLB_REBALANCE_INTERVAL ;
bond_info - > unbalanced_load = 0 ;
}
}
read_unlock ( & bond - > curr_slave_lock ) ;
bond_info - > tx_rebalance_counter = 0 ;
}
/* handle rlb stuff */
if ( bond_info - > rlb_enabled ) {
if ( bond_info - > primary_is_promisc & &
( + + bond_info - > rlb_promisc_timeout_counter > = RLB_PROMISC_TIMEOUT ) ) {
2007-10-17 18:37:51 -06:00
/*
* dev_set_promiscuity requires rtnl and
bonding: eliminate bond_close race conditions
This patch resolves two sets of race conditions.
Mitsuo Hayasaka <mitsuo.hayasaka.hu@hitachi.com> reported the
first, as follows:
The bond_close() calls cancel_delayed_work() to cancel delayed works.
It, however, cannot cancel works that were already queued in workqueue.
The bond_open() initializes work->data, and proccess_one_work() refers
get_work_cwq(work)->wq->flags. The get_work_cwq() returns NULL when
work->data has been initialized. Thus, a panic occurs.
He included a patch that converted the cancel_delayed_work calls
in bond_close to flush_delayed_work_sync, which eliminated the above
problem.
His patch is incorporated, at least in principle, into this
patch. In this patch, we use cancel_delayed_work_sync in place of
flush_delayed_work_sync, and also convert bond_uninit in addition to
bond_close.
This conversion to _sync, however, opens new races between
bond_close and three periodically executing workqueue functions:
bond_mii_monitor, bond_alb_monitor and bond_activebackup_arp_mon.
The race occurs because bond_close and bond_uninit are always
called with RTNL held, and these workqueue functions may acquire RTNL to
perform failover-related activities. If bond_close or bond_uninit is
waiting in cancel_delayed_work_sync, deadlock occurs.
These deadlocks are resolved by having the workqueue functions
acquire RTNL conditionally. If the rtnl_trylock() fails, the functions
reschedule and return immediately. For the cases that are attempting to
perform link failover, a delay of 1 is used; for the other cases, the
normal interval is used (as those activities are not as time critical).
Additionally, the bond_mii_monitor function now stores the delay
in a variable (mimicing the structure of activebackup_arp_mon).
Lastly, all of the above renders the kill_timers sentinel moot,
and therefore it has been removed.
Tested-by: Mitsuo Hayasaka <mitsuo.hayasaka.hu@hitachi.com>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2011-10-28 09:42:50 -06:00
* nothing else . Avoid race with bond_close .
2007-10-17 18:37:51 -06:00
*/
read_unlock ( & bond - > lock ) ;
bonding: eliminate bond_close race conditions
This patch resolves two sets of race conditions.
Mitsuo Hayasaka <mitsuo.hayasaka.hu@hitachi.com> reported the
first, as follows:
The bond_close() calls cancel_delayed_work() to cancel delayed works.
It, however, cannot cancel works that were already queued in workqueue.
The bond_open() initializes work->data, and proccess_one_work() refers
get_work_cwq(work)->wq->flags. The get_work_cwq() returns NULL when
work->data has been initialized. Thus, a panic occurs.
He included a patch that converted the cancel_delayed_work calls
in bond_close to flush_delayed_work_sync, which eliminated the above
problem.
His patch is incorporated, at least in principle, into this
patch. In this patch, we use cancel_delayed_work_sync in place of
flush_delayed_work_sync, and also convert bond_uninit in addition to
bond_close.
This conversion to _sync, however, opens new races between
bond_close and three periodically executing workqueue functions:
bond_mii_monitor, bond_alb_monitor and bond_activebackup_arp_mon.
The race occurs because bond_close and bond_uninit are always
called with RTNL held, and these workqueue functions may acquire RTNL to
perform failover-related activities. If bond_close or bond_uninit is
waiting in cancel_delayed_work_sync, deadlock occurs.
These deadlocks are resolved by having the workqueue functions
acquire RTNL conditionally. If the rtnl_trylock() fails, the functions
reschedule and return immediately. For the cases that are attempting to
perform link failover, a delay of 1 is used; for the other cases, the
normal interval is used (as those activities are not as time critical).
Additionally, the bond_mii_monitor function now stores the delay
in a variable (mimicing the structure of activebackup_arp_mon).
Lastly, all of the above renders the kill_timers sentinel moot,
and therefore it has been removed.
Tested-by: Mitsuo Hayasaka <mitsuo.hayasaka.hu@hitachi.com>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2011-10-28 09:42:50 -06:00
if ( ! rtnl_trylock ( ) ) {
read_lock ( & bond - > lock ) ;
goto re_arm ;
}
2007-10-17 18:37:51 -06:00
2005-04-16 16:20:36 -06:00
bond_info - > rlb_promisc_timeout_counter = 0 ;
/* If the primary was set to promiscuous mode
* because a slave was disabled then
* it can now leave promiscuous mode .
*/
dev_set_promiscuity ( bond - > curr_active_slave - > dev , - 1 ) ;
bond_info - > primary_is_promisc = 0 ;
2007-10-17 18:37:51 -06:00
rtnl_unlock ( ) ;
read_lock ( & bond - > lock ) ;
}
2005-04-16 16:20:36 -06:00
if ( bond_info - > rlb_rebalance ) {
bond_info - > rlb_rebalance = 0 ;
rlb_rebalance ( bond ) ;
}
/* check if clients need updating */
if ( bond_info - > rx_ntt ) {
if ( bond_info - > rlb_update_delay_counter ) {
- - bond_info - > rlb_update_delay_counter ;
} else {
rlb_update_rx_clients ( bond ) ;
if ( bond_info - > rlb_update_retry_counter ) {
- - bond_info - > rlb_update_retry_counter ;
} else {
bond_info - > rx_ntt = 0 ;
}
}
}
}
re_arm :
bonding: eliminate bond_close race conditions
This patch resolves two sets of race conditions.
Mitsuo Hayasaka <mitsuo.hayasaka.hu@hitachi.com> reported the
first, as follows:
The bond_close() calls cancel_delayed_work() to cancel delayed works.
It, however, cannot cancel works that were already queued in workqueue.
The bond_open() initializes work->data, and proccess_one_work() refers
get_work_cwq(work)->wq->flags. The get_work_cwq() returns NULL when
work->data has been initialized. Thus, a panic occurs.
He included a patch that converted the cancel_delayed_work calls
in bond_close to flush_delayed_work_sync, which eliminated the above
problem.
His patch is incorporated, at least in principle, into this
patch. In this patch, we use cancel_delayed_work_sync in place of
flush_delayed_work_sync, and also convert bond_uninit in addition to
bond_close.
This conversion to _sync, however, opens new races between
bond_close and three periodically executing workqueue functions:
bond_mii_monitor, bond_alb_monitor and bond_activebackup_arp_mon.
The race occurs because bond_close and bond_uninit are always
called with RTNL held, and these workqueue functions may acquire RTNL to
perform failover-related activities. If bond_close or bond_uninit is
waiting in cancel_delayed_work_sync, deadlock occurs.
These deadlocks are resolved by having the workqueue functions
acquire RTNL conditionally. If the rtnl_trylock() fails, the functions
reschedule and return immediately. For the cases that are attempting to
perform link failover, a delay of 1 is used; for the other cases, the
normal interval is used (as those activities are not as time critical).
Additionally, the bond_mii_monitor function now stores the delay
in a variable (mimicing the structure of activebackup_arp_mon).
Lastly, all of the above renders the kill_timers sentinel moot,
and therefore it has been removed.
Tested-by: Mitsuo Hayasaka <mitsuo.hayasaka.hu@hitachi.com>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2011-10-28 09:42:50 -06:00
queue_delayed_work ( bond - > wq , & bond - > alb_work , alb_delta_in_ticks ) ;
2005-04-16 16:20:36 -06:00
read_unlock ( & bond - > lock ) ;
}
/* assumption: called before the slave is attached to the bond
* and not locked by the bond lock
*/
int bond_alb_init_slave ( struct bonding * bond , struct slave * slave )
{
int res ;
2012-01-18 05:24:54 -07:00
res = alb_set_slave_mac_addr ( slave , slave - > perm_hwaddr ) ;
2005-04-16 16:20:36 -06:00
if ( res ) {
return res ;
}
res = alb_handle_addr_collision_on_attach ( bond , slave ) ;
if ( res ) {
return res ;
}
tlb_init_slave ( slave ) ;
/* order a rebalance ASAP */
bond - > alb_info . tx_rebalance_counter = BOND_TLB_REBALANCE_TICKS ;
if ( bond - > alb_info . rlb_enabled ) {
bond - > alb_info . rlb_rebalance = 1 ;
}
return 0 ;
}
2008-01-17 17:24:59 -07:00
/*
* Remove slave from tlb and rlb hash tables , and fix up MAC addresses
* if necessary .
*
* Caller must hold RTNL and no other locks
*/
2005-04-16 16:20:36 -06:00
void bond_alb_deinit_slave ( struct bonding * bond , struct slave * slave )
{
if ( bond - > slave_cnt > 1 ) {
alb_change_hw_addr_on_detach ( bond , slave ) ;
}
tlb_clear_slave ( bond , slave , 0 ) ;
if ( bond - > alb_info . rlb_enabled ) {
bond - > alb_info . next_rx_slave = NULL ;
rlb_clear_slave ( bond , slave ) ;
}
}
/* Caller must hold bond lock for read */
void bond_alb_handle_link_change ( struct bonding * bond , struct slave * slave , char link )
{
struct alb_bond_info * bond_info = & ( BOND_ALB_INFO ( bond ) ) ;
if ( link = = BOND_LINK_DOWN ) {
tlb_clear_slave ( bond , slave , 0 ) ;
if ( bond - > alb_info . rlb_enabled ) {
rlb_clear_slave ( bond , slave ) ;
}
} else if ( link = = BOND_LINK_UP ) {
/* order a rebalance ASAP */
bond_info - > tx_rebalance_counter = BOND_TLB_REBALANCE_TICKS ;
if ( bond - > alb_info . rlb_enabled ) {
bond - > alb_info . rlb_rebalance = 1 ;
/* If the updelay module parameter is smaller than the
* forwarding delay of the switch the rebalance will
* not work because the rebalance arp replies will
* not be forwarded to the clients . .
*/
}
}
}
/**
* bond_alb_handle_active_change - assign new curr_active_slave
* @ bond : our bonding struct
* @ new_slave : new slave to assign
*
* Set the bond - > curr_active_slave to @ new_slave and handle
* mac address swapping and promiscuity changes as needed .
*
2007-10-17 18:37:49 -06:00
* If new_slave is NULL , caller must hold curr_slave_lock or
* bond - > lock for write .
*
* If new_slave is not NULL , caller must hold RTNL , bond - > lock for
* read and curr_slave_lock for write . Processing here may sleep , so
* no other locks may be held .
2005-04-16 16:20:36 -06:00
*/
void bond_alb_handle_active_change ( struct bonding * bond , struct slave * new_slave )
2009-02-14 04:15:33 -07:00
__releases ( & bond - > curr_slave_lock )
__releases ( & bond - > lock )
__acquires ( & bond - > lock )
__acquires ( & bond - > curr_slave_lock )
2005-04-16 16:20:36 -06:00
{
struct slave * swap_slave ;
int i ;
if ( bond - > curr_active_slave = = new_slave ) {
return ;
}
if ( bond - > curr_active_slave & & bond - > alb_info . primary_is_promisc ) {
dev_set_promiscuity ( bond - > curr_active_slave - > dev , - 1 ) ;
bond - > alb_info . primary_is_promisc = 0 ;
bond - > alb_info . rlb_promisc_timeout_counter = 0 ;
}
swap_slave = bond - > curr_active_slave ;
bond - > curr_active_slave = new_slave ;
if ( ! new_slave | | ( bond - > slave_cnt = = 0 ) ) {
return ;
}
/* set the new curr_active_slave to the bonds mac address
* i . e . swap mac addresses of old curr_active_slave and new curr_active_slave
*/
if ( ! swap_slave ) {
struct slave * tmp_slave ;
/* find slave that is holding the bond's mac address */
bond_for_each_slave ( bond , tmp_slave , i ) {
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ether_addr_equal_64bits ( tmp_slave - > dev - > dev_addr ,
bond - > dev - > dev_addr ) ) {
2005-04-16 16:20:36 -06:00
swap_slave = tmp_slave ;
break ;
}
}
}
2007-10-17 18:37:49 -06:00
/*
* Arrange for swap_slave and new_slave to temporarily be
* ignored so we can mess with their MAC addresses without
* fear of interference from transmit activity .
*/
if ( swap_slave ) {
tlb_clear_slave ( bond , swap_slave , 1 ) ;
}
tlb_clear_slave ( bond , new_slave , 1 ) ;
write_unlock_bh ( & bond - > curr_slave_lock ) ;
read_unlock ( & bond - > lock ) ;
2008-01-17 17:24:58 -07:00
ASSERT_RTNL ( ) ;
2005-04-16 16:20:36 -06:00
/* curr_active_slave must be set before calling alb_swap_mac_addr */
if ( swap_slave ) {
/* swap mac address */
2013-05-27 17:14:51 -06:00
alb_swap_mac_addr ( swap_slave , new_slave ) ;
2005-04-16 16:20:36 -06:00
} else {
/* set the new_slave to the bond mac address */
2012-01-18 05:24:54 -07:00
alb_set_slave_mac_addr ( new_slave , bond - > dev - > dev_addr ) ;
2007-10-17 18:37:49 -06:00
}
if ( swap_slave ) {
alb_fasten_mac_swap ( bond , swap_slave , new_slave ) ;
2008-01-17 17:24:59 -07:00
read_lock ( & bond - > lock ) ;
2007-10-17 18:37:49 -06:00
} else {
2008-01-17 17:24:59 -07:00
read_lock ( & bond - > lock ) ;
2005-04-16 16:20:36 -06:00
alb_send_learning_packets ( new_slave , bond - > dev - > dev_addr ) ;
}
2007-10-17 18:37:49 -06:00
write_lock_bh ( & bond - > curr_slave_lock ) ;
2005-04-16 16:20:36 -06:00
}
2007-10-17 18:37:49 -06:00
/*
* Called with RTNL
*/
2005-04-16 16:20:36 -06:00
int bond_alb_set_mac_address ( struct net_device * bond_dev , void * addr )
2009-02-14 04:15:33 -07:00
__acquires ( & bond - > lock )
2009-05-04 03:03:37 -06:00
__releases ( & bond - > lock )
2005-04-16 16:20:36 -06:00
{
2008-11-13 00:37:49 -07:00
struct bonding * bond = netdev_priv ( bond_dev ) ;
2005-04-16 16:20:36 -06:00
struct sockaddr * sa = addr ;
struct slave * slave , * swap_slave ;
int res ;
int i ;
if ( ! is_valid_ether_addr ( sa - > sa_data ) ) {
return - EADDRNOTAVAIL ;
}
res = alb_set_mac_address ( bond , addr ) ;
if ( res ) {
return res ;
}
memcpy ( bond_dev - > dev_addr , sa - > sa_data , bond_dev - > addr_len ) ;
/* If there is no curr_active_slave there is nothing else to do.
* Otherwise we ' ll need to pass the new address to it and handle
* duplications .
*/
if ( ! bond - > curr_active_slave ) {
return 0 ;
}
swap_slave = NULL ;
bond_for_each_slave ( bond , slave , i ) {
net, drivers/net: Convert compare_ether_addr_64bits to ether_addr_equal_64bits
Use the new bool function ether_addr_equal_64bits to add
some clarity and reduce the likelihood for misuse of
compare_ether_addr_64bits for sorting.
Done via cocci script:
$ cat compare_ether_addr_64bits.cocci
@@
expression a,b;
@@
- !compare_ether_addr_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- compare_ether_addr_64bits(a, b)
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) == 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !ether_addr_equal_64bits(a, b) != 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) == 0
+ !ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- ether_addr_equal_64bits(a, b) != 0
+ ether_addr_equal_64bits(a, b)
@@
expression a,b;
@@
- !!ether_addr_equal_64bits(a, b)
+ ether_addr_equal_64bits(a, b)
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2012-05-09 11:04:04 -06:00
if ( ether_addr_equal_64bits ( slave - > dev - > dev_addr ,
bond_dev - > dev_addr ) ) {
2005-04-16 16:20:36 -06:00
swap_slave = slave ;
break ;
}
}
if ( swap_slave ) {
2013-05-27 17:14:51 -06:00
alb_swap_mac_addr ( swap_slave , bond - > curr_active_slave ) ;
2007-10-17 18:37:49 -06:00
alb_fasten_mac_swap ( bond , swap_slave , bond - > curr_active_slave ) ;
2005-04-16 16:20:36 -06:00
} else {
2012-01-18 05:24:54 -07:00
alb_set_slave_mac_addr ( bond - > curr_active_slave , bond_dev - > dev_addr ) ;
2005-04-16 16:20:36 -06:00
2009-05-04 03:03:37 -06:00
read_lock ( & bond - > lock ) ;
2005-04-16 16:20:36 -06:00
alb_send_learning_packets ( bond - > curr_active_slave , bond_dev - > dev_addr ) ;
if ( bond - > alb_info . rlb_enabled ) {
/* inform clients mac address has changed */
rlb_req_update_slave_clients ( bond , bond - > curr_active_slave ) ;
}
2009-05-04 03:03:37 -06:00
read_unlock ( & bond - > lock ) ;
2005-04-16 16:20:36 -06:00
}
return 0 ;
}
void bond_alb_clear_vlan ( struct bonding * bond , unsigned short vlan_id )
{
if ( bond - > alb_info . current_alb_vlan & &
( bond - > alb_info . current_alb_vlan - > vlan_id = = vlan_id ) ) {
bond - > alb_info . current_alb_vlan = NULL ;
}
if ( bond - > alb_info . rlb_enabled ) {
rlb_clear_vlan ( bond , vlan_id ) ;
}
}