Revert "kernel: disable multicast-to-unicast translation for ipv6 neighbor solicitation (#17625)"
This reverts commit a080e8e1943156168913d0353a2e99d1151102aa. It did not fix the problem but just hid some symptom. The real issue was that IGMP/MLD report suppression was not considered for the multicast-to-unicast feature. A recent netifd which isolates IGMP/MLD reports between STAs by utilizing AP-isolation and bridge-hairpinning should have fixed this. It is perfectly fine to apply multicast-to-unicast to IPv6 Neighbor Solicitations, too (once that feature is configured correctly). Signed-off-by: Linus Lüssing <linus.luessing@c0d3.blue> Signed-off-by: Felix Fietkau <nbd@openwrt.org> Backport of r46720 git-svn-id: svn://svn.openwrt.org/openwrt/branches/chaos_calmer@47000 3c298f89-4303-0410-b956-a3cf2f4a3e73master
parent
3747e68014
commit
fcffd63a99
|
@ -87,19 +87,16 @@
|
||||||
{
|
{
|
||||||
struct br_ip br_group;
|
struct br_ip br_group;
|
||||||
|
|
||||||
@@ -736,7 +758,10 @@ static int br_ip6_multicast_add_group(st
|
@@ -736,7 +758,7 @@ static int br_ip6_multicast_add_group(st
|
||||||
br_group.proto = htons(ETH_P_IPV6);
|
br_group.proto = htons(ETH_P_IPV6);
|
||||||
br_group.vid = vid;
|
br_group.vid = vid;
|
||||||
|
|
||||||
- return br_multicast_add_group(br, port, &br_group);
|
- return br_multicast_add_group(br, port, &br_group);
|
||||||
+ if (ipv6_addr_is_solict_mult(group))
|
|
||||||
+ src = NULL;
|
|
||||||
+
|
|
||||||
+ return br_multicast_add_group(br, port, &br_group, src);
|
+ return br_multicast_add_group(br, port, &br_group, src);
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
@@ -965,6 +990,7 @@ static int br_ip4_multicast_igmp3_report
|
@@ -965,6 +987,7 @@ static int br_ip4_multicast_igmp3_report
|
||||||
struct sk_buff *skb,
|
struct sk_buff *skb,
|
||||||
u16 vid)
|
u16 vid)
|
||||||
{
|
{
|
||||||
|
@ -107,7 +104,7 @@
|
||||||
struct igmpv3_report *ih;
|
struct igmpv3_report *ih;
|
||||||
struct igmpv3_grec *grec;
|
struct igmpv3_grec *grec;
|
||||||
int i;
|
int i;
|
||||||
@@ -1008,7 +1034,7 @@ static int br_ip4_multicast_igmp3_report
|
@@ -1008,7 +1031,7 @@ static int br_ip4_multicast_igmp3_report
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -116,7 +113,7 @@
|
||||||
if (err)
|
if (err)
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
@@ -1022,6 +1048,7 @@ static int br_ip6_multicast_mld2_report(
|
@@ -1022,6 +1045,7 @@ static int br_ip6_multicast_mld2_report(
|
||||||
struct sk_buff *skb,
|
struct sk_buff *skb,
|
||||||
u16 vid)
|
u16 vid)
|
||||||
{
|
{
|
||||||
|
@ -124,7 +121,7 @@
|
||||||
struct icmp6hdr *icmp6h;
|
struct icmp6hdr *icmp6h;
|
||||||
struct mld2_grec *grec;
|
struct mld2_grec *grec;
|
||||||
int i;
|
int i;
|
||||||
@@ -1070,7 +1097,7 @@ static int br_ip6_multicast_mld2_report(
|
@@ -1070,7 +1094,7 @@ static int br_ip6_multicast_mld2_report(
|
||||||
}
|
}
|
||||||
|
|
||||||
err = br_ip6_multicast_add_group(br, port, &grec->grec_mca,
|
err = br_ip6_multicast_add_group(br, port, &grec->grec_mca,
|
||||||
|
@ -133,7 +130,7 @@
|
||||||
if (err)
|
if (err)
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
@@ -1406,7 +1433,8 @@ br_multicast_leave_group(struct net_brid
|
@@ -1407,7 +1431,8 @@ br_multicast_leave_group(struct net_brid
|
||||||
struct net_bridge_port *port,
|
struct net_bridge_port *port,
|
||||||
struct br_ip *group,
|
struct br_ip *group,
|
||||||
struct bridge_mcast_other_query *other_query,
|
struct bridge_mcast_other_query *other_query,
|
||||||
|
@ -143,7 +140,7 @@
|
||||||
{
|
{
|
||||||
struct net_bridge_mdb_htable *mdb;
|
struct net_bridge_mdb_htable *mdb;
|
||||||
struct net_bridge_mdb_entry *mp;
|
struct net_bridge_mdb_entry *mp;
|
||||||
@@ -1456,7 +1484,7 @@ br_multicast_leave_group(struct net_brid
|
@@ -1457,7 +1482,7 @@ br_multicast_leave_group(struct net_brid
|
||||||
for (pp = &mp->ports;
|
for (pp = &mp->ports;
|
||||||
(p = mlock_dereference(*pp, br)) != NULL;
|
(p = mlock_dereference(*pp, br)) != NULL;
|
||||||
pp = &p->next) {
|
pp = &p->next) {
|
||||||
|
@ -152,7 +149,7 @@
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
rcu_assign_pointer(*pp, p->next);
|
rcu_assign_pointer(*pp, p->next);
|
||||||
@@ -1490,7 +1518,7 @@ br_multicast_leave_group(struct net_brid
|
@@ -1491,7 +1516,7 @@ br_multicast_leave_group(struct net_brid
|
||||||
for (p = mlock_dereference(mp->ports, br);
|
for (p = mlock_dereference(mp->ports, br);
|
||||||
p != NULL;
|
p != NULL;
|
||||||
p = mlock_dereference(p->next, br)) {
|
p = mlock_dereference(p->next, br)) {
|
||||||
|
@ -161,7 +158,7 @@
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
if (!hlist_unhashed(&p->mglist) &&
|
if (!hlist_unhashed(&p->mglist) &&
|
||||||
@@ -1508,8 +1536,8 @@ out:
|
@@ -1509,8 +1534,8 @@ out:
|
||||||
|
|
||||||
static void br_ip4_multicast_leave_group(struct net_bridge *br,
|
static void br_ip4_multicast_leave_group(struct net_bridge *br,
|
||||||
struct net_bridge_port *port,
|
struct net_bridge_port *port,
|
||||||
|
@ -172,7 +169,7 @@
|
||||||
{
|
{
|
||||||
struct br_ip br_group;
|
struct br_ip br_group;
|
||||||
struct bridge_mcast_own_query *own_query;
|
struct bridge_mcast_own_query *own_query;
|
||||||
@@ -1524,14 +1552,14 @@ static void br_ip4_multicast_leave_group
|
@@ -1525,14 +1550,14 @@ static void br_ip4_multicast_leave_group
|
||||||
br_group.vid = vid;
|
br_group.vid = vid;
|
||||||
|
|
||||||
br_multicast_leave_group(br, port, &br_group, &br->ip4_other_query,
|
br_multicast_leave_group(br, port, &br_group, &br->ip4_other_query,
|
||||||
|
@ -189,7 +186,7 @@
|
||||||
{
|
{
|
||||||
struct br_ip br_group;
|
struct br_ip br_group;
|
||||||
struct bridge_mcast_own_query *own_query;
|
struct bridge_mcast_own_query *own_query;
|
||||||
@@ -1546,7 +1574,7 @@ static void br_ip6_multicast_leave_group
|
@@ -1547,7 +1572,7 @@ static void br_ip6_multicast_leave_group
|
||||||
br_group.vid = vid;
|
br_group.vid = vid;
|
||||||
|
|
||||||
br_multicast_leave_group(br, port, &br_group, &br->ip6_other_query,
|
br_multicast_leave_group(br, port, &br_group, &br->ip6_other_query,
|
||||||
|
@ -198,7 +195,7 @@
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
@@ -1555,6 +1583,7 @@ static int br_multicast_ipv4_rcv(struct
|
@@ -1556,6 +1581,7 @@ static int br_multicast_ipv4_rcv(struct
|
||||||
struct sk_buff *skb,
|
struct sk_buff *skb,
|
||||||
u16 vid)
|
u16 vid)
|
||||||
{
|
{
|
||||||
|
@ -206,7 +203,7 @@
|
||||||
struct sk_buff *skb2 = skb;
|
struct sk_buff *skb2 = skb;
|
||||||
const struct iphdr *iph;
|
const struct iphdr *iph;
|
||||||
struct igmphdr *ih;
|
struct igmphdr *ih;
|
||||||
@@ -1628,7 +1657,7 @@ static int br_multicast_ipv4_rcv(struct
|
@@ -1629,7 +1655,7 @@ static int br_multicast_ipv4_rcv(struct
|
||||||
case IGMP_HOST_MEMBERSHIP_REPORT:
|
case IGMP_HOST_MEMBERSHIP_REPORT:
|
||||||
case IGMPV2_HOST_MEMBERSHIP_REPORT:
|
case IGMPV2_HOST_MEMBERSHIP_REPORT:
|
||||||
BR_INPUT_SKB_CB(skb)->mrouters_only = 1;
|
BR_INPUT_SKB_CB(skb)->mrouters_only = 1;
|
||||||
|
@ -215,7 +212,7 @@
|
||||||
break;
|
break;
|
||||||
case IGMPV3_HOST_MEMBERSHIP_REPORT:
|
case IGMPV3_HOST_MEMBERSHIP_REPORT:
|
||||||
err = br_ip4_multicast_igmp3_report(br, port, skb2, vid);
|
err = br_ip4_multicast_igmp3_report(br, port, skb2, vid);
|
||||||
@@ -1637,7 +1666,7 @@ static int br_multicast_ipv4_rcv(struct
|
@@ -1638,7 +1664,7 @@ static int br_multicast_ipv4_rcv(struct
|
||||||
err = br_ip4_multicast_query(br, port, skb2, vid);
|
err = br_ip4_multicast_query(br, port, skb2, vid);
|
||||||
break;
|
break;
|
||||||
case IGMP_HOST_LEAVE_MESSAGE:
|
case IGMP_HOST_LEAVE_MESSAGE:
|
||||||
|
@ -224,7 +221,7 @@
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1655,6 +1684,7 @@ static int br_multicast_ipv6_rcv(struct
|
@@ -1656,6 +1682,7 @@ static int br_multicast_ipv6_rcv(struct
|
||||||
struct sk_buff *skb,
|
struct sk_buff *skb,
|
||||||
u16 vid)
|
u16 vid)
|
||||||
{
|
{
|
||||||
|
@ -232,7 +229,7 @@
|
||||||
struct sk_buff *skb2;
|
struct sk_buff *skb2;
|
||||||
const struct ipv6hdr *ip6h;
|
const struct ipv6hdr *ip6h;
|
||||||
u8 icmp6_type;
|
u8 icmp6_type;
|
||||||
@@ -1764,7 +1794,8 @@ static int br_multicast_ipv6_rcv(struct
|
@@ -1765,7 +1792,8 @@ static int br_multicast_ipv6_rcv(struct
|
||||||
}
|
}
|
||||||
mld = (struct mld_msg *)skb_transport_header(skb2);
|
mld = (struct mld_msg *)skb_transport_header(skb2);
|
||||||
BR_INPUT_SKB_CB(skb)->mrouters_only = 1;
|
BR_INPUT_SKB_CB(skb)->mrouters_only = 1;
|
||||||
|
@ -242,7 +239,7 @@
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
case ICMPV6_MLD2_REPORT:
|
case ICMPV6_MLD2_REPORT:
|
||||||
@@ -1781,7 +1812,7 @@ static int br_multicast_ipv6_rcv(struct
|
@@ -1782,7 +1810,7 @@ static int br_multicast_ipv6_rcv(struct
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
mld = (struct mld_msg *)skb_transport_header(skb2);
|
mld = (struct mld_msg *)skb_transport_header(skb2);
|
||||||
|
|
Loading…
Reference in New Issue