108 lines
3.3 KiB
Diff
108 lines
3.3 KiB
Diff
From: Felix Fietkau <nbd@nbd.name>
|
|
Subject: [PATCH] bridge: port isolate
|
|
|
|
Isolating individual bridge ports
|
|
---
|
|
--- a/include/linux/if_bridge.h
|
|
+++ b/include/linux/if_bridge.h
|
|
@@ -45,6 +45,7 @@ struct br_ip_list {
|
|
#define BR_PROXYARP BIT(8)
|
|
#define BR_LEARNING_SYNC BIT(9)
|
|
#define BR_PROXYARP_WIFI BIT(10)
|
|
+#define BR_ISOLATE_MODE BIT(11)
|
|
|
|
extern void brioctl_set(int (*ioctl_hook)(struct net *, unsigned int, void __user *));
|
|
|
|
--- a/net/bridge/br_sysfs_if.c
|
|
+++ b/net/bridge/br_sysfs_if.c
|
|
@@ -173,6 +173,22 @@ BRPORT_ATTR_FLAG(unicast_flood, BR_FLOOD
|
|
BRPORT_ATTR_FLAG(proxyarp, BR_PROXYARP);
|
|
BRPORT_ATTR_FLAG(proxyarp_wifi, BR_PROXYARP_WIFI);
|
|
|
|
+static ssize_t show_isolate_mode(struct net_bridge_port *p, char *buf)
|
|
+{
|
|
+ int isolate_mode = (p->flags & BR_ISOLATE_MODE) ? 1 : 0;
|
|
+ return sprintf(buf, "%d\n", isolate_mode);
|
|
+}
|
|
+static ssize_t store_isolate_mode(struct net_bridge_port *p, unsigned long v)
|
|
+{
|
|
+ if (v)
|
|
+ p->flags |= BR_ISOLATE_MODE;
|
|
+ else
|
|
+ p->flags &= ~BR_ISOLATE_MODE;
|
|
+ return 0;
|
|
+}
|
|
+static BRPORT_ATTR(isolate_mode, S_IRUGO | S_IWUSR,
|
|
+ show_isolate_mode, store_isolate_mode);
|
|
+
|
|
#ifdef CONFIG_BRIDGE_IGMP_SNOOPING
|
|
static ssize_t show_multicast_router(struct net_bridge_port *p, char *buf)
|
|
{
|
|
@@ -217,6 +233,7 @@ static const struct brport_attribute *br
|
|
#endif
|
|
&brport_attr_proxyarp,
|
|
&brport_attr_proxyarp_wifi,
|
|
+ &brport_attr_isolate_mode,
|
|
NULL
|
|
};
|
|
|
|
--- a/net/bridge/br_input.c
|
|
+++ b/net/bridge/br_input.c
|
|
@@ -185,8 +185,8 @@ int br_handle_frame_finish(struct sock *
|
|
|
|
unicast = false;
|
|
br->dev->stats.multicast++;
|
|
- } else if ((dst = __br_fdb_get(br, dest, vid)) &&
|
|
- dst->is_local) {
|
|
+ } else if ((p->flags & BR_ISOLATE_MODE) ||
|
|
+ ((dst = __br_fdb_get(br, dest, vid)) && dst->is_local)) {
|
|
skb2 = skb;
|
|
/* Do not forward the packet since it's local. */
|
|
skb = NULL;
|
|
--- a/net/bridge/br_forward.c
|
|
+++ b/net/bridge/br_forward.c
|
|
@@ -119,7 +119,7 @@ EXPORT_SYMBOL_GPL(br_deliver);
|
|
/* called with rcu_read_lock */
|
|
void br_forward(const struct net_bridge_port *to, struct sk_buff *skb, struct sk_buff *skb0)
|
|
{
|
|
- if (should_deliver(to, skb)) {
|
|
+ if (should_deliver(to, skb) && !(to->flags & BR_ISOLATE_MODE)) {
|
|
if (skb0)
|
|
deliver_clone(to, skb, __br_forward);
|
|
else
|
|
@@ -175,7 +175,7 @@ static void br_flood(struct net_bridge *
|
|
struct sk_buff *skb0,
|
|
void (*__packet_hook)(const struct net_bridge_port *p,
|
|
struct sk_buff *skb),
|
|
- bool unicast)
|
|
+ bool unicast, bool forward)
|
|
{
|
|
struct net_bridge_port *p;
|
|
struct net_bridge_port *prev;
|
|
@@ -183,6 +183,8 @@ static void br_flood(struct net_bridge *
|
|
prev = NULL;
|
|
|
|
list_for_each_entry_rcu(p, &br->port_list, list) {
|
|
+ if (forward && (p->flags & BR_ISOLATE_MODE))
|
|
+ continue;
|
|
/* Do not flood unicast traffic to ports that turn it off */
|
|
if (unicast && !(p->flags & BR_FLOOD))
|
|
continue;
|
|
@@ -217,14 +219,14 @@ out:
|
|
/* called with rcu_read_lock */
|
|
void br_flood_deliver(struct net_bridge *br, struct sk_buff *skb, bool unicast)
|
|
{
|
|
- br_flood(br, skb, NULL, __br_deliver, unicast);
|
|
+ br_flood(br, skb, NULL, __br_deliver, unicast, false);
|
|
}
|
|
|
|
/* called under bridge lock */
|
|
void br_flood_forward(struct net_bridge *br, struct sk_buff *skb,
|
|
struct sk_buff *skb2, bool unicast)
|
|
{
|
|
- br_flood(br, skb, skb2, __br_forward, unicast);
|
|
+ br_flood(br, skb, skb2, __br_forward, unicast, true);
|
|
}
|
|
|
|
#ifdef CONFIG_BRIDGE_IGMP_SNOOPING
|