diff options
author | Felix Fietkau <nbd@nbd.name> | 2021-09-01 19:00:41 +0200 |
---|---|---|
committer | Felix Fietkau <nbd@nbd.name> | 2021-09-01 19:03:30 +0200 |
commit | 300b1220fab38600f102bb8cfcc59a29ce41b095 (patch) | |
tree | 103ae559ccdf4f34065fb761bcd25057b3d5ddb0 /wireless.c | |
parent | f8ff6d82028367620893872636d455702a1c1ea1 (diff) | |
download | netifd-300b1220fab38600f102bb8cfcc59a29ce41b095.tar.gz |
wireless: improve reliability of proxyarp support
instead of relying on hostapd to manipulate bridge attributes (which can race
against netifd adding/removing of member ports), set the proxyarp related
attributes in netifd directly when bringing up the member port
Signed-off-by: Felix Fietkau <nbd@nbd.name>
Diffstat (limited to 'wireless.c')
-rw-r--r-- | wireless.c | 7 |
1 files changed, 7 insertions, 0 deletions
@@ -64,6 +64,7 @@ enum { VIF_ATTR_NETWORK, VIF_ATTR_ISOLATE, VIF_ATTR_MODE, + VIF_ATTR_PROXYARP, __VIF_ATTR_MAX, }; @@ -72,6 +73,7 @@ static const struct blobmsg_policy vif_policy[__VIF_ATTR_MAX] = { [VIF_ATTR_NETWORK] = { .name = "network", .type = BLOBMSG_TYPE_ARRAY }, [VIF_ATTR_ISOLATE] = { .name = "isolate", .type = BLOBMSG_TYPE_BOOL }, [VIF_ATTR_MODE] = { .name = "mode", .type = BLOBMSG_TYPE_STRING }, + [VIF_ATTR_PROXYARP] = { .name = "proxy_arp", .type = BLOBMSG_TYPE_BOOL }, }; static const struct uci_blob_param_list vif_param = { @@ -330,6 +332,7 @@ static void wireless_interface_handle_link(struct wireless_interface *vif, const struct device *dev = device_get(ifname, 2); if (dev) { dev->wireless_isolate = vif->isolate; + dev->wireless_proxyarp = vif->proxyarp; dev->wireless = true; dev->wireless_ap = vif->ap_mode; dev->bpdu_filter = dev->wireless_ap && ifname == vif->ifname; @@ -794,6 +797,10 @@ wireless_interface_init_config(struct wireless_interface *vif) if (cur) vif->isolate = blobmsg_get_bool(cur); + cur = tb[VIF_ATTR_PROXYARP]; + if (cur) + vif->proxyarp = blobmsg_get_bool(cur); + cur = tb[VIF_ATTR_MODE]; if (cur) vif->ap_mode = !strcmp(blobmsg_get_string(cur), "ap"); |