projects
/
project
/
netifd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
bring all interfaces down on restart
[project/netifd.git]
/
interface.c
diff --git
a/interface.c
b/interface.c
index
3b3e2e3
..
4b22807
100644
(file)
--- a/
interface.c
+++ b/
interface.c
@@
-5,6
+5,7
@@
#include "netifd.h"
#include "device.h"
#include "interface.h"
#include "netifd.h"
#include "device.h"
#include "interface.h"
+#include "interface-ip.h"
#include "proto.h"
#include "ubus.h"
#include "config.h"
#include "proto.h"
#include "ubus.h"
#include "config.h"
@@
-12,28
+13,21
@@
static LIST_HEAD(interfaces);
enum {
static LIST_HEAD(interfaces);
enum {
- IFACE_ATTR_TYPE,
IFACE_ATTR_IFNAME,
IFACE_ATTR_PROTO,
IFACE_ATTR_AUTO,
IFACE_ATTR_MAX
};
IFACE_ATTR_IFNAME,
IFACE_ATTR_PROTO,
IFACE_ATTR_AUTO,
IFACE_ATTR_MAX
};
-static const union config_param_info iface_attr_info[IFACE_ATTR_MAX] = {
- [IFACE_ATTR_IFNAME].type = BLOBMSG_TYPE_STRING,
-};
-
static const struct blobmsg_policy iface_attrs[IFACE_ATTR_MAX] = {
static const struct blobmsg_policy iface_attrs[IFACE_ATTR_MAX] = {
- [IFACE_ATTR_TYPE] = { .name = "type", .type = BLOBMSG_TYPE_STRING },
[IFACE_ATTR_PROTO] = { .name = "proto", .type = BLOBMSG_TYPE_STRING },
[IFACE_ATTR_PROTO] = { .name = "proto", .type = BLOBMSG_TYPE_STRING },
- [IFACE_ATTR_IFNAME] = { .name = "ifname", .type = BLOBMSG_TYPE_
ARRAY
},
+ [IFACE_ATTR_IFNAME] = { .name = "ifname", .type = BLOBMSG_TYPE_
STRING
},
[IFACE_ATTR_AUTO] = { .name = "auto", .type = BLOBMSG_TYPE_BOOL },
};
const struct config_param_list interface_attr_list = {
.n_params = IFACE_ATTR_MAX,
.params = iface_attrs,
[IFACE_ATTR_AUTO] = { .name = "auto", .type = BLOBMSG_TYPE_BOOL },
};
const struct config_param_list interface_attr_list = {
.n_params = IFACE_ATTR_MAX,
.params = iface_attrs,
- .info = iface_attr_info,
};
static void
};
static void
@@
-90,8
+84,9
@@
interface_event(struct interface *iface, enum interface_event ev)
static void
mark_interface_down(struct interface *iface)
{
static void
mark_interface_down(struct interface *iface)
{
- interface_del_ctx_addr(iface, NULL);
- device_release(iface->main_dev.dev);
+ vlist_flush_all(&iface->proto_addr);
+ vlist_flush_all(&iface->proto_route);
+ device_release(&iface->main_dev);
iface->state = IFS_DOWN;
}
iface->state = IFS_DOWN;
}
@@
-103,7
+98,7
@@
__interface_set_up(struct interface *iface)
if (iface->state != IFS_DOWN)
return 0;
if (iface->state != IFS_DOWN)
return 0;
- ret = device_claim(
iface->main_dev.
dev);
+ ret = device_claim(
&iface->main_
dev);
if (ret)
return ret;
if (ret)
return ret;
@@
-129,8
+124,6
@@
__interface_set_down(struct interface *iface, bool force)
iface->state = IFS_TEARDOWN;
interface_event(iface, IFEV_DOWN);
iface->state = IFS_TEARDOWN;
interface_event(iface, IFEV_DOWN);
-
- interface_del_all_routes(iface);
interface_proto_event(iface->proto, PROTO_CMD_TEARDOWN, force);
}
interface_proto_event(iface->proto, PROTO_CMD_TEARDOWN, force);
}
@@
-152,10
+145,10
@@
interface_cb(struct device_user *dep, enum device_event ev)
return;
}
return;
}
- if (iface->a
ctiv
e == new_state)
+ if (iface->a
vailabl
e == new_state)
return;
return;
- iface->a
ctiv
e = new_state;
+ iface->a
vailabl
e = new_state;
if (new_state) {
if (iface->autostart && !config_init)
if (new_state) {
if (iface->autostart && !config_init)
@@
-203,12
+196,13
@@
void interface_set_proto_state(struct interface *iface, struct interface_proto_s
}
struct interface *
}
struct interface *
-interface_alloc(const char *name, struct
uci_section *s, struct
blob_attr *attr)
+interface_alloc(const char *name, struct blob_attr *attr)
{
struct interface *iface;
struct blob_attr *tb[IFACE_ATTR_MAX];
struct blob_attr *cur;
struct device *dev;
{
struct interface *iface;
struct blob_attr *tb[IFACE_ATTR_MAX];
struct blob_attr *cur;
struct device *dev;
+ const char *proto_name = NULL;
iface = interface_get(name);
if (iface)
iface = interface_get(name);
if (iface)
@@
-218,22
+212,18
@@
interface_alloc(const char *name, struct uci_section *s, struct blob_attr *attr)
iface->main_dev.cb = interface_cb;
iface->l3_iface = &iface->main_dev;
strncpy(iface->name, name, sizeof(iface->name) - 1);
iface->main_dev.cb = interface_cb;
iface->l3_iface = &iface->main_dev;
strncpy(iface->name, name, sizeof(iface->name) - 1);
- list_add(&iface->list, &interfaces);
+ list_add
_tail
(&iface->list, &interfaces);
INIT_LIST_HEAD(&iface->errors);
INIT_LIST_HEAD(&iface->errors);
- INIT_LIST_HEAD(&iface->address);
- INIT_LIST_HEAD(&iface->routes);
- proto_attach_interface(iface, s);
-
- netifd_ubus_add_interface(iface);
+ interface_ip_init(iface);
blobmsg_parse(iface_attrs, IFACE_ATTR_MAX, tb,
blob_data(attr), blob_len(attr));
blobmsg_parse(iface_attrs, IFACE_ATTR_MAX, tb,
blob_data(attr), blob_len(attr));
- if ((cur = tb[IFACE_ATTR_
TYPE])) {
- if (!strcmp(blobmsg_data(cur), "bridge"))
- interface_attach_bridge(iface, s);
- }
+ if ((cur = tb[IFACE_ATTR_
PROTO]))
+ proto_name = blobmsg_data(cur);
+
+ proto_attach_interface(iface, proto_name);
if ((cur = tb[IFACE_ATTR_IFNAME])) {
dev = device_get(blobmsg_data(cur), true);
if ((cur = tb[IFACE_ATTR_IFNAME])) {
dev = device_get(blobmsg_data(cur), true);
@@
-241,6
+231,14
@@
interface_alloc(const char *name, struct uci_section *s, struct blob_attr *attr)
device_add_user(&iface->main_dev, dev);
}
device_add_user(&iface->main_dev, dev);
}
+ if ((cur = tb[IFACE_ATTR_AUTO]))
+ iface->autostart = blobmsg_get_bool(cur);
+ else
+ iface->autostart = true;
+
+ netifd_ubus_add_interface(iface);
+ config_set_state(&iface->config, attr);
+
return iface;
}
return iface;
}
@@
-300,7
+298,7
@@
interface_set_up(struct interface *iface)
{
iface->autostart = true;
{
iface->autostart = true;
- if (!iface->a
ctiv
e) {
+ if (!iface->a
vailabl
e) {
interface_add_error(iface, "interface", "NO_DEVICE", NULL, 0);
return -1;
}
interface_add_error(iface, "interface", "NO_DEVICE", NULL, 0);
return -1;
}
@@
-314,8
+312,13
@@
interface_set_up(struct interface *iface)
int
interface_set_down(struct interface *iface)
{
int
interface_set_down(struct interface *iface)
{
- iface->autostart = false;
- __interface_set_down(iface, false);
+ if (!iface) {
+ list_for_each_entry(iface, &interfaces, list)
+ __interface_set_down(iface, false);
+ } else {
+ iface->autostart = false;
+ __interface_set_down(iface, false);
+ }
return 0;
}
return 0;
}
@@
-326,7
+329,7
@@
interface_start_pending(void)
struct interface *iface;
list_for_each_entry(iface, &interfaces, list) {
struct interface *iface;
list_for_each_entry(iface, &interfaces, list) {
- if (iface->a
ctiv
e && iface->autostart)
+ if (iface->a
vailabl
e && iface->autostart)
interface_set_up(iface);
}
}
interface_set_up(iface);
}
}