X-Git-Url: http://git.archive.openwrt.org/?p=project%2Fnetifd.git;a=blobdiff_plain;f=device.c;h=6929177cd59caee5c4feb233aaf9370c773e64e2;hp=e206a765669101d19f473fa8cdb927709df0e2a8;hb=c1db77f01848b4da256426383559a553547a81d5;hpb=5f739f10e9d7af0eb412102356434f905ed8f467 diff --git a/device.c b/device.c index e206a76..6929177 100644 --- a/device.c +++ b/device.c @@ -7,6 +7,10 @@ #include #include +#ifdef linux +#include +#endif + #include "netifd.h" #include "system.h" #include "config.h" @@ -95,7 +99,7 @@ device_init_settings(struct device *dev, struct blob_attr **tb) static void __init dev_init(void) { - avl_init(&devices, avl_strcmp, false, NULL); + avl_init(&devices, avl_strcmp, true, NULL); } static void device_broadcast_event(struct device *dev, enum device_event ev) @@ -129,7 +133,7 @@ int device_claim(struct device_user *dep) return 0; dep->claimed = true; - DPRINTF("claim device %s, new refcount: %d\n", dev->ifname, dev->active + 1); + D(DEVICE, "claim device %s, new refcount: %d\n", dev->ifname, dev->active + 1); if (++dev->active != 1) return 0; @@ -137,8 +141,11 @@ int device_claim(struct device_user *dep) ret = dev->set_state(dev, true); if (ret == 0) device_broadcast_event(dev, DEV_EVENT_UP); - else + else { + D(DEVICE, "claim device %s failed: %d\n", dev->ifname, ret); dev->active = 0; + dep->claimed = false; + } return ret; } @@ -152,7 +159,7 @@ void device_release(struct device_user *dep) dep->claimed = false; dev->active--; - DPRINTF("release device %s, new refcount: %d\n", dev->ifname, dev->active); + D(DEVICE, "release device %s, new refcount: %d\n", dev->ifname, dev->active); assert(dev->active >= 0); if (dev->active) @@ -163,7 +170,7 @@ void device_release(struct device_user *dep) device_broadcast_event(dev, DEV_EVENT_DOWN); } -int check_device_state(struct device *dev) +int device_check_state(struct device *dev) { if (!dev->type->check_state) return 0; @@ -179,7 +186,7 @@ void device_init_virtual(struct device *dev, const struct device_type *type, con if (name) strncpy(dev->ifname, name, IFNAMSIZ); - fprintf(stderr, "Initialize device '%s'\n", dev->ifname); + D(DEVICE, "Initialize device '%s'\n", dev->ifname); INIT_LIST_HEAD(&dev->users); dev->type = type; } @@ -199,12 +206,26 @@ int device_init(struct device *dev, const struct device_type *type, const char * if (ret < 0) return ret; - check_device_state(dev); + system_if_clear_state(dev); + device_check_state(dev); return 0; } -struct device *device_get(const char *name, bool create) +static struct device * +device_create_default(const char *name) +{ + struct device *dev; + + D(DEVICE, "Create simple device '%s'\n", name); + dev = calloc(1, sizeof(*dev)); + device_init(dev, &simple_device_type, name); + dev->default_config = true; + return dev; +} + +struct device * +device_get(const char *name, bool create) { struct device *dev; @@ -218,26 +239,34 @@ struct device *device_get(const char *name, bool create) if (!create) return NULL; - dev = calloc(1, sizeof(*dev)); - device_init(dev, &simple_device_type, name); + return device_create_default(name); +} - return dev; +static void +device_delete(struct device *dev) +{ + if (!dev->avl.key) + return; + + D(DEVICE, "Delete device '%s' from list\n", dev->ifname); + avl_delete(&devices, &dev->avl); + dev->avl.key = NULL; } void device_cleanup(struct device *dev) { struct device_user *dep, *tmp; - fprintf(stderr, "Clean up device '%s'\n", dev->ifname); + D(DEVICE, "Clean up device '%s'\n", dev->ifname); list_for_each_entry_safe(dep, tmp, &dev->users, list) { if (!dep->cb) continue; dep->cb(dep, DEV_EVENT_REMOVE); + device_release(dep); } - if (dev->avl.key) - avl_delete(&devices, &dev->avl); + device_delete(dev); } void device_set_present(struct device *dev, bool state) @@ -245,7 +274,7 @@ void device_set_present(struct device *dev, bool state) if (dev->present == state) return; - DPRINTF("Device '%s' %s present\n", dev->ifname, state ? "is now" : "is no longer" ); + D(DEVICE, "Device '%s' %s present\n", dev->ifname, state ? "is now" : "is no longer" ); dev->present = state; device_broadcast_event(dev, state ? DEV_EVENT_ADD : DEV_EVENT_REMOVE); } @@ -261,6 +290,15 @@ void device_add_user(struct device_user *dep, struct device *dev) } } +static void +__device_free_unused(struct device *dev) +{ + if (!list_empty(&dev->users) || dev->current_config || config_init) + return; + + device_free(dev); +} + void device_remove_user(struct device_user *dep) { struct device *dev = dep->dev; @@ -269,32 +307,165 @@ void device_remove_user(struct device_user *dep) device_release(dep); list_del(&dep->list); + dep->dev = NULL; + __device_free_unused(dev); +} - if (list_empty(&dev->users)) { - /* all references have gone away, remove this device */ - device_free(dev); +void +device_free_unused(struct device *dev) +{ + struct device *tmp; + + if (dev) + return __device_free_unused(dev); + + avl_for_each_element_safe(&devices, dev, avl, tmp) + __device_free_unused(dev); +} + +void +device_init_pending(void) +{ + struct device *dev, *tmp; + + avl_for_each_element_safe(&devices, dev, avl, tmp) { + if (!dev->config_pending) + continue; + + dev->type->config_init(dev); + dev->config_pending = false; } +} - dep->dev = NULL; +enum dev_change_type +device_reload_config(struct device *dev, struct blob_attr *attr) +{ + struct blob_attr *tb[__DEV_ATTR_MAX]; + const struct config_param_list *cfg = dev->type->config_params; + + if (config_check_equal(dev->config, attr, cfg)) + return DEV_CONFIG_NO_CHANGE; + + if (cfg == &device_attr_list) { + memset(tb, 0, sizeof(tb)); + + if (dev->config) + blobmsg_parse(dev_attrs, __DEV_ATTR_MAX, tb, + blob_data(attr), blob_len(attr)); + + device_init_settings(dev, tb); + return DEV_CONFIG_APPLIED; + } else + return DEV_CONFIG_RECREATE; +} + +static enum dev_change_type +device_check_config(struct device *dev, const struct device_type *type, + struct blob_attr *attr) +{ + if (type != dev->type) + return DEV_CONFIG_RECREATE; + + if (dev->type->reload) + return dev->type->reload(dev, attr); + + return device_reload_config(dev, attr); } static void -__device_free_unused(struct device *dev) +device_replace(struct device *dev, struct device *odev) { - if (!list_empty(&dev->users)) - return; + struct device_user *dep, *tmp; + bool present = odev->present; - device_free(dev); + if (present) + device_set_present(odev, false); + + list_for_each_entry_safe(dep, tmp, &odev->users, list) { + device_release(dep); + list_move_tail(&dep->list, &dev->users); + dep->dev = dev; + } + device_free(odev); + + if (present) + device_set_present(dev, true); } void -device_free_unused(struct device *dev) +device_reset_config(void) { - struct device *tmp; + struct device *dev; - if (dev) - return __device_free_unused(dev); + avl_for_each_element(&devices, dev, avl) + dev->current_config = false; +} - avl_for_each_element_safe(&devices, dev, avl, tmp) - __device_free_unused(dev); +void +device_reset_old(void) +{ + struct device *dev, *tmp, *ndev; + + avl_for_each_element_safe(&devices, dev, avl, tmp) { + if (dev->current_config || dev->default_config) + continue; + + if (dev->type != &simple_device_type) + continue; + + ndev = device_create_default(dev->ifname); + device_replace(ndev, dev); + } +} + +struct device * +device_create(const char *name, const struct device_type *type, + struct blob_attr *config) +{ + struct device *odev = NULL, *dev; + enum dev_change_type change; + + config = config_memdup(config); + if (!config) + return NULL; + + odev = device_get(name, false); + if (odev) { + odev->current_config = true; + change = device_check_config(odev, type, config); + switch (change) { + case DEV_CONFIG_APPLIED: + D(DEVICE, "Device '%s': config applied\n", odev->ifname); + free(odev->config); + odev->config = config; + if (odev->present) { + device_set_present(odev, false); + device_set_present(odev, true); + } + return odev; + case DEV_CONFIG_NO_CHANGE: + D(DEVICE, "Device '%s': no configuration change\n", odev->ifname); + free(config); + return odev; + case DEV_CONFIG_RECREATE: + D(DEVICE, "Device '%s': recreate device\n", odev->ifname); + device_delete(odev); + break; + } + } else + D(DEVICE, "Create new device '%s' (%s)\n", name, type->name); + + dev = type->create(config); + if (!dev) + return NULL; + + dev->current_config = true; + dev->config = config; + if (odev) + device_replace(dev, odev); + + if (!config_init && dev->config_pending) + type->config_init(dev); + + return dev; }