Commit 060ad609 authored by David S. Miller's avatar David S. Miller

Merge branch 'bonding-fixes'

Hangbin Liu says:

====================
bonding: fix lladdr finding and confirmation

This patch set fixed 3 issues when setting lladdr as bonding IPv6 target.
Please see each patch for the details.

v2: separate the patch to 3 parts
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents c9071401 592335a4
...@@ -3167,6 +3167,9 @@ static void bond_ns_send_all(struct bonding *bond, struct slave *slave) ...@@ -3167,6 +3167,9 @@ static void bond_ns_send_all(struct bonding *bond, struct slave *slave)
found: found:
if (!ipv6_dev_get_saddr(dev_net(dst->dev), dst->dev, &targets[i], 0, &saddr)) if (!ipv6_dev_get_saddr(dev_net(dst->dev), dst->dev, &targets[i], 0, &saddr))
bond_ns_send(slave, &targets[i], &saddr, tags); bond_ns_send(slave, &targets[i], &saddr, tags);
else
bond_ns_send(slave, &targets[i], &in6addr_any, tags);
dst_release(dst); dst_release(dst);
kfree(tags); kfree(tags);
} }
...@@ -3198,12 +3201,19 @@ static bool bond_has_this_ip6(struct bonding *bond, struct in6_addr *addr) ...@@ -3198,12 +3201,19 @@ static bool bond_has_this_ip6(struct bonding *bond, struct in6_addr *addr)
return ret; return ret;
} }
static void bond_validate_ns(struct bonding *bond, struct slave *slave, static void bond_validate_na(struct bonding *bond, struct slave *slave,
struct in6_addr *saddr, struct in6_addr *daddr) struct in6_addr *saddr, struct in6_addr *daddr)
{ {
int i; int i;
if (ipv6_addr_any(saddr) || !bond_has_this_ip6(bond, daddr)) { /* Ignore NAs that:
* 1. Source address is unspecified address.
* 2. Dest address is neither all-nodes multicast address nor
* exist on bond interface.
*/
if (ipv6_addr_any(saddr) ||
(!ipv6_addr_equal(daddr, &in6addr_linklocal_allnodes) &&
!bond_has_this_ip6(bond, daddr))) {
slave_dbg(bond->dev, slave->dev, "%s: sip %pI6c tip %pI6c not found\n", slave_dbg(bond->dev, slave->dev, "%s: sip %pI6c tip %pI6c not found\n",
__func__, saddr, daddr); __func__, saddr, daddr);
return; return;
...@@ -3246,14 +3256,14 @@ static int bond_na_rcv(const struct sk_buff *skb, struct bonding *bond, ...@@ -3246,14 +3256,14 @@ static int bond_na_rcv(const struct sk_buff *skb, struct bonding *bond,
* see bond_arp_rcv(). * see bond_arp_rcv().
*/ */
if (bond_is_active_slave(slave)) if (bond_is_active_slave(slave))
bond_validate_ns(bond, slave, saddr, daddr); bond_validate_na(bond, slave, saddr, daddr);
else if (curr_active_slave && else if (curr_active_slave &&
time_after(slave_last_rx(bond, curr_active_slave), time_after(slave_last_rx(bond, curr_active_slave),
curr_active_slave->last_link_up)) curr_active_slave->last_link_up))
bond_validate_ns(bond, slave, saddr, daddr); bond_validate_na(bond, slave, saddr, daddr);
else if (curr_arp_slave && else if (curr_arp_slave &&
bond_time_in_interval(bond, slave_last_tx(curr_arp_slave), 1)) bond_time_in_interval(bond, slave_last_tx(curr_arp_slave), 1))
bond_validate_ns(bond, slave, saddr, daddr); bond_validate_na(bond, slave, saddr, daddr);
out: out:
return RX_HANDLER_ANOTHER; return RX_HANDLER_ANOTHER;
......
...@@ -3557,11 +3557,15 @@ static int addrconf_notify(struct notifier_block *this, unsigned long event, ...@@ -3557,11 +3557,15 @@ static int addrconf_notify(struct notifier_block *this, unsigned long event,
fallthrough; fallthrough;
case NETDEV_UP: case NETDEV_UP:
case NETDEV_CHANGE: case NETDEV_CHANGE:
if (dev->flags & IFF_SLAVE) if (idev && idev->cnf.disable_ipv6)
break; break;
if (idev && idev->cnf.disable_ipv6) if (dev->flags & IFF_SLAVE) {
if (event == NETDEV_UP && !IS_ERR_OR_NULL(idev) &&
dev->flags & IFF_UP && dev->flags & IFF_MULTICAST)
ipv6_mc_up(idev);
break; break;
}
if (event == NETDEV_UP) { if (event == NETDEV_UP) {
/* restore routes for permanent addresses */ /* restore routes for permanent addresses */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment