X-Git-Url: http://git.archive.openwrt.org/?p=project%2Fnetifd.git;a=blobdiff_plain;f=device.c;h=f856f229294d5aea871e2ec9e605eef23c236b7c;hp=bc870c24eb74fce34953ee462839bf88f9e4eadb;hb=37441269fbc27fc9af0562c6a4af50c1d6d353dd;hpb=4ed89555c4b5e748ed407f7694b5c9b32eea2e54 diff --git a/device.c b/device.c index bc870c2..f856f22 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) @@ -120,11 +124,16 @@ static int set_device_state(struct device *dev, bool state) return 0; } -int device_claim(struct device *dev) +int device_claim(struct device_user *dep) { + struct device *dev = dep->dev; int ret; - DPRINTF("claim device %s, new refcount: %d\n", dev->ifname, dev->active + 1); + if (dep->claimed) + return 0; + + dep->claimed = true; + D(DEVICE, "claim device %s, new refcount: %d\n", dev->ifname, dev->active + 1); if (++dev->active != 1) return 0; @@ -138,10 +147,16 @@ int device_claim(struct device *dev) return ret; } -void device_release(struct device *dev) +void device_release(struct device_user *dep) { + struct device *dev = dep->dev; + + if (!dep->claimed) + return; + + 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) @@ -152,7 +167,7 @@ void device_release(struct device *dev) 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; @@ -168,7 +183,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; } @@ -188,12 +203,25 @@ int device_init(struct device *dev, const struct device_type *type, const char * if (ret < 0) return ret; - check_device_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; @@ -207,17 +235,25 @@ 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; @@ -225,8 +261,7 @@ void device_cleanup(struct device *dev) dep->cb(dep, DEV_EVENT_REMOVE); } - if (dev->avl.key) - avl_delete(&devices, &dev->avl); + device_delete(dev); } void device_set_present(struct device *dev, bool state) @@ -234,7 +269,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); } @@ -242,7 +277,7 @@ void device_set_present(struct device *dev, bool state) void device_add_user(struct device_user *dep, struct device *dev) { dep->dev = dev; - list_add(&dep->list, &dev->users); + list_add_tail(&dep->list, &dev->users); if (dep->cb && dev->present) { dep->cb(dep, DEV_EVENT_ADD); if (dev->active) @@ -250,29 +285,180 @@ 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) + return; + + device_free(dev); +} + void device_remove_user(struct device_user *dep) { struct device *dev = dep->dev; + if (dep->claimed) + 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; - dep->dev = NULL; + if (dev) + return __device_free_unused(dev); + + avl_for_each_element_safe(&devices, dev, avl, tmp) + __device_free_unused(dev); } void -device_free_all(void) +device_init_pending(void) { struct device *dev, *tmp; avl_for_each_element_safe(&devices, dev, avl, tmp) { - if (!list_empty(&dev->users)) + if (!dev->config_pending) continue; - device_free(dev); + dev->type->config_init(dev); + dev->config_pending = false; } } + +enum dev_change_type +device_reload_config(struct device *dev, struct blob_attr *attr) +{ + struct blob_attr *tb[__DEV_ATTR_MAX], *tb1[__DEV_ATTR_MAX]; + + blobmsg_parse(dev_attrs, __DEV_ATTR_MAX, tb, + blob_data(attr), blob_len(attr)); + if (dev->config) + blobmsg_parse(dev_attrs, __DEV_ATTR_MAX, tb1, + blob_data(dev->config), blob_len(dev->config)); + else + memset(tb1, 0, sizeof(tb1)); + + if (!config_diff(tb, tb1, &device_attr_list, NULL)) + return DEV_CONFIG_NO_CHANGE; + + device_init_settings(dev, tb); + return DEV_CONFIG_APPLIED; +} + +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_replace(struct device *dev, struct device *odev) +{ + struct device_user *dep, *tmp; + bool present = odev->present; + + if (present) + device_set_present(odev, false); + + list_for_each_entry_safe(dep, tmp, &odev->users, list) { + device_remove_user(dep); + device_add_user(dep, dev); + } + device_free(odev); + + if (present) + device_set_present(dev, true); +} + +void +device_reset_config(void) +{ + struct device *dev; + + avl_for_each_element(&devices, dev, avl) + dev->current_config = false; +} + +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; + + D(DEVICE, "Create new device '%s' (%s)\n", name, type->name); + 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); + } + free(config); + 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; + } + } + + 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; +}