linux.git/drivers/net/bonding, branch v3.12-rc2

bonding: Make alb learning packet interval configurable

2013-09-16T02:20:44+00:00

running bonding in ALB mode requires that learning packets be sent periodically,
so that the switch knows where to send responding traffic.  However, depending
on switch configuration, there may not be any need to send traffic at the
default rate of 3 packets per second, which represents little more than wasted
data.  Allow the ALB learning packet interval to be made configurable via sysfs

Signed-off-by: Neil Horman 
Acked-by: Acked-by: Veaceslav Falico 
CC: Jay Vosburgh 
CC: Andy Gospodarek 
CC: "David S. Miller" 
Signed-off-by: Andy Gospodarek 
Signed-off-by: David S. Miller

bonding: fix bond_arp_rcv setting and arp validate desync state

2013-09-11T19:55:17+00:00

We make bond_arp_rcv global so it can be used in bond_sysfs if the bond
interface is up and arp_interval is being changed to a positive value
and cleared otherwise as per Jay's suggestion.
This also fixes a problem where bond_arp_rcv was set even though
arp_validate was disabled while the bond was up by unsetting recv_probe
in bond_store_arp_validate and respectively setting it if enabled.

Signed-off-by: Nikolay Aleksandrov 
Signed-off-by: Marcelo Ricardo Leitner 
Acked-by: Veaceslav Falico 
Signed-off-by: David S. Miller

bonding: fix store_arp_validate race with mode change

2013-09-11T19:55:17+00:00

We need to protect store_arp_validate via rtnl because it can race with
mode changing and we can end up having arp_validate set in a mode
different from active-backup.

Signed-off-by: Nikolay Aleksandrov 
Acked-by: Veaceslav Falico 
Signed-off-by: David S. Miller

bonding: drop read_lock in bond_compute_features

2013-09-04T04:27:25+00:00

bond_compute_features is always called with RTNL held, so we can safely
drop the read bond->lock.

Signed-off-by: Nikolay Aleksandrov 
Signed-off-by: David S. Miller

bonding: drop read_lock in bond_fix_features

2013-09-04T04:27:25+00:00

We're protected by RTNL so nothing can happen and we can safely drop the
read bond->lock.

Signed-off-by: Nikolay Aleksandrov 
Signed-off-by: David S. Miller

bonding: simplify bond_3ad_update_lacp_rate and use RTNL for sync

2013-09-04T04:27:24+00:00

We can drop the use of bond->lock for mutual exclusion in
bond_3ad_update_lacp_rate and use RTNL in the sysfs store function
instead. This way we'll prevent races with mode change and interface
up/down as well as simplify update_lacp_rate by removing the check for
port->slave because it'll always be initialized (done while enslaving
with RTNL). This change will also help in the future removal of reader
bond->lock from bond_enslave.

Signed-off-by: Nikolay Aleksandrov 
Signed-off-by: David S. Miller

bonding: trivial: remove outdated comment and braces

2013-09-04T04:27:24+00:00

We don't have to release all slaves when closing the bond dev, so remove
the outdated comment and the braces around the left single statement.

Signed-off-by: Nikolay Aleksandrov 
Signed-off-by: David S. Miller

bonding: simplify and fix peer notification

2013-09-04T04:27:24+00:00

This patch aims to remove a use of the bond->lock for mutual exclusion
which will later allow easier migration to RCU of the users of this
functionality. We use RTNL as a synchronizing mechanism since it's
always held when send_peer_notif is set, and when it is decremented from
the notifier function. We can also drop some locking, and fix the
leakage of the send_peer_notif counter.

Signed-off-by: Nikolay Aleksandrov 
Signed-off-by: David S. Miller

bonding: use rlb_client_info->vlan_id instead of ->tag

2013-09-04T02:02:32+00:00

Store VID in ->vlan_id (if any), and remove the useless ->tag.

CC: Jay Vosburgh 
CC: Andy Gospodarek 
Signed-off-by: Veaceslav Falico 
Signed-off-by: David S. Miller

bonding: remove bond_vlan_used()

2013-09-04T02:02:32+00:00

We're using it currently to verify if we have vlans before getting the tag
from the skb we're about to send. It's useless because the vlan_get_tag()
verifies if the skb has the tag (and returns an error if not), and we can
receive tagged skbs only if we *already* have vlans.

Plus, the current RCUed implementation is kind of useless anyway - the we
can remove the last vlan in the moment we return from the function.

So remove the only usage of it and the whole function.

CC: Jay Vosburgh 
CC: Andy Gospodarek 
Signed-off-by: Veaceslav Falico 
Signed-off-by: David S. Miller