2005-04-17 06:20:36 +08:00
|
|
|
/*
|
|
|
|
|
* Bond several ethernet interfaces into a Cisco, running 'Etherchannel'.
|
|
|
|
|
*
|
|
|
|
|
* Portions are (c) Copyright 1995 Simon "Guru Aleph-Null" Janes
|
|
|
|
|
* NCM: Network and Communications Management, Inc.
|
|
|
|
|
*
|
|
|
|
|
* BUT, I'm the one who modified it for ethernet, so:
|
|
|
|
|
* (c) Copyright 1999, Thomas Davis, tadavis@lbl.gov
|
|
|
|
|
*
|
|
|
|
|
* This software may be used and distributed according to the terms
|
|
|
|
|
* of the GNU Public License, incorporated herein by reference.
|
|
|
|
|
*
|
|
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
#ifndef _LINUX_BONDING_H
|
|
|
|
|
#define _LINUX_BONDING_H
|
|
|
|
|
|
|
|
|
|
#include <linux/timer.h>
|
|
|
|
|
#include <linux/proc_fs.h>
|
|
|
|
|
#include <linux/if_bonding.h>
|
2010-10-14 00:01:50 +08:00
|
|
|
#include <linux/cpumask.h>
|
bonding: send IPv6 neighbor advertisement on failover
This patch adds better IPv6 failover support for bonding devices,
especially when in active-backup mode and there are only IPv6 addresses
configured, as reported by Alex Sidorenko.
- Creates a new file, net/drivers/bonding/bond_ipv6.c, for the
IPv6-specific routines. Both regular bonds and VLANs over bonds
are supported.
- Adds a new tunable, num_unsol_na, to limit the number of unsolicited
IPv6 Neighbor Advertisements that are sent on a failover event.
Default is 1.
- Creates two new IPv6 neighbor discovery functions:
ndisc_build_skb()
ndisc_send_skb()
These were required to support VLANs since we have to be able to
add the VLAN id to the skb since ndisc_send_na() and friends
shouldn't be asked to do this. These two routines are basically
__ndisc_send() split into two pieces, in a slightly different order.
- Updates Documentation/networking/bonding.txt and bumps the rev of bond
support to 3.4.0.
On failover, this new code will generate one packet:
- An unsolicited IPv6 Neighbor Advertisement, which helps the switch
learn that the address has moved to the new slave.
Testing has shown that sending just the NA results in pretty good
behavior when in active-back mode, I saw no lost ping packets for example.
Signed-off-by: Brian Haley <brian.haley@hp.com>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
2008-11-05 09:51:14 +08:00
|
|
|
#include <linux/in6.h>
|
2011-02-18 07:43:32 +08:00
|
|
|
#include <linux/netpoll.h>
|
2012-03-23 00:14:29 +08:00
|
|
|
#include <linux/inetdevice.h>
|
2012-11-28 07:57:04 +08:00
|
|
|
#include <linux/etherdevice.h>
|
reciprocal_divide: update/correction of the algorithm
Jakub Zawadzki noticed that some divisions by reciprocal_divide()
were not correct [1][2], which he could also show with BPF code
after divisions are transformed into reciprocal_value() for runtime
invariance which can be passed to reciprocal_divide() later on;
reverse in BPF dump ended up with a different, off-by-one K in
some situations.
This has been fixed by Eric Dumazet in commit aee636c4809fa5
("bpf: do not use reciprocal divide"). This follow-up patch
improves reciprocal_value() and reciprocal_divide() to work in
all cases by using Granlund and Montgomery method, so that also
future use is safe and without any non-obvious side-effects.
Known problems with the old implementation were that division by 1
always returned 0 and some off-by-ones when the dividend and divisor
where very large. This seemed to not be problematic with its
current users, as far as we can tell. Eric Dumazet checked for
the slab usage, we cannot surely say so in the case of flex_array.
Still, in order to fix that, we propose an extension from the
original implementation from commit 6a2d7a955d8d resp. [3][4],
by using the algorithm proposed in "Division by Invariant Integers
Using Multiplication" [5], Torbjörn Granlund and Peter L.
Montgomery, that is, pseudocode for q = n/d where q, n, d is in
u32 universe:
1) Initialization:
int l = ceil(log_2 d)
uword m' = floor((1<<32)*((1<<l)-d)/d)+1
int sh_1 = min(l,1)
int sh_2 = max(l-1,0)
2) For q = n/d, all uword:
uword t = (n*m')>>32
q = (t+((n-t)>>sh_1))>>sh_2
The assembler implementation from Agner Fog [6] also helped a lot
while implementing. We have tested the implementation on x86_64,
ppc64, i686, s390x; on x86_64/haswell we're still half the latency
compared to normal divide.
Joint work with Daniel Borkmann.
[1] http://www.wireshark.org/~darkjames/reciprocal-buggy.c
[2] http://www.wireshark.org/~darkjames/set-and-dump-filter-k-bug.c
[3] https://gmplib.org/~tege/division-paper.pdf
[4] http://homepage.cs.uiowa.edu/~jones/bcd/divide.html
[5] http://citeseerx.ist.psu.edu/viewdoc/summary?doi=10.1.1.1.2556
[6] http://www.agner.org/optimize/asmlib.zip
Reported-by: Jakub Zawadzki <darkjames-ws@darkjames.pl>
Cc: Eric Dumazet <eric.dumazet@gmail.com>
Cc: Austin S Hemmelgarn <ahferroin7@gmail.com>
Cc: linux-kernel@vger.kernel.org
Cc: Jesse Gross <jesse@nicira.com>
Cc: Jamal Hadi Salim <jhs@mojatatu.com>
Cc: Stephen Hemminger <stephen@networkplumber.org>
Cc: Matt Mackall <mpm@selenic.com>
Cc: Pekka Enberg <penberg@kernel.org>
Cc: Christoph Lameter <cl@linux-foundation.org>
Cc: Andy Gospodarek <andy@greyhouse.net>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Jakub Zawadzki <darkjames-ws@darkjames.pl>
Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-01-22 09:29:41 +08:00
|
|
|
#include <linux/reciprocal_div.h>
|
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
#include "bond_3ad.h"
|
|
|
|
|
#include "bond_alb.h"
|
bonding: add infrastructure for an option API
This patch adds the necessary basic infrastructure to support
centralized and unified option manipulation API for the bonding. The new
structure bond_option will be used to describe each option with its
dependencies on modes which will be checked automatically thus removing a
lot of duplicated code. Also automatic range checking is added for
some options. Currently the option setting function requires RTNL to
be acquired prior to calling it, since many options already rely on RTNL
it seemed like the best choice to protect all against common race
conditions.
In order to add an option the following steps need to be done:
1. Add an entry BOND_OPT_<option> to bond_options.h so it gets a unique id
and a bit corresponding to the id
2. Add a bond_option entry to the bond_opts[] array in bond_options.c which
describes the option, its dependencies and its manipulation function
3. Add code to export the option through sysfs and/or as a module parameter
(the sysfs export will be made automatically in the future)
The options can have different flags set, currently the following are
supported:
BOND_OPTFLAG_NOSLAVES - require that the bond device has no slaves prior
to setting the option
BOND_OPTFLAG_IFDOWN - require that the bond device is down prior to
setting the option
BOND_OPTFLAG_RAWVAL - don't parse the value but return it raw for the
option to parse
There's a new value structure to describe different types of values
which can have the following flags:
BOND_VALFLAG_DEFAULT - marks the default option (permanent string alias
to this option is "default")
BOND_VALFLAG_MIN - the minimum value that this option can have
BOND_VALFLAG_MAX - the maximum value that this option can have
An example would be nice here, so if we have an option which can have
the values "off"(2), "special"(4, default) and supports a range, say
16 - 32, it should be defined as follows:
"off", 2,
"special", 4, BOND_VALFLAG_DEFAULT,
"rangemin", 16, BOND_VALFLAG_MIN,
"rangemax", 32, BOND_VALFLAG_MAX
So we have the valid intervals: [2, 2], [4, 4], [16, 32]
Also the valid strings: "off" = 2, "special" and "default" = 4
"rangemin" = 16, "rangemax" = 32
BOND_VALFLAG_(MIN|MAX) can be used to specify a valid range for an
option, if MIN is omitted then 0 is considered as a minimum. If an
exact match is found in the values[] table it will be returned,
otherwise the range is tried (if available).
The option parameter passing is done by using a special structure called
bond_opt_value which can take either a string or a value to parse. One
of the bond_opt_init(val|str) macros should be used depending on which
one does the user want to parse (string or value). Then a call to
__bond_opt_set should be done under RTNL.
Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-01-22 21:53:16 +08:00
|
|
|
#include "bond_options.h"
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2011-04-26 23:25:52 +08:00
|
|
|
#define DRV_VERSION "3.7.1"
|
|
|
|
|
#define DRV_RELDATE "April 27, 2011"
|
2005-04-17 06:20:36 +08:00
|
|
|
#define DRV_NAME "bonding"
|
|
|
|
|
#define DRV_DESCRIPTION "Ethernet Channel Bonding Driver"
|
|
|
|
|
|
2011-03-07 05:58:46 +08:00
|
|
|
#define bond_version DRV_DESCRIPTION ": v" DRV_VERSION " (" DRV_RELDATE ")\n"
|
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
#define BOND_MAX_ARP_TARGETS 16
|
|
|
|
|
|
bonding: disable arp and enable mii monitoring when bond change to no uses arp mode
Because the ARP monitoring is not support for 802.3ad, but I still
could change the mode to 802.3ad from ab mode while ARP monitoring
is running, it is incorrect.
So add a check for 802.3ad in bonding_store_mode to fix the problem,
and make a new macro BOND_NO_USES_ARP() to simplify the code.
v2: according to the Dan Williams's suggestion, bond mode is the most
important bond option, it should override any of the other sub-options.
So when the mode is changed, the conficting values should be cleared
or reset, otherwise the user has to duplicate more operations to modify
the logic. I disable the arp and enable mii monitoring when the bond mode
is changed to AB, TB and 8023AD if the arp interval is true.
v3: according to the Nik's suggestion, the default value of miimon should need
a name, there is several place to use it, and the bond_store_arp_interval()
could use micro BOND_NO_USES_ARP to make the code more simpify.
Suggested-by: Dan Williams <dcbw@redhat.com>
Suggested-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Reviewed-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-11-22 22:28:43 +08:00
|
|
|
#define BOND_DEFAULT_MIIMON 100
|
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
#define IS_UP(dev) \
|
|
|
|
|
((((dev)->flags & IFF_UP) == IFF_UP) && \
|
|
|
|
|
netif_running(dev) && \
|
|
|
|
|
netif_carrier_ok(dev))
|
|
|
|
|
|
|
|
|
|
/*
|
|
|
|
|
* Checks whether slave is ready for transmit.
|
|
|
|
|
*/
|
|
|
|
|
#define SLAVE_IS_OK(slave) \
|
|
|
|
|
(((slave)->dev->flags & IFF_UP) && \
|
|
|
|
|
netif_running((slave)->dev) && \
|
|
|
|
|
((slave)->link == BOND_LINK_UP) && \
|
2011-03-12 11:14:37 +08:00
|
|
|
bond_is_active_slave(slave))
|
2005-04-17 06:20:36 +08:00
|
|
|
|
|
|
|
|
|
|
|
|
|
#define USES_PRIMARY(mode) \
|
|
|
|
|
(((mode) == BOND_MODE_ACTIVEBACKUP) || \
|
|
|
|
|
((mode) == BOND_MODE_TLB) || \
|
|
|
|
|
((mode) == BOND_MODE_ALB))
|
|
|
|
|
|
bonding: disable arp and enable mii monitoring when bond change to no uses arp mode
Because the ARP monitoring is not support for 802.3ad, but I still
could change the mode to 802.3ad from ab mode while ARP monitoring
is running, it is incorrect.
So add a check for 802.3ad in bonding_store_mode to fix the problem,
and make a new macro BOND_NO_USES_ARP() to simplify the code.
v2: according to the Dan Williams's suggestion, bond mode is the most
important bond option, it should override any of the other sub-options.
So when the mode is changed, the conficting values should be cleared
or reset, otherwise the user has to duplicate more operations to modify
the logic. I disable the arp and enable mii monitoring when the bond mode
is changed to AB, TB and 8023AD if the arp interval is true.
v3: according to the Nik's suggestion, the default value of miimon should need
a name, there is several place to use it, and the bond_store_arp_interval()
could use micro BOND_NO_USES_ARP to make the code more simpify.
Suggested-by: Dan Williams <dcbw@redhat.com>
Suggested-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Reviewed-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-11-22 22:28:43 +08:00
|
|
|
#define BOND_NO_USES_ARP(mode) \
|
|
|
|
|
(((mode) == BOND_MODE_8023AD) || \
|
|
|
|
|
((mode) == BOND_MODE_TLB) || \
|
|
|
|
|
((mode) == BOND_MODE_ALB))
|
|
|
|
|
|
2010-06-02 16:40:18 +08:00
|
|
|
#define TX_QUEUE_OVERRIDE(mode) \
|
|
|
|
|
(((mode) == BOND_MODE_ACTIVEBACKUP) || \
|
|
|
|
|
((mode) == BOND_MODE_ROUNDROBIN))
|
2013-10-18 23:43:34 +08:00
|
|
|
|
|
|
|
|
#define BOND_MODE_IS_LB(mode) \
|
|
|
|
|
(((mode) == BOND_MODE_TLB) || \
|
|
|
|
|
((mode) == BOND_MODE_ALB))
|
|
|
|
|
|
2013-11-15 23:34:30 +08:00
|
|
|
#define IS_IP_TARGET_UNUSABLE_ADDRESS(a) \
|
|
|
|
|
((htonl(INADDR_BROADCAST) == a) || \
|
|
|
|
|
ipv4_is_zeronet(a))
|
2005-04-17 06:20:36 +08:00
|
|
|
/*
|
|
|
|
|
* Less bad way to call ioctl from within the kernel; this needs to be
|
|
|
|
|
* done some other way to get the call out of interrupt context.
|
|
|
|
|
* Needs "ioctl" variable to be supplied by calling context.
|
|
|
|
|
*/
|
|
|
|
|
#define IOCTL(dev, arg, cmd) ({ \
|
|
|
|
|
int res = 0; \
|
|
|
|
|
mm_segment_t fs = get_fs(); \
|
|
|
|
|
set_fs(get_ds()); \
|
|
|
|
|
res = ioctl(dev, arg, cmd); \
|
|
|
|
|
set_fs(fs); \
|
|
|
|
|
res; })
|
|
|
|
|
|
2013-08-01 22:54:47 +08:00
|
|
|
/* slave list primitives */
|
2013-09-25 15:20:24 +08:00
|
|
|
#define bond_slave_list(bond) (&(bond)->dev->adj_list.lower)
|
|
|
|
|
|
|
|
|
|
#define bond_has_slaves(bond) !list_empty(bond_slave_list(bond))
|
2013-09-25 15:20:21 +08:00
|
|
|
|
2013-08-01 22:54:47 +08:00
|
|
|
/* IMPORTANT: bond_first/last_slave can return NULL in case of an empty list */
|
|
|
|
|
#define bond_first_slave(bond) \
|
2013-09-25 15:20:24 +08:00
|
|
|
(bond_has_slaves(bond) ? \
|
|
|
|
|
netdev_adjacent_get_private(bond_slave_list(bond)->next) : \
|
|
|
|
|
NULL)
|
2013-08-01 22:54:47 +08:00
|
|
|
#define bond_last_slave(bond) \
|
2013-09-25 15:20:24 +08:00
|
|
|
(bond_has_slaves(bond) ? \
|
|
|
|
|
netdev_adjacent_get_private(bond_slave_list(bond)->prev) : \
|
|
|
|
|
NULL)
|
2013-08-01 22:54:47 +08:00
|
|
|
|
2013-12-13 10:19:55 +08:00
|
|
|
/* Caller must have rcu_read_lock */
|
|
|
|
|
#define bond_first_slave_rcu(bond) \
|
|
|
|
|
netdev_lower_get_first_private_rcu(bond->dev)
|
|
|
|
|
|
2013-09-25 15:20:24 +08:00
|
|
|
#define bond_is_first_slave(bond, pos) (pos == bond_first_slave(bond))
|
|
|
|
|
#define bond_is_last_slave(bond, pos) (pos == bond_last_slave(bond))
|
2013-08-01 22:54:47 +08:00
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
/**
|
2013-08-01 22:54:47 +08:00
|
|
|
* bond_for_each_slave - iterate over all slaves
|
|
|
|
|
* @bond: the bond holding this list
|
|
|
|
|
* @pos: current slave
|
2013-09-25 15:20:14 +08:00
|
|
|
* @iter: list_head * iterator
|
2005-04-17 06:20:36 +08:00
|
|
|
*
|
|
|
|
|
* Caller must hold bond->lock
|
|
|
|
|
*/
|
2013-09-25 15:20:14 +08:00
|
|
|
#define bond_for_each_slave(bond, pos, iter) \
|
|
|
|
|
netdev_for_each_lower_private((bond)->dev, pos, iter)
|
2005-04-17 06:20:36 +08:00
|
|
|
|
bonding: initial RCU conversion
This patch does the initial bonding conversion to RCU. After it the
following modes are protected by RCU alone: roundrobin, active-backup,
broadcast and xor. Modes ALB/TLB and 3ad still acquire bond->lock for
reading, and will be dealt with later. curr_active_slave needs to be
dereferenced via rcu in the converted modes because the only thing
protecting the slave after this patch is rcu_read_lock, so we need the
proper barrier for weakly ordered archs and to make sure we don't have
stale pointer. It's not tagged with __rcu yet because there's still work
to be done to remove the curr_slave_lock, so sparse will complain when
rcu_assign_pointer and rcu_dereference are used, but the alternative to use
rcu_dereference_protected would've created much bigger code churn which is
more difficult to test and review. That will be converted in time.
1. Active-backup mode
1.1 Perf recording while doing iperf -P 4
- old bonding: iperf spent 0.55% in bonding, system spent 0.29% CPU
in bonding
- new bonding: iperf spent 0.29% in bonding, system spent 0.15% CPU
in bonding
1.2. Bandwidth measurements
- old bonding: 16.1 gbps consistently
- new bonding: 17.5 gbps consistently
2. Round-robin mode
2.1 Perf recording while doing iperf -P 4
- old bonding: iperf spent 0.51% in bonding, system spent 0.24% CPU
in bonding
- new bonding: iperf spent 0.16% in bonding, system spent 0.11% CPU
in bonding
2.2 Bandwidth measurements
- old bonding: 8 gbps (variable due to packet reorderings)
- new bonding: 10 gbps (variable due to packet reorderings)
Of course the latency has improved in all converted modes, and moreover
while
doing enslave/release (since it doesn't affect tx anymore).
Also I've stress tested all modes doing enslave/release in a loop while
transmitting traffic.
Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-08-01 22:54:51 +08:00
|
|
|
/* Caller must have rcu_read_lock */
|
2013-09-25 15:20:14 +08:00
|
|
|
#define bond_for_each_slave_rcu(bond, pos, iter) \
|
|
|
|
|
netdev_for_each_lower_private_rcu((bond)->dev, pos, iter)
|
bonding: initial RCU conversion
This patch does the initial bonding conversion to RCU. After it the
following modes are protected by RCU alone: roundrobin, active-backup,
broadcast and xor. Modes ALB/TLB and 3ad still acquire bond->lock for
reading, and will be dealt with later. curr_active_slave needs to be
dereferenced via rcu in the converted modes because the only thing
protecting the slave after this patch is rcu_read_lock, so we need the
proper barrier for weakly ordered archs and to make sure we don't have
stale pointer. It's not tagged with __rcu yet because there's still work
to be done to remove the curr_slave_lock, so sparse will complain when
rcu_assign_pointer and rcu_dereference are used, but the alternative to use
rcu_dereference_protected would've created much bigger code churn which is
more difficult to test and review. That will be converted in time.
1. Active-backup mode
1.1 Perf recording while doing iperf -P 4
- old bonding: iperf spent 0.55% in bonding, system spent 0.29% CPU
in bonding
- new bonding: iperf spent 0.29% in bonding, system spent 0.15% CPU
in bonding
1.2. Bandwidth measurements
- old bonding: 16.1 gbps consistently
- new bonding: 17.5 gbps consistently
2. Round-robin mode
2.1 Perf recording while doing iperf -P 4
- old bonding: iperf spent 0.51% in bonding, system spent 0.24% CPU
in bonding
- new bonding: iperf spent 0.16% in bonding, system spent 0.11% CPU
in bonding
2.2 Bandwidth measurements
- old bonding: 8 gbps (variable due to packet reorderings)
- new bonding: 10 gbps (variable due to packet reorderings)
Of course the latency has improved in all converted modes, and moreover
while
doing enslave/release (since it doesn't affect tx anymore).
Also I've stress tested all modes doing enslave/release in a loop while
transmitting traffic.
Signed-off-by: Nikolay Aleksandrov <nikolay@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-08-01 22:54:51 +08:00
|
|
|
|
2010-10-14 00:01:50 +08:00
|
|
|
#ifdef CONFIG_NET_POLL_CONTROLLER
|
net: Convert netpoll blocking api in bonding driver to be a counter
A while back I made some changes to enable netpoll in the bonding driver. Among
them was a per-cpu flag that indicated we were in a path that held locks which
could cause the netpoll path to block in during tx, and as such the tx path
should queue the frame for later use. This appears to have given rise to a
regression. If one of those paths on which we hold the per-cpu flag yields the
cpu, its possible for us to come back on a different cpu, leading to us clearing
a different flag than we set. This results in odd netpoll drops, and BUG
backtraces appearing in the log, as we check to make sure that we only clear set
bits, and only set clear bits. I had though briefly about changing the
offending paths so that they wouldn't sleep, but looking at my origional work
more closely, it doesn't appear that a per-cpu flag is warranted. We alrady
gate the checking of this flag on IFF_IN_NETPOLL, so we don't hit this in the
normal tx case anyway. And practically speaking, the normal use case for
netpoll is to only have one client anyway, so we're not going to erroneously
queue netpoll frames when its actually safe to do so. As such, lets just
convert that per-cpu flag to an atomic counter. It fixes the rescheduling bugs,
is equivalent from a performance perspective and actually eliminates some code
in the process.
Tested by the reporter and myself, successfully
Reported-by: Liang Zheng <lzheng@redhat.com>
CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
CC: David S. Miller <davem@davemloft.net>
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2010-12-06 17:05:50 +08:00
|
|
|
extern atomic_t netpoll_block_tx;
|
2010-10-14 00:01:50 +08:00
|
|
|
|
|
|
|
|
static inline void block_netpoll_tx(void)
|
|
|
|
|
{
|
net: Convert netpoll blocking api in bonding driver to be a counter
A while back I made some changes to enable netpoll in the bonding driver. Among
them was a per-cpu flag that indicated we were in a path that held locks which
could cause the netpoll path to block in during tx, and as such the tx path
should queue the frame for later use. This appears to have given rise to a
regression. If one of those paths on which we hold the per-cpu flag yields the
cpu, its possible for us to come back on a different cpu, leading to us clearing
a different flag than we set. This results in odd netpoll drops, and BUG
backtraces appearing in the log, as we check to make sure that we only clear set
bits, and only set clear bits. I had though briefly about changing the
offending paths so that they wouldn't sleep, but looking at my origional work
more closely, it doesn't appear that a per-cpu flag is warranted. We alrady
gate the checking of this flag on IFF_IN_NETPOLL, so we don't hit this in the
normal tx case anyway. And practically speaking, the normal use case for
netpoll is to only have one client anyway, so we're not going to erroneously
queue netpoll frames when its actually safe to do so. As such, lets just
convert that per-cpu flag to an atomic counter. It fixes the rescheduling bugs,
is equivalent from a performance perspective and actually eliminates some code
in the process.
Tested by the reporter and myself, successfully
Reported-by: Liang Zheng <lzheng@redhat.com>
CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
CC: David S. Miller <davem@davemloft.net>
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2010-12-06 17:05:50 +08:00
|
|
|
atomic_inc(&netpoll_block_tx);
|
2010-10-14 00:01:50 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static inline void unblock_netpoll_tx(void)
|
|
|
|
|
{
|
net: Convert netpoll blocking api in bonding driver to be a counter
A while back I made some changes to enable netpoll in the bonding driver. Among
them was a per-cpu flag that indicated we were in a path that held locks which
could cause the netpoll path to block in during tx, and as such the tx path
should queue the frame for later use. This appears to have given rise to a
regression. If one of those paths on which we hold the per-cpu flag yields the
cpu, its possible for us to come back on a different cpu, leading to us clearing
a different flag than we set. This results in odd netpoll drops, and BUG
backtraces appearing in the log, as we check to make sure that we only clear set
bits, and only set clear bits. I had though briefly about changing the
offending paths so that they wouldn't sleep, but looking at my origional work
more closely, it doesn't appear that a per-cpu flag is warranted. We alrady
gate the checking of this flag on IFF_IN_NETPOLL, so we don't hit this in the
normal tx case anyway. And practically speaking, the normal use case for
netpoll is to only have one client anyway, so we're not going to erroneously
queue netpoll frames when its actually safe to do so. As such, lets just
convert that per-cpu flag to an atomic counter. It fixes the rescheduling bugs,
is equivalent from a performance perspective and actually eliminates some code
in the process.
Tested by the reporter and myself, successfully
Reported-by: Liang Zheng <lzheng@redhat.com>
CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
CC: David S. Miller <davem@davemloft.net>
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2010-12-06 17:05:50 +08:00
|
|
|
atomic_dec(&netpoll_block_tx);
|
2010-10-14 00:01:50 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static inline int is_netpoll_tx_blocked(struct net_device *dev)
|
|
|
|
|
{
|
2011-02-18 07:43:33 +08:00
|
|
|
if (unlikely(netpoll_tx_running(dev)))
|
net: Convert netpoll blocking api in bonding driver to be a counter
A while back I made some changes to enable netpoll in the bonding driver. Among
them was a per-cpu flag that indicated we were in a path that held locks which
could cause the netpoll path to block in during tx, and as such the tx path
should queue the frame for later use. This appears to have given rise to a
regression. If one of those paths on which we hold the per-cpu flag yields the
cpu, its possible for us to come back on a different cpu, leading to us clearing
a different flag than we set. This results in odd netpoll drops, and BUG
backtraces appearing in the log, as we check to make sure that we only clear set
bits, and only set clear bits. I had though briefly about changing the
offending paths so that they wouldn't sleep, but looking at my origional work
more closely, it doesn't appear that a per-cpu flag is warranted. We alrady
gate the checking of this flag on IFF_IN_NETPOLL, so we don't hit this in the
normal tx case anyway. And practically speaking, the normal use case for
netpoll is to only have one client anyway, so we're not going to erroneously
queue netpoll frames when its actually safe to do so. As such, lets just
convert that per-cpu flag to an atomic counter. It fixes the rescheduling bugs,
is equivalent from a performance perspective and actually eliminates some code
in the process.
Tested by the reporter and myself, successfully
Reported-by: Liang Zheng <lzheng@redhat.com>
CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
CC: David S. Miller <davem@davemloft.net>
Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2010-12-06 17:05:50 +08:00
|
|
|
return atomic_read(&netpoll_block_tx);
|
2010-10-14 00:01:50 +08:00
|
|
|
return 0;
|
|
|
|
|
}
|
|
|
|
|
#else
|
|
|
|
|
#define block_netpoll_tx()
|
|
|
|
|
#define unblock_netpoll_tx()
|
|
|
|
|
#define is_netpoll_tx_blocked(dev) (0)
|
|
|
|
|
#endif
|
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
struct bond_params {
|
|
|
|
|
int mode;
|
2005-06-27 05:54:11 +08:00
|
|
|
int xmit_policy;
|
2005-04-17 06:20:36 +08:00
|
|
|
int miimon;
|
2011-04-26 23:25:52 +08:00
|
|
|
u8 num_peer_notif;
|
2005-04-17 06:20:36 +08:00
|
|
|
int arp_interval;
|
2006-09-23 12:54:53 +08:00
|
|
|
int arp_validate;
|
bonding: add an option to fail when any of arp_ip_target is inaccessible
Currently, we fail only when all of the ips in arp_ip_target are gone.
However, in some situations we might need to fail if even one host from
arp_ip_target becomes unavailable.
All situations, obviously, rely on the idea that we need *completely*
functional network, with all interfaces/addresses working correctly.
One real world example might be:
vlans on top on bond (hybrid port). If bond and vlans have ips assigned
and we have their peers monitored via arp_ip_target - in case of switch
misconfiguration (trunk/access port), slave driver malfunction or
tagged/untagged traffic dropped on the way - we will be able to switch
to another slave.
Though any other configuration needs that if we need to have access to all
arp_ip_targets.
This patch adds this possibility by adding a new parameter -
arp_all_targets (both as a module parameter and as a sysfs knob). It can be
set to:
0 or any (the default) - which works exactly as it's working now -
the slave is up if any of the arp_ip_targets are up.
1 or all - the slave is up if all of the arp_ip_targets are up.
This parameter can be changed on the fly (via sysfs), and requires the mode
to be active-backup and arp_validate to be enabled (it obeys the
arp_validate config on which slaves to validate).
Internally it's done through:
1) Add target_last_arp_rx[BOND_MAX_ARP_TARGETS] array to slave struct. It's
an array of jiffies, meaning that slave->target_last_arp_rx[i] is the
last time we've received arp from bond->params.arp_targets[i] on this
slave.
2) If we successfully validate an arp from bond->params.arp_targets[i] in
bond_validate_arp() - update the slave->target_last_arp_rx[i] with the
current jiffies value.
3) When getting slave's last_rx via slave_last_rx(), we return the oldest
time when we've received an arp from any address in
bond->params.arp_targets[].
If the value of arp_all_targets == 0 - we still work the same way as
before.
Also, update the documentation to reflect the new parameter.
v3->v4:
Kill the forgotten rtnl_unlock(), rephrase the documentation part to be
more clear, don't fail setting arp_all_targets if arp_validate is not set -
it has no effect anyway but can be easier to set up. Also, print a warning
if the last arp_ip_target is removed while the arp_interval is on, but not
the arp_validate.
v2->v3:
Use _bh spinlock, remove useless rtnl_lock() and use jiffies for new
arp_ip_target last arp, instead of slave_last_rx(). On bond_enslave(),
use the same initialization value for target_last_arp_rx[] as is used
for the default last_arp_rx, to avoid useless interface flaps.
Also, instead of failing to remove the last arp_ip_target just print a
warning - otherwise it might break existing scripts.
v1->v2:
Correctly handle adding/removing hosts in arp_ip_target - we need to
shift/initialize all slave's target_last_arp_rx. Also, don't fail module
loading on arp_all_targets misconfiguration, just disable it, and some
minor style fixes.
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-06-24 17:49:34 +08:00
|
|
|
int arp_all_targets;
|
2005-04-17 06:20:36 +08:00
|
|
|
int use_carrier;
|
2007-10-10 10:57:24 +08:00
|
|
|
int fail_over_mac;
|
2005-04-17 06:20:36 +08:00
|
|
|
int updelay;
|
|
|
|
|
int downdelay;
|
|
|
|
|
int lacp_fast;
|
2011-06-22 17:54:39 +08:00
|
|
|
unsigned int min_links;
|
2008-11-05 09:51:16 +08:00
|
|
|
int ad_select;
|
2005-04-17 06:20:36 +08:00
|
|
|
char primary[IFNAMSIZ];
|
2009-09-25 11:28:09 +08:00
|
|
|
int primary_reselect;
|
2007-08-23 08:06:58 +08:00
|
|
|
__be32 arp_targets[BOND_MAX_ARP_TARGETS];
|
2010-06-02 16:40:18 +08:00
|
|
|
int tx_queues;
|
2010-06-02 16:39:21 +08:00
|
|
|
int all_slaves_active;
|
2010-10-05 22:23:59 +08:00
|
|
|
int resend_igmp;
|
2013-09-13 23:05:33 +08:00
|
|
|
int lp_interval;
|
2013-11-05 20:51:41 +08:00
|
|
|
int packets_per_slave;
|
reciprocal_divide: update/correction of the algorithm
Jakub Zawadzki noticed that some divisions by reciprocal_divide()
were not correct [1][2], which he could also show with BPF code
after divisions are transformed into reciprocal_value() for runtime
invariance which can be passed to reciprocal_divide() later on;
reverse in BPF dump ended up with a different, off-by-one K in
some situations.
This has been fixed by Eric Dumazet in commit aee636c4809fa5
("bpf: do not use reciprocal divide"). This follow-up patch
improves reciprocal_value() and reciprocal_divide() to work in
all cases by using Granlund and Montgomery method, so that also
future use is safe and without any non-obvious side-effects.
Known problems with the old implementation were that division by 1
always returned 0 and some off-by-ones when the dividend and divisor
where very large. This seemed to not be problematic with its
current users, as far as we can tell. Eric Dumazet checked for
the slab usage, we cannot surely say so in the case of flex_array.
Still, in order to fix that, we propose an extension from the
original implementation from commit 6a2d7a955d8d resp. [3][4],
by using the algorithm proposed in "Division by Invariant Integers
Using Multiplication" [5], Torbjörn Granlund and Peter L.
Montgomery, that is, pseudocode for q = n/d where q, n, d is in
u32 universe:
1) Initialization:
int l = ceil(log_2 d)
uword m' = floor((1<<32)*((1<<l)-d)/d)+1
int sh_1 = min(l,1)
int sh_2 = max(l-1,0)
2) For q = n/d, all uword:
uword t = (n*m')>>32
q = (t+((n-t)>>sh_1))>>sh_2
The assembler implementation from Agner Fog [6] also helped a lot
while implementing. We have tested the implementation on x86_64,
ppc64, i686, s390x; on x86_64/haswell we're still half the latency
compared to normal divide.
Joint work with Daniel Borkmann.
[1] http://www.wireshark.org/~darkjames/reciprocal-buggy.c
[2] http://www.wireshark.org/~darkjames/set-and-dump-filter-k-bug.c
[3] https://gmplib.org/~tege/division-paper.pdf
[4] http://homepage.cs.uiowa.edu/~jones/bcd/divide.html
[5] http://citeseerx.ist.psu.edu/viewdoc/summary?doi=10.1.1.1.2556
[6] http://www.agner.org/optimize/asmlib.zip
Reported-by: Jakub Zawadzki <darkjames-ws@darkjames.pl>
Cc: Eric Dumazet <eric.dumazet@gmail.com>
Cc: Austin S Hemmelgarn <ahferroin7@gmail.com>
Cc: linux-kernel@vger.kernel.org
Cc: Jesse Gross <jesse@nicira.com>
Cc: Jamal Hadi Salim <jhs@mojatatu.com>
Cc: Stephen Hemminger <stephen@networkplumber.org>
Cc: Matt Mackall <mpm@selenic.com>
Cc: Pekka Enberg <penberg@kernel.org>
Cc: Christoph Lameter <cl@linux-foundation.org>
Cc: Andy Gospodarek <andy@greyhouse.net>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Jakub Zawadzki <darkjames-ws@darkjames.pl>
Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-01-22 09:29:41 +08:00
|
|
|
struct reciprocal_value reciprocal_packets_per_slave;
|
2005-04-17 06:20:36 +08:00
|
|
|
};
|
|
|
|
|
|
2005-11-10 02:35:44 +08:00
|
|
|
struct bond_parm_tbl {
|
|
|
|
|
char *modename;
|
|
|
|
|
int mode;
|
|
|
|
|
};
|
|
|
|
|
|
2008-01-18 08:25:01 +08:00
|
|
|
#define BOND_MAX_MODENAME_LEN 20
|
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
struct slave {
|
2005-11-10 02:36:50 +08:00
|
|
|
struct net_device *dev; /* first - useful for panic debug */
|
2011-03-22 10:38:12 +08:00
|
|
|
struct bonding *bond; /* our master */
|
2006-09-23 12:52:51 +08:00
|
|
|
int delay;
|
2014-02-18 14:48:46 +08:00
|
|
|
/* all three in jiffies */
|
|
|
|
|
unsigned long last_link_up;
|
2014-02-18 14:48:47 +08:00
|
|
|
unsigned long last_rx;
|
bonding: add an option to fail when any of arp_ip_target is inaccessible
Currently, we fail only when all of the ips in arp_ip_target are gone.
However, in some situations we might need to fail if even one host from
arp_ip_target becomes unavailable.
All situations, obviously, rely on the idea that we need *completely*
functional network, with all interfaces/addresses working correctly.
One real world example might be:
vlans on top on bond (hybrid port). If bond and vlans have ips assigned
and we have their peers monitored via arp_ip_target - in case of switch
misconfiguration (trunk/access port), slave driver malfunction or
tagged/untagged traffic dropped on the way - we will be able to switch
to another slave.
Though any other configuration needs that if we need to have access to all
arp_ip_targets.
This patch adds this possibility by adding a new parameter -
arp_all_targets (both as a module parameter and as a sysfs knob). It can be
set to:
0 or any (the default) - which works exactly as it's working now -
the slave is up if any of the arp_ip_targets are up.
1 or all - the slave is up if all of the arp_ip_targets are up.
This parameter can be changed on the fly (via sysfs), and requires the mode
to be active-backup and arp_validate to be enabled (it obeys the
arp_validate config on which slaves to validate).
Internally it's done through:
1) Add target_last_arp_rx[BOND_MAX_ARP_TARGETS] array to slave struct. It's
an array of jiffies, meaning that slave->target_last_arp_rx[i] is the
last time we've received arp from bond->params.arp_targets[i] on this
slave.
2) If we successfully validate an arp from bond->params.arp_targets[i] in
bond_validate_arp() - update the slave->target_last_arp_rx[i] with the
current jiffies value.
3) When getting slave's last_rx via slave_last_rx(), we return the oldest
time when we've received an arp from any address in
bond->params.arp_targets[].
If the value of arp_all_targets == 0 - we still work the same way as
before.
Also, update the documentation to reflect the new parameter.
v3->v4:
Kill the forgotten rtnl_unlock(), rephrase the documentation part to be
more clear, don't fail setting arp_all_targets if arp_validate is not set -
it has no effect anyway but can be easier to set up. Also, print a warning
if the last arp_ip_target is removed while the arp_interval is on, but not
the arp_validate.
v2->v3:
Use _bh spinlock, remove useless rtnl_lock() and use jiffies for new
arp_ip_target last arp, instead of slave_last_rx(). On bond_enslave(),
use the same initialization value for target_last_arp_rx[] as is used
for the default last_arp_rx, to avoid useless interface flaps.
Also, instead of failing to remove the last arp_ip_target just print a
warning - otherwise it might break existing scripts.
v1->v2:
Correctly handle adding/removing hosts in arp_ip_target - we need to
shift/initialize all slave's target_last_arp_rx. Also, don't fail module
loading on arp_all_targets misconfiguration, just disable it, and some
minor style fixes.
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-06-24 17:49:34 +08:00
|
|
|
unsigned long target_last_arp_rx[BOND_MAX_ARP_TARGETS];
|
2005-04-17 06:20:36 +08:00
|
|
|
s8 link; /* one of BOND_LINK_XXXX */
|
2008-05-18 12:10:13 +08:00
|
|
|
s8 new_link;
|
2011-03-16 16:46:43 +08:00
|
|
|
u8 backup:1, /* indicates backup slave. Value corresponds with
|
|
|
|
|
BOND_STATE_ACTIVE and BOND_STATE_BACKUP */
|
bonding: Fix RTNL: assertion failed at net/core/rtnetlink.c for 802.3ad mode
The problem was introduced by the commit 1d3ee88ae0d
(bonding: add netlink attributes to slave link dev).
The bond_set_active_slave() and bond_set_backup_slave()
will use rtmsg_ifinfo to send slave's states, so these
two functions should be called in RTNL.
In 802.3ad mode, acquiring RTNL for the __enable_port and
__disable_port cases is difficult, as those calls generally
already hold the state machine lock, and cannot unconditionally
call rtnl_lock because either they already hold RTNL (for calls
via bond_3ad_unbind_slave) or due to the potential for deadlock
with bond_3ad_adapter_speed_changed, bond_3ad_adapter_duplex_changed,
bond_3ad_link_change, or bond_3ad_update_lacp_rate. All four of
those are called with RTNL held, and acquire the state machine lock
second. The calling contexts for __enable_port and __disable_port
already hold the state machine lock, and may or may not need RTNL.
According to the Jay's opinion, I don't think it is a problem that
the slave don't send notify message synchronously when the status
changed, normally the state machine is running every 100 ms, send
the notify message at the end of the state machine if the slave's
state changed should be better.
I fix the problem through these steps:
1). add a new function bond_set_slave_state() which could change
the slave's state and call rtmsg_ifinfo() according to the input
parameters called notify.
2). Add a new slave parameter which called should_notify, if the slave's state
changed and don't notify yet, the parameter will be set to 1, and then if
the slave's state changed again, the param will be set to 0, it indicate that
the slave's state has been restored, no need to notify any one.
3). the __enable_port and __disable_port should not call rtmsg_ifinfo
in the state machine lock, any change in the state of slave could
set a flag in the slave, it will indicated that an rtmsg_ifinfo
should be called at the end of the state machine.
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-02-26 11:05:22 +08:00
|
|
|
inactive:1, /* indicates inactive slave */
|
|
|
|
|
should_notify:1; /* indicateds whether the state changed */
|
2011-04-13 23:22:31 +08:00
|
|
|
u8 duplex;
|
2007-10-10 10:43:41 +08:00
|
|
|
u32 original_mtu;
|
2005-04-17 06:20:36 +08:00
|
|
|
u32 link_failure_count;
|
2011-04-13 23:22:31 +08:00
|
|
|
u32 speed;
|
2010-06-02 16:40:18 +08:00
|
|
|
u16 queue_id;
|
2011-04-13 23:22:31 +08:00
|
|
|
u8 perm_hwaddr[ETH_ALEN];
|
2005-04-17 06:20:36 +08:00
|
|
|
struct ad_slave_info ad_info; /* HUGE - better to dynamically alloc */
|
|
|
|
|
struct tlb_slave_info tlb_info;
|
2011-02-18 07:43:32 +08:00
|
|
|
#ifdef CONFIG_NET_POLL_CONTROLLER
|
|
|
|
|
struct netpoll *np;
|
|
|
|
|
#endif
|
2014-01-17 14:57:49 +08:00
|
|
|
struct kobject kobj;
|
2005-04-17 06:20:36 +08:00
|
|
|
};
|
|
|
|
|
|
2008-05-18 12:10:13 +08:00
|
|
|
/*
|
|
|
|
|
* Link pseudo-state only used internally by monitors
|
|
|
|
|
*/
|
|
|
|
|
#define BOND_LINK_NOCHANGE -1
|
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
/*
|
|
|
|
|
* Here are the locking policies for the two bonding locks:
|
|
|
|
|
*
|
|
|
|
|
* 1) Get bond->lock when reading/writing slave list.
|
|
|
|
|
* 2) Get bond->curr_slave_lock when reading/writing bond->curr_active_slave.
|
|
|
|
|
* (It is unnecessary when the write-lock is put with bond->lock.)
|
|
|
|
|
* 3) When we lock with bond->curr_slave_lock, we must lock with bond->lock
|
|
|
|
|
* beforehand.
|
|
|
|
|
*/
|
|
|
|
|
struct bonding {
|
2005-11-10 02:36:50 +08:00
|
|
|
struct net_device *dev; /* first - useful for panic debug */
|
2005-04-17 06:20:36 +08:00
|
|
|
struct slave *curr_active_slave;
|
|
|
|
|
struct slave *current_arp_slave;
|
|
|
|
|
struct slave *primary_slave;
|
2009-09-25 11:28:09 +08:00
|
|
|
bool force_primary;
|
2005-04-17 06:20:36 +08:00
|
|
|
s32 slave_cnt; /* never change this value outside the attach/detach wrappers */
|
2012-06-12 03:23:07 +08:00
|
|
|
int (*recv_probe)(const struct sk_buff *, struct bonding *,
|
|
|
|
|
struct slave *);
|
2005-04-17 06:20:36 +08:00
|
|
|
rwlock_t lock;
|
|
|
|
|
rwlock_t curr_slave_lock;
|
2011-04-26 23:25:52 +08:00
|
|
|
u8 send_peer_notif;
|
2013-06-12 06:07:02 +08:00
|
|
|
u8 igmp_retrans;
|
2005-04-17 06:20:36 +08:00
|
|
|
#ifdef CONFIG_PROC_FS
|
|
|
|
|
struct proc_dir_entry *proc_entry;
|
|
|
|
|
char proc_file_name[IFNAMSIZ];
|
|
|
|
|
#endif /* CONFIG_PROC_FS */
|
|
|
|
|
struct list_head bond_list;
|
2013-11-05 20:51:41 +08:00
|
|
|
u32 rr_tx_counter;
|
2005-04-17 06:20:36 +08:00
|
|
|
struct ad_bond_info ad_info;
|
|
|
|
|
struct alb_bond_info alb_info;
|
|
|
|
|
struct bond_params params;
|
2007-10-18 08:37:45 +08:00
|
|
|
struct workqueue_struct *wq;
|
|
|
|
|
struct delayed_work mii_work;
|
|
|
|
|
struct delayed_work arp_work;
|
|
|
|
|
struct delayed_work alb_work;
|
|
|
|
|
struct delayed_work ad_work;
|
2010-10-05 22:23:57 +08:00
|
|
|
struct delayed_work mcast_work;
|
2010-12-09 23:17:13 +08:00
|
|
|
#ifdef CONFIG_DEBUG_FS
|
2012-08-22 18:11:26 +08:00
|
|
|
/* debugging support via debugfs */
|
2010-12-09 23:17:13 +08:00
|
|
|
struct dentry *debug_dir;
|
|
|
|
|
#endif /* CONFIG_DEBUG_FS */
|
2005-04-17 06:20:36 +08:00
|
|
|
};
|
|
|
|
|
|
2011-03-12 11:14:35 +08:00
|
|
|
#define bond_slave_get_rcu(dev) \
|
|
|
|
|
((struct slave *) rcu_dereference(dev->rx_handler_data))
|
|
|
|
|
|
2013-01-04 06:49:01 +08:00
|
|
|
#define bond_slave_get_rtnl(dev) \
|
|
|
|
|
((struct slave *) rtnl_dereference(dev->rx_handler_data))
|
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
/**
|
|
|
|
|
* Returns NULL if the net_device does not belong to any of the bond's slaves
|
|
|
|
|
*
|
|
|
|
|
* Caller must hold bond lock for read
|
|
|
|
|
*/
|
2011-02-23 15:40:33 +08:00
|
|
|
static inline struct slave *bond_get_slave_by_dev(struct bonding *bond,
|
|
|
|
|
struct net_device *slave_dev)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2013-09-25 15:20:11 +08:00
|
|
|
return netdev_lower_dev_get_private(bond->dev, slave_dev);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
|
2006-01-06 14:45:42 +08:00
|
|
|
static inline struct bonding *bond_get_bond_by_slave(struct slave *slave)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2013-01-04 06:49:01 +08:00
|
|
|
if (!slave || !slave->bond)
|
2005-04-17 06:20:36 +08:00
|
|
|
return NULL;
|
2013-01-04 06:49:01 +08:00
|
|
|
return slave->bond;
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
|
2008-12-10 15:07:13 +08:00
|
|
|
static inline bool bond_is_lb(const struct bonding *bond)
|
|
|
|
|
{
|
2013-10-18 23:43:34 +08:00
|
|
|
return BOND_MODE_IS_LB(bond->params.mode);
|
2008-12-10 15:07:13 +08:00
|
|
|
}
|
|
|
|
|
|
2011-03-12 11:14:37 +08:00
|
|
|
static inline void bond_set_active_slave(struct slave *slave)
|
|
|
|
|
{
|
2014-01-17 14:57:56 +08:00
|
|
|
if (slave->backup) {
|
|
|
|
|
slave->backup = 0;
|
2014-03-06 22:33:22 +08:00
|
|
|
rtmsg_ifinfo(RTM_NEWLINK, slave->dev, 0, GFP_ATOMIC);
|
2014-01-17 14:57:56 +08:00
|
|
|
}
|
2011-03-12 11:14:37 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static inline void bond_set_backup_slave(struct slave *slave)
|
|
|
|
|
{
|
2014-01-17 14:57:56 +08:00
|
|
|
if (!slave->backup) {
|
|
|
|
|
slave->backup = 1;
|
2014-03-06 22:33:22 +08:00
|
|
|
rtmsg_ifinfo(RTM_NEWLINK, slave->dev, 0, GFP_ATOMIC);
|
2014-01-17 14:57:56 +08:00
|
|
|
}
|
2011-03-12 11:14:37 +08:00
|
|
|
}
|
|
|
|
|
|
bonding: Fix RTNL: assertion failed at net/core/rtnetlink.c for 802.3ad mode
The problem was introduced by the commit 1d3ee88ae0d
(bonding: add netlink attributes to slave link dev).
The bond_set_active_slave() and bond_set_backup_slave()
will use rtmsg_ifinfo to send slave's states, so these
two functions should be called in RTNL.
In 802.3ad mode, acquiring RTNL for the __enable_port and
__disable_port cases is difficult, as those calls generally
already hold the state machine lock, and cannot unconditionally
call rtnl_lock because either they already hold RTNL (for calls
via bond_3ad_unbind_slave) or due to the potential for deadlock
with bond_3ad_adapter_speed_changed, bond_3ad_adapter_duplex_changed,
bond_3ad_link_change, or bond_3ad_update_lacp_rate. All four of
those are called with RTNL held, and acquire the state machine lock
second. The calling contexts for __enable_port and __disable_port
already hold the state machine lock, and may or may not need RTNL.
According to the Jay's opinion, I don't think it is a problem that
the slave don't send notify message synchronously when the status
changed, normally the state machine is running every 100 ms, send
the notify message at the end of the state machine if the slave's
state changed should be better.
I fix the problem through these steps:
1). add a new function bond_set_slave_state() which could change
the slave's state and call rtmsg_ifinfo() according to the input
parameters called notify.
2). Add a new slave parameter which called should_notify, if the slave's state
changed and don't notify yet, the parameter will be set to 1, and then if
the slave's state changed again, the param will be set to 0, it indicate that
the slave's state has been restored, no need to notify any one.
3). the __enable_port and __disable_port should not call rtmsg_ifinfo
in the state machine lock, any change in the state of slave could
set a flag in the slave, it will indicated that an rtmsg_ifinfo
should be called at the end of the state machine.
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-02-26 11:05:22 +08:00
|
|
|
static inline void bond_set_slave_state(struct slave *slave,
|
|
|
|
|
int slave_state, bool notify)
|
|
|
|
|
{
|
|
|
|
|
if (slave->backup == slave_state)
|
|
|
|
|
return;
|
|
|
|
|
|
|
|
|
|
slave->backup = slave_state;
|
|
|
|
|
if (notify) {
|
2014-03-06 22:33:22 +08:00
|
|
|
rtmsg_ifinfo(RTM_NEWLINK, slave->dev, 0, GFP_ATOMIC);
|
bonding: Fix RTNL: assertion failed at net/core/rtnetlink.c for 802.3ad mode
The problem was introduced by the commit 1d3ee88ae0d
(bonding: add netlink attributes to slave link dev).
The bond_set_active_slave() and bond_set_backup_slave()
will use rtmsg_ifinfo to send slave's states, so these
two functions should be called in RTNL.
In 802.3ad mode, acquiring RTNL for the __enable_port and
__disable_port cases is difficult, as those calls generally
already hold the state machine lock, and cannot unconditionally
call rtnl_lock because either they already hold RTNL (for calls
via bond_3ad_unbind_slave) or due to the potential for deadlock
with bond_3ad_adapter_speed_changed, bond_3ad_adapter_duplex_changed,
bond_3ad_link_change, or bond_3ad_update_lacp_rate. All four of
those are called with RTNL held, and acquire the state machine lock
second. The calling contexts for __enable_port and __disable_port
already hold the state machine lock, and may or may not need RTNL.
According to the Jay's opinion, I don't think it is a problem that
the slave don't send notify message synchronously when the status
changed, normally the state machine is running every 100 ms, send
the notify message at the end of the state machine if the slave's
state changed should be better.
I fix the problem through these steps:
1). add a new function bond_set_slave_state() which could change
the slave's state and call rtmsg_ifinfo() according to the input
parameters called notify.
2). Add a new slave parameter which called should_notify, if the slave's state
changed and don't notify yet, the parameter will be set to 1, and then if
the slave's state changed again, the param will be set to 0, it indicate that
the slave's state has been restored, no need to notify any one.
3). the __enable_port and __disable_port should not call rtmsg_ifinfo
in the state machine lock, any change in the state of slave could
set a flag in the slave, it will indicated that an rtmsg_ifinfo
should be called at the end of the state machine.
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-02-26 11:05:22 +08:00
|
|
|
slave->should_notify = 0;
|
|
|
|
|
} else {
|
|
|
|
|
if (slave->should_notify)
|
|
|
|
|
slave->should_notify = 0;
|
|
|
|
|
else
|
|
|
|
|
slave->should_notify = 1;
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2014-01-28 11:48:53 +08:00
|
|
|
static inline void bond_slave_state_change(struct bonding *bond)
|
|
|
|
|
{
|
|
|
|
|
struct list_head *iter;
|
|
|
|
|
struct slave *tmp;
|
|
|
|
|
|
|
|
|
|
bond_for_each_slave(bond, tmp, iter) {
|
|
|
|
|
if (tmp->link == BOND_LINK_UP)
|
|
|
|
|
bond_set_active_slave(tmp);
|
|
|
|
|
else if (tmp->link == BOND_LINK_DOWN)
|
|
|
|
|
bond_set_backup_slave(tmp);
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2014-02-26 11:05:23 +08:00
|
|
|
static inline void bond_slave_state_notify(struct bonding *bond)
|
|
|
|
|
{
|
|
|
|
|
struct list_head *iter;
|
|
|
|
|
struct slave *tmp;
|
|
|
|
|
|
|
|
|
|
bond_for_each_slave(bond, tmp, iter) {
|
|
|
|
|
if (tmp->should_notify) {
|
2014-03-06 22:33:22 +08:00
|
|
|
rtmsg_ifinfo(RTM_NEWLINK, tmp->dev, 0, GFP_ATOMIC);
|
2014-02-26 11:05:23 +08:00
|
|
|
tmp->should_notify = 0;
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2011-03-12 11:14:37 +08:00
|
|
|
static inline int bond_slave_state(struct slave *slave)
|
|
|
|
|
{
|
|
|
|
|
return slave->backup;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static inline bool bond_is_active_slave(struct slave *slave)
|
|
|
|
|
{
|
|
|
|
|
return !bond_slave_state(slave);
|
|
|
|
|
}
|
|
|
|
|
|
2009-09-25 11:28:09 +08:00
|
|
|
#define BOND_PRI_RESELECT_ALWAYS 0
|
|
|
|
|
#define BOND_PRI_RESELECT_BETTER 1
|
|
|
|
|
#define BOND_PRI_RESELECT_FAILURE 2
|
|
|
|
|
|
2008-05-18 12:10:14 +08:00
|
|
|
#define BOND_FOM_NONE 0
|
|
|
|
|
#define BOND_FOM_ACTIVE 1
|
|
|
|
|
#define BOND_FOM_FOLLOW 2
|
|
|
|
|
|
bonding: add an option to fail when any of arp_ip_target is inaccessible
Currently, we fail only when all of the ips in arp_ip_target are gone.
However, in some situations we might need to fail if even one host from
arp_ip_target becomes unavailable.
All situations, obviously, rely on the idea that we need *completely*
functional network, with all interfaces/addresses working correctly.
One real world example might be:
vlans on top on bond (hybrid port). If bond and vlans have ips assigned
and we have their peers monitored via arp_ip_target - in case of switch
misconfiguration (trunk/access port), slave driver malfunction or
tagged/untagged traffic dropped on the way - we will be able to switch
to another slave.
Though any other configuration needs that if we need to have access to all
arp_ip_targets.
This patch adds this possibility by adding a new parameter -
arp_all_targets (both as a module parameter and as a sysfs knob). It can be
set to:
0 or any (the default) - which works exactly as it's working now -
the slave is up if any of the arp_ip_targets are up.
1 or all - the slave is up if all of the arp_ip_targets are up.
This parameter can be changed on the fly (via sysfs), and requires the mode
to be active-backup and arp_validate to be enabled (it obeys the
arp_validate config on which slaves to validate).
Internally it's done through:
1) Add target_last_arp_rx[BOND_MAX_ARP_TARGETS] array to slave struct. It's
an array of jiffies, meaning that slave->target_last_arp_rx[i] is the
last time we've received arp from bond->params.arp_targets[i] on this
slave.
2) If we successfully validate an arp from bond->params.arp_targets[i] in
bond_validate_arp() - update the slave->target_last_arp_rx[i] with the
current jiffies value.
3) When getting slave's last_rx via slave_last_rx(), we return the oldest
time when we've received an arp from any address in
bond->params.arp_targets[].
If the value of arp_all_targets == 0 - we still work the same way as
before.
Also, update the documentation to reflect the new parameter.
v3->v4:
Kill the forgotten rtnl_unlock(), rephrase the documentation part to be
more clear, don't fail setting arp_all_targets if arp_validate is not set -
it has no effect anyway but can be easier to set up. Also, print a warning
if the last arp_ip_target is removed while the arp_interval is on, but not
the arp_validate.
v2->v3:
Use _bh spinlock, remove useless rtnl_lock() and use jiffies for new
arp_ip_target last arp, instead of slave_last_rx(). On bond_enslave(),
use the same initialization value for target_last_arp_rx[] as is used
for the default last_arp_rx, to avoid useless interface flaps.
Also, instead of failing to remove the last arp_ip_target just print a
warning - otherwise it might break existing scripts.
v1->v2:
Correctly handle adding/removing hosts in arp_ip_target - we need to
shift/initialize all slave's target_last_arp_rx. Also, don't fail module
loading on arp_all_targets misconfiguration, just disable it, and some
minor style fixes.
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-06-24 17:49:34 +08:00
|
|
|
#define BOND_ARP_TARGETS_ANY 0
|
|
|
|
|
#define BOND_ARP_TARGETS_ALL 1
|
|
|
|
|
|
2006-09-23 12:54:53 +08:00
|
|
|
#define BOND_ARP_VALIDATE_NONE 0
|
|
|
|
|
#define BOND_ARP_VALIDATE_ACTIVE (1 << BOND_STATE_ACTIVE)
|
|
|
|
|
#define BOND_ARP_VALIDATE_BACKUP (1 << BOND_STATE_BACKUP)
|
|
|
|
|
#define BOND_ARP_VALIDATE_ALL (BOND_ARP_VALIDATE_ACTIVE | \
|
|
|
|
|
BOND_ARP_VALIDATE_BACKUP)
|
2014-02-18 14:48:40 +08:00
|
|
|
#define BOND_ARP_FILTER (BOND_ARP_VALIDATE_ALL + 1)
|
|
|
|
|
#define BOND_ARP_FILTER_ACTIVE (BOND_ARP_VALIDATE_ACTIVE | \
|
|
|
|
|
BOND_ARP_FILTER)
|
|
|
|
|
#define BOND_ARP_FILTER_BACKUP (BOND_ARP_VALIDATE_BACKUP | \
|
|
|
|
|
BOND_ARP_FILTER)
|
2006-09-23 12:54:53 +08:00
|
|
|
|
bonding: Fix RTNL: assertion failed at net/core/rtnetlink.c for 802.3ad mode
The problem was introduced by the commit 1d3ee88ae0d
(bonding: add netlink attributes to slave link dev).
The bond_set_active_slave() and bond_set_backup_slave()
will use rtmsg_ifinfo to send slave's states, so these
two functions should be called in RTNL.
In 802.3ad mode, acquiring RTNL for the __enable_port and
__disable_port cases is difficult, as those calls generally
already hold the state machine lock, and cannot unconditionally
call rtnl_lock because either they already hold RTNL (for calls
via bond_3ad_unbind_slave) or due to the potential for deadlock
with bond_3ad_adapter_speed_changed, bond_3ad_adapter_duplex_changed,
bond_3ad_link_change, or bond_3ad_update_lacp_rate. All four of
those are called with RTNL held, and acquire the state machine lock
second. The calling contexts for __enable_port and __disable_port
already hold the state machine lock, and may or may not need RTNL.
According to the Jay's opinion, I don't think it is a problem that
the slave don't send notify message synchronously when the status
changed, normally the state machine is running every 100 ms, send
the notify message at the end of the state machine if the slave's
state changed should be better.
I fix the problem through these steps:
1). add a new function bond_set_slave_state() which could change
the slave's state and call rtmsg_ifinfo() according to the input
parameters called notify.
2). Add a new slave parameter which called should_notify, if the slave's state
changed and don't notify yet, the parameter will be set to 1, and then if
the slave's state changed again, the param will be set to 0, it indicate that
the slave's state has been restored, no need to notify any one.
3). the __enable_port and __disable_port should not call rtmsg_ifinfo
in the state machine lock, any change in the state of slave could
set a flag in the slave, it will indicated that an rtmsg_ifinfo
should be called at the end of the state machine.
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-02-26 11:05:22 +08:00
|
|
|
#define BOND_SLAVE_NOTIFY_NOW true
|
|
|
|
|
#define BOND_SLAVE_NOTIFY_LATER false
|
|
|
|
|
|
2006-12-13 00:24:39 +08:00
|
|
|
static inline int slave_do_arp_validate(struct bonding *bond,
|
|
|
|
|
struct slave *slave)
|
2006-09-23 12:54:53 +08:00
|
|
|
{
|
2011-03-12 11:14:37 +08:00
|
|
|
return bond->params.arp_validate & (1 << bond_slave_state(slave));
|
2006-09-23 12:54:53 +08:00
|
|
|
}
|
|
|
|
|
|
2014-02-18 14:48:40 +08:00
|
|
|
static inline int slave_do_arp_validate_only(struct bonding *bond,
|
|
|
|
|
struct slave *slave)
|
|
|
|
|
{
|
|
|
|
|
return bond->params.arp_validate & BOND_ARP_FILTER;
|
|
|
|
|
}
|
|
|
|
|
|
bonding: add an option to fail when any of arp_ip_target is inaccessible
Currently, we fail only when all of the ips in arp_ip_target are gone.
However, in some situations we might need to fail if even one host from
arp_ip_target becomes unavailable.
All situations, obviously, rely on the idea that we need *completely*
functional network, with all interfaces/addresses working correctly.
One real world example might be:
vlans on top on bond (hybrid port). If bond and vlans have ips assigned
and we have their peers monitored via arp_ip_target - in case of switch
misconfiguration (trunk/access port), slave driver malfunction or
tagged/untagged traffic dropped on the way - we will be able to switch
to another slave.
Though any other configuration needs that if we need to have access to all
arp_ip_targets.
This patch adds this possibility by adding a new parameter -
arp_all_targets (both as a module parameter and as a sysfs knob). It can be
set to:
0 or any (the default) - which works exactly as it's working now -
the slave is up if any of the arp_ip_targets are up.
1 or all - the slave is up if all of the arp_ip_targets are up.
This parameter can be changed on the fly (via sysfs), and requires the mode
to be active-backup and arp_validate to be enabled (it obeys the
arp_validate config on which slaves to validate).
Internally it's done through:
1) Add target_last_arp_rx[BOND_MAX_ARP_TARGETS] array to slave struct. It's
an array of jiffies, meaning that slave->target_last_arp_rx[i] is the
last time we've received arp from bond->params.arp_targets[i] on this
slave.
2) If we successfully validate an arp from bond->params.arp_targets[i] in
bond_validate_arp() - update the slave->target_last_arp_rx[i] with the
current jiffies value.
3) When getting slave's last_rx via slave_last_rx(), we return the oldest
time when we've received an arp from any address in
bond->params.arp_targets[].
If the value of arp_all_targets == 0 - we still work the same way as
before.
Also, update the documentation to reflect the new parameter.
v3->v4:
Kill the forgotten rtnl_unlock(), rephrase the documentation part to be
more clear, don't fail setting arp_all_targets if arp_validate is not set -
it has no effect anyway but can be easier to set up. Also, print a warning
if the last arp_ip_target is removed while the arp_interval is on, but not
the arp_validate.
v2->v3:
Use _bh spinlock, remove useless rtnl_lock() and use jiffies for new
arp_ip_target last arp, instead of slave_last_rx(). On bond_enslave(),
use the same initialization value for target_last_arp_rx[] as is used
for the default last_arp_rx, to avoid useless interface flaps.
Also, instead of failing to remove the last arp_ip_target just print a
warning - otherwise it might break existing scripts.
v1->v2:
Correctly handle adding/removing hosts in arp_ip_target - we need to
shift/initialize all slave's target_last_arp_rx. Also, don't fail module
loading on arp_all_targets misconfiguration, just disable it, and some
minor style fixes.
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-06-24 17:49:34 +08:00
|
|
|
/* Get the oldest arp which we've received on this slave for bond's
|
|
|
|
|
* arp_targets.
|
|
|
|
|
*/
|
|
|
|
|
static inline unsigned long slave_oldest_target_arp_rx(struct bonding *bond,
|
|
|
|
|
struct slave *slave)
|
|
|
|
|
{
|
|
|
|
|
int i = 1;
|
|
|
|
|
unsigned long ret = slave->target_last_arp_rx[0];
|
|
|
|
|
|
|
|
|
|
for (; (i < BOND_MAX_ARP_TARGETS) && bond->params.arp_targets[i]; i++)
|
|
|
|
|
if (time_before(slave->target_last_arp_rx[i], ret))
|
|
|
|
|
ret = slave->target_last_arp_rx[i];
|
|
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
|
}
|
|
|
|
|
|
2006-12-13 00:24:39 +08:00
|
|
|
static inline unsigned long slave_last_rx(struct bonding *bond,
|
2007-01-30 04:08:38 +08:00
|
|
|
struct slave *slave)
|
2006-09-23 12:54:53 +08:00
|
|
|
{
|
2014-02-18 14:48:43 +08:00
|
|
|
if (bond->params.arp_all_targets == BOND_ARP_TARGETS_ALL)
|
|
|
|
|
return slave_oldest_target_arp_rx(bond, slave);
|
2006-09-23 12:54:53 +08:00
|
|
|
|
2014-02-18 14:48:47 +08:00
|
|
|
return slave->last_rx;
|
2006-09-23 12:54:53 +08:00
|
|
|
}
|
|
|
|
|
|
2011-02-18 07:43:32 +08:00
|
|
|
#ifdef CONFIG_NET_POLL_CONTROLLER
|
|
|
|
|
static inline void bond_netpoll_send_skb(const struct slave *slave,
|
|
|
|
|
struct sk_buff *skb)
|
|
|
|
|
{
|
|
|
|
|
struct netpoll *np = slave->np;
|
|
|
|
|
|
|
|
|
|
if (np)
|
|
|
|
|
netpoll_send_skb(np, skb);
|
|
|
|
|
}
|
|
|
|
|
#else
|
|
|
|
|
static inline void bond_netpoll_send_skb(const struct slave *slave,
|
|
|
|
|
struct sk_buff *skb)
|
|
|
|
|
{
|
|
|
|
|
}
|
|
|
|
|
#endif
|
|
|
|
|
|
bonding: Fix RTNL: assertion failed at net/core/rtnetlink.c for 802.3ad mode
The problem was introduced by the commit 1d3ee88ae0d
(bonding: add netlink attributes to slave link dev).
The bond_set_active_slave() and bond_set_backup_slave()
will use rtmsg_ifinfo to send slave's states, so these
two functions should be called in RTNL.
In 802.3ad mode, acquiring RTNL for the __enable_port and
__disable_port cases is difficult, as those calls generally
already hold the state machine lock, and cannot unconditionally
call rtnl_lock because either they already hold RTNL (for calls
via bond_3ad_unbind_slave) or due to the potential for deadlock
with bond_3ad_adapter_speed_changed, bond_3ad_adapter_duplex_changed,
bond_3ad_link_change, or bond_3ad_update_lacp_rate. All four of
those are called with RTNL held, and acquire the state machine lock
second. The calling contexts for __enable_port and __disable_port
already hold the state machine lock, and may or may not need RTNL.
According to the Jay's opinion, I don't think it is a problem that
the slave don't send notify message synchronously when the status
changed, normally the state machine is running every 100 ms, send
the notify message at the end of the state machine if the slave's
state changed should be better.
I fix the problem through these steps:
1). add a new function bond_set_slave_state() which could change
the slave's state and call rtmsg_ifinfo() according to the input
parameters called notify.
2). Add a new slave parameter which called should_notify, if the slave's state
changed and don't notify yet, the parameter will be set to 1, and then if
the slave's state changed again, the param will be set to 0, it indicate that
the slave's state has been restored, no need to notify any one.
3). the __enable_port and __disable_port should not call rtmsg_ifinfo
in the state machine lock, any change in the state of slave could
set a flag in the slave, it will indicated that an rtmsg_ifinfo
should be called at the end of the state machine.
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-02-26 11:05:22 +08:00
|
|
|
static inline void bond_set_slave_inactive_flags(struct slave *slave,
|
|
|
|
|
bool notify)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2013-01-04 06:49:01 +08:00
|
|
|
if (!bond_is_lb(slave->bond))
|
bonding: Fix RTNL: assertion failed at net/core/rtnetlink.c for 802.3ad mode
The problem was introduced by the commit 1d3ee88ae0d
(bonding: add netlink attributes to slave link dev).
The bond_set_active_slave() and bond_set_backup_slave()
will use rtmsg_ifinfo to send slave's states, so these
two functions should be called in RTNL.
In 802.3ad mode, acquiring RTNL for the __enable_port and
__disable_port cases is difficult, as those calls generally
already hold the state machine lock, and cannot unconditionally
call rtnl_lock because either they already hold RTNL (for calls
via bond_3ad_unbind_slave) or due to the potential for deadlock
with bond_3ad_adapter_speed_changed, bond_3ad_adapter_duplex_changed,
bond_3ad_link_change, or bond_3ad_update_lacp_rate. All four of
those are called with RTNL held, and acquire the state machine lock
second. The calling contexts for __enable_port and __disable_port
already hold the state machine lock, and may or may not need RTNL.
According to the Jay's opinion, I don't think it is a problem that
the slave don't send notify message synchronously when the status
changed, normally the state machine is running every 100 ms, send
the notify message at the end of the state machine if the slave's
state changed should be better.
I fix the problem through these steps:
1). add a new function bond_set_slave_state() which could change
the slave's state and call rtmsg_ifinfo() according to the input
parameters called notify.
2). Add a new slave parameter which called should_notify, if the slave's state
changed and don't notify yet, the parameter will be set to 1, and then if
the slave's state changed again, the param will be set to 0, it indicate that
the slave's state has been restored, no need to notify any one.
3). the __enable_port and __disable_port should not call rtmsg_ifinfo
in the state machine lock, any change in the state of slave could
set a flag in the slave, it will indicated that an rtmsg_ifinfo
should be called at the end of the state machine.
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-02-26 11:05:22 +08:00
|
|
|
bond_set_slave_state(slave, BOND_STATE_BACKUP, notify);
|
2013-01-04 06:49:01 +08:00
|
|
|
if (!slave->bond->params.all_slaves_active)
|
2011-03-16 16:46:43 +08:00
|
|
|
slave->inactive = 1;
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
|
bonding: Fix RTNL: assertion failed at net/core/rtnetlink.c for 802.3ad mode
The problem was introduced by the commit 1d3ee88ae0d
(bonding: add netlink attributes to slave link dev).
The bond_set_active_slave() and bond_set_backup_slave()
will use rtmsg_ifinfo to send slave's states, so these
two functions should be called in RTNL.
In 802.3ad mode, acquiring RTNL for the __enable_port and
__disable_port cases is difficult, as those calls generally
already hold the state machine lock, and cannot unconditionally
call rtnl_lock because either they already hold RTNL (for calls
via bond_3ad_unbind_slave) or due to the potential for deadlock
with bond_3ad_adapter_speed_changed, bond_3ad_adapter_duplex_changed,
bond_3ad_link_change, or bond_3ad_update_lacp_rate. All four of
those are called with RTNL held, and acquire the state machine lock
second. The calling contexts for __enable_port and __disable_port
already hold the state machine lock, and may or may not need RTNL.
According to the Jay's opinion, I don't think it is a problem that
the slave don't send notify message synchronously when the status
changed, normally the state machine is running every 100 ms, send
the notify message at the end of the state machine if the slave's
state changed should be better.
I fix the problem through these steps:
1). add a new function bond_set_slave_state() which could change
the slave's state and call rtmsg_ifinfo() according to the input
parameters called notify.
2). Add a new slave parameter which called should_notify, if the slave's state
changed and don't notify yet, the parameter will be set to 1, and then if
the slave's state changed again, the param will be set to 0, it indicate that
the slave's state has been restored, no need to notify any one.
3). the __enable_port and __disable_port should not call rtmsg_ifinfo
in the state machine lock, any change in the state of slave could
set a flag in the slave, it will indicated that an rtmsg_ifinfo
should be called at the end of the state machine.
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-02-26 11:05:22 +08:00
|
|
|
static inline void bond_set_slave_active_flags(struct slave *slave,
|
|
|
|
|
bool notify)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
bonding: Fix RTNL: assertion failed at net/core/rtnetlink.c for 802.3ad mode
The problem was introduced by the commit 1d3ee88ae0d
(bonding: add netlink attributes to slave link dev).
The bond_set_active_slave() and bond_set_backup_slave()
will use rtmsg_ifinfo to send slave's states, so these
two functions should be called in RTNL.
In 802.3ad mode, acquiring RTNL for the __enable_port and
__disable_port cases is difficult, as those calls generally
already hold the state machine lock, and cannot unconditionally
call rtnl_lock because either they already hold RTNL (for calls
via bond_3ad_unbind_slave) or due to the potential for deadlock
with bond_3ad_adapter_speed_changed, bond_3ad_adapter_duplex_changed,
bond_3ad_link_change, or bond_3ad_update_lacp_rate. All four of
those are called with RTNL held, and acquire the state machine lock
second. The calling contexts for __enable_port and __disable_port
already hold the state machine lock, and may or may not need RTNL.
According to the Jay's opinion, I don't think it is a problem that
the slave don't send notify message synchronously when the status
changed, normally the state machine is running every 100 ms, send
the notify message at the end of the state machine if the slave's
state changed should be better.
I fix the problem through these steps:
1). add a new function bond_set_slave_state() which could change
the slave's state and call rtmsg_ifinfo() according to the input
parameters called notify.
2). Add a new slave parameter which called should_notify, if the slave's state
changed and don't notify yet, the parameter will be set to 1, and then if
the slave's state changed again, the param will be set to 0, it indicate that
the slave's state has been restored, no need to notify any one.
3). the __enable_port and __disable_port should not call rtmsg_ifinfo
in the state machine lock, any change in the state of slave could
set a flag in the slave, it will indicated that an rtmsg_ifinfo
should be called at the end of the state machine.
Cc: Jay Vosburgh <fubar@us.ibm.com>
Cc: Veaceslav Falico <vfalico@redhat.com>
Cc: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2014-02-26 11:05:22 +08:00
|
|
|
bond_set_slave_state(slave, BOND_STATE_ACTIVE, notify);
|
2011-03-16 16:46:43 +08:00
|
|
|
slave->inactive = 0;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static inline bool bond_is_slave_inactive(struct slave *slave)
|
|
|
|
|
{
|
|
|
|
|
return slave->inactive;
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
|
|
|
|
|
2012-03-23 00:14:29 +08:00
|
|
|
static inline __be32 bond_confirm_addr(struct net_device *dev, __be32 dst, __be32 local)
|
|
|
|
|
{
|
|
|
|
|
struct in_device *in_dev;
|
|
|
|
|
__be32 addr = 0;
|
|
|
|
|
|
|
|
|
|
rcu_read_lock();
|
|
|
|
|
in_dev = __in_dev_get_rcu(dev);
|
|
|
|
|
|
|
|
|
|
if (in_dev)
|
2013-12-10 22:02:40 +08:00
|
|
|
addr = inet_confirm_addr(dev_net(dev), in_dev, dst, local,
|
|
|
|
|
RT_SCOPE_HOST);
|
2012-03-23 00:14:29 +08:00
|
|
|
rcu_read_unlock();
|
|
|
|
|
return addr;
|
|
|
|
|
}
|
|
|
|
|
|
2013-08-01 22:54:50 +08:00
|
|
|
static inline bool slave_can_tx(struct slave *slave)
|
|
|
|
|
{
|
|
|
|
|
if (IS_UP(slave->dev) && slave->link == BOND_LINK_UP &&
|
|
|
|
|
bond_is_active_slave(slave))
|
|
|
|
|
return true;
|
|
|
|
|
else
|
|
|
|
|
return false;
|
|
|
|
|
}
|
|
|
|
|
|
2011-10-13 05:56:25 +08:00
|
|
|
struct bond_net;
|
|
|
|
|
|
2013-09-07 06:00:26 +08:00
|
|
|
int bond_arp_rcv(const struct sk_buff *skb, struct bonding *bond, struct slave *slave);
|
2014-01-02 09:13:09 +08:00
|
|
|
void bond_dev_queue_xmit(struct bonding *bond, struct sk_buff *skb, struct net_device *slave_dev);
|
2009-10-29 22:18:26 +08:00
|
|
|
int bond_create(struct net *net, const char *name);
|
2011-10-13 05:56:25 +08:00
|
|
|
int bond_create_sysfs(struct bond_net *net);
|
|
|
|
|
void bond_destroy_sysfs(struct bond_net *net);
|
2009-10-29 22:18:22 +08:00
|
|
|
void bond_prepare_sysfs_group(struct bonding *bond);
|
2014-01-17 14:57:49 +08:00
|
|
|
int bond_sysfs_slave_add(struct slave *slave);
|
|
|
|
|
void bond_sysfs_slave_del(struct slave *slave);
|
2005-11-10 02:35:51 +08:00
|
|
|
int bond_enslave(struct net_device *bond_dev, struct net_device *slave_dev);
|
|
|
|
|
int bond_release(struct net_device *bond_dev, struct net_device *slave_dev);
|
2013-10-02 19:39:25 +08:00
|
|
|
int bond_xmit_hash(struct bonding *bond, struct sk_buff *skb, int count);
|
2005-11-10 02:35:51 +08:00
|
|
|
void bond_select_active_slave(struct bonding *bond);
|
|
|
|
|
void bond_change_active_slave(struct bonding *bond, struct slave *new_active);
|
2010-12-09 23:17:13 +08:00
|
|
|
void bond_create_debugfs(void);
|
|
|
|
|
void bond_destroy_debugfs(void);
|
|
|
|
|
void bond_debug_register(struct bonding *bond);
|
|
|
|
|
void bond_debug_unregister(struct bonding *bond);
|
|
|
|
|
void bond_debug_reregister(struct bonding *bond);
|
2011-04-13 23:22:29 +08:00
|
|
|
const char *bond_mode_name(int mode);
|
2013-10-18 23:43:33 +08:00
|
|
|
void bond_setup(struct net_device *bond_dev);
|
|
|
|
|
unsigned int bond_get_num_tx_queues(void);
|
|
|
|
|
int bond_netlink_init(void);
|
|
|
|
|
void bond_netlink_fini(void);
|
2013-10-18 23:43:37 +08:00
|
|
|
struct net_device *bond_option_active_slave_get_rcu(struct bonding *bond);
|
|
|
|
|
struct net_device *bond_option_active_slave_get(struct bonding *bond);
|
2014-01-17 14:57:49 +08:00
|
|
|
const char *bond_slave_link_status(s8 link);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2009-10-29 22:18:26 +08:00
|
|
|
struct bond_net {
|
|
|
|
|
struct net * net; /* Associated network namespace */
|
|
|
|
|
struct list_head dev_list;
|
|
|
|
|
#ifdef CONFIG_PROC_FS
|
|
|
|
|
struct proc_dir_entry * proc_dir;
|
|
|
|
|
#endif
|
2011-10-13 05:56:25 +08:00
|
|
|
struct class_attribute class_attr_bonding_masters;
|
2009-10-29 22:18:26 +08:00
|
|
|
};
|
|
|
|
|
|
2011-03-07 05:58:46 +08:00
|
|
|
#ifdef CONFIG_PROC_FS
|
|
|
|
|
void bond_create_proc_entry(struct bonding *bond);
|
|
|
|
|
void bond_remove_proc_entry(struct bonding *bond);
|
|
|
|
|
void bond_create_proc_dir(struct bond_net *bn);
|
|
|
|
|
void bond_destroy_proc_dir(struct bond_net *bn);
|
|
|
|
|
#else
|
|
|
|
|
static inline void bond_create_proc_entry(struct bonding *bond)
|
|
|
|
|
{
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static inline void bond_remove_proc_entry(struct bonding *bond)
|
|
|
|
|
{
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static inline void bond_create_proc_dir(struct bond_net *bn)
|
|
|
|
|
{
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
static inline void bond_destroy_proc_dir(struct bond_net *bn)
|
|
|
|
|
{
|
|
|
|
|
}
|
|
|
|
|
#endif
|
|
|
|
|
|
2012-11-28 07:57:04 +08:00
|
|
|
static inline struct slave *bond_slave_has_mac(struct bonding *bond,
|
|
|
|
|
const u8 *mac)
|
|
|
|
|
{
|
2013-09-25 15:20:14 +08:00
|
|
|
struct list_head *iter;
|
2012-11-28 07:57:04 +08:00
|
|
|
struct slave *tmp;
|
|
|
|
|
|
2013-09-25 15:20:14 +08:00
|
|
|
bond_for_each_slave(bond, tmp, iter)
|
2012-11-28 07:57:04 +08:00
|
|
|
if (ether_addr_equal_64bits(mac, tmp->dev->dev_addr))
|
|
|
|
|
return tmp;
|
|
|
|
|
|
|
|
|
|
return NULL;
|
|
|
|
|
}
|
2011-03-07 05:58:46 +08:00
|
|
|
|
2013-10-15 16:28:39 +08:00
|
|
|
/* Caller must hold rcu_read_lock() for read */
|
|
|
|
|
static inline struct slave *bond_slave_has_mac_rcu(struct bonding *bond,
|
|
|
|
|
const u8 *mac)
|
|
|
|
|
{
|
|
|
|
|
struct list_head *iter;
|
|
|
|
|
struct slave *tmp;
|
|
|
|
|
|
|
|
|
|
bond_for_each_slave_rcu(bond, tmp, iter)
|
|
|
|
|
if (ether_addr_equal_64bits(mac, tmp->dev->dev_addr))
|
|
|
|
|
return tmp;
|
|
|
|
|
|
|
|
|
|
return NULL;
|
|
|
|
|
}
|
|
|
|
|
|
2013-06-24 17:49:29 +08:00
|
|
|
/* Check if the ip is present in arp ip list, or first free slot if ip == 0
|
|
|
|
|
* Returns -1 if not found, index if found
|
|
|
|
|
*/
|
|
|
|
|
static inline int bond_get_targets_ip(__be32 *targets, __be32 ip)
|
|
|
|
|
{
|
|
|
|
|
int i;
|
|
|
|
|
|
|
|
|
|
for (i = 0; i < BOND_MAX_ARP_TARGETS; i++)
|
|
|
|
|
if (targets[i] == ip)
|
|
|
|
|
return i;
|
|
|
|
|
else if (targets[i] == 0)
|
|
|
|
|
break;
|
|
|
|
|
|
|
|
|
|
return -1;
|
|
|
|
|
}
|
|
|
|
|
|
2008-09-14 23:56:12 +08:00
|
|
|
/* exported from bond_main.c */
|
2009-10-29 22:18:26 +08:00
|
|
|
extern int bond_net_id;
|
2008-12-10 15:10:38 +08:00
|
|
|
extern const struct bond_parm_tbl bond_lacp_tbl[];
|
|
|
|
|
extern const struct bond_parm_tbl xmit_hashtype_tbl[];
|
|
|
|
|
extern const struct bond_parm_tbl arp_validate_tbl[];
|
bonding: add an option to fail when any of arp_ip_target is inaccessible
Currently, we fail only when all of the ips in arp_ip_target are gone.
However, in some situations we might need to fail if even one host from
arp_ip_target becomes unavailable.
All situations, obviously, rely on the idea that we need *completely*
functional network, with all interfaces/addresses working correctly.
One real world example might be:
vlans on top on bond (hybrid port). If bond and vlans have ips assigned
and we have their peers monitored via arp_ip_target - in case of switch
misconfiguration (trunk/access port), slave driver malfunction or
tagged/untagged traffic dropped on the way - we will be able to switch
to another slave.
Though any other configuration needs that if we need to have access to all
arp_ip_targets.
This patch adds this possibility by adding a new parameter -
arp_all_targets (both as a module parameter and as a sysfs knob). It can be
set to:
0 or any (the default) - which works exactly as it's working now -
the slave is up if any of the arp_ip_targets are up.
1 or all - the slave is up if all of the arp_ip_targets are up.
This parameter can be changed on the fly (via sysfs), and requires the mode
to be active-backup and arp_validate to be enabled (it obeys the
arp_validate config on which slaves to validate).
Internally it's done through:
1) Add target_last_arp_rx[BOND_MAX_ARP_TARGETS] array to slave struct. It's
an array of jiffies, meaning that slave->target_last_arp_rx[i] is the
last time we've received arp from bond->params.arp_targets[i] on this
slave.
2) If we successfully validate an arp from bond->params.arp_targets[i] in
bond_validate_arp() - update the slave->target_last_arp_rx[i] with the
current jiffies value.
3) When getting slave's last_rx via slave_last_rx(), we return the oldest
time when we've received an arp from any address in
bond->params.arp_targets[].
If the value of arp_all_targets == 0 - we still work the same way as
before.
Also, update the documentation to reflect the new parameter.
v3->v4:
Kill the forgotten rtnl_unlock(), rephrase the documentation part to be
more clear, don't fail setting arp_all_targets if arp_validate is not set -
it has no effect anyway but can be easier to set up. Also, print a warning
if the last arp_ip_target is removed while the arp_interval is on, but not
the arp_validate.
v2->v3:
Use _bh spinlock, remove useless rtnl_lock() and use jiffies for new
arp_ip_target last arp, instead of slave_last_rx(). On bond_enslave(),
use the same initialization value for target_last_arp_rx[] as is used
for the default last_arp_rx, to avoid useless interface flaps.
Also, instead of failing to remove the last arp_ip_target just print a
warning - otherwise it might break existing scripts.
v1->v2:
Correctly handle adding/removing hosts in arp_ip_target - we need to
shift/initialize all slave's target_last_arp_rx. Also, don't fail module
loading on arp_all_targets misconfiguration, just disable it, and some
minor style fixes.
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
2013-06-24 17:49:34 +08:00
|
|
|
extern const struct bond_parm_tbl arp_all_targets_tbl[];
|
2008-12-10 15:10:38 +08:00
|
|
|
extern const struct bond_parm_tbl fail_over_mac_tbl[];
|
2009-09-25 11:28:09 +08:00
|
|
|
extern const struct bond_parm_tbl pri_reselect_tbl[];
|
2008-12-26 15:58:57 +08:00
|
|
|
extern struct bond_parm_tbl ad_select_tbl[];
|
|
|
|
|
|
2013-10-18 23:43:33 +08:00
|
|
|
/* exported from bond_netlink.c */
|
|
|
|
|
extern struct rtnl_link_ops bond_link_ops;
|
|
|
|
|
|
2005-04-17 06:20:36 +08:00
|
|
|
#endif /* _LINUX_BONDING_H */
|
