X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=blobdiff_plain;f=src%2Fcore%2Fdevice.c;h=c7bc1e25e7bd5225a9ce089f79fb3244baf19be1;hp=91e37e09dbeca196f5eec3d359b92985498336b7;hb=6a6751fe24bf456cf5c1efad785a4d11e78b42d0;hpb=718db96199eb307751264e4163555662c9a389fa diff --git a/src/core/device.c b/src/core/device.c index 91e37e09d..c7bc1e25e 100644 --- a/src/core/device.c +++ b/src/core/device.c @@ -23,8 +23,6 @@ #include #include -#include "unit.h" -#include "device.h" #include "strv.h" #include "log.h" #include "unit-name.h" @@ -32,6 +30,9 @@ #include "def.h" #include "path-util.h" #include "udev-util.h" +#include "unit.h" +#include "swap.h" +#include "device.h" static const UnitActiveState state_translation_table[_DEVICE_STATE_MAX] = { [DEVICE_DEAD] = UNIT_INACTIVE, @@ -41,6 +42,7 @@ static const UnitActiveState state_translation_table[_DEVICE_STATE_MAX] = { static int device_dispatch_io(sd_event_source *source, int fd, uint32_t revents, void *userdata); static void device_unset_sysfs(Device *d) { + Hashmap *devices; Device *first; assert(d); @@ -50,13 +52,14 @@ static void device_unset_sysfs(Device *d) { /* Remove this unit from the chain of devices which share the * same sysfs path. */ - first = hashmap_get(UNIT(d)->manager->devices_by_sysfs, d->sysfs); + devices = UNIT(d)->manager->devices_by_sysfs; + first = hashmap_get(devices, d->sysfs); LIST_REMOVE(same_sysfs, first, d); if (first) - hashmap_remove_and_replace(UNIT(d)->manager->devices_by_sysfs, d->sysfs, first->sysfs, first); + hashmap_remove_and_replace(devices, d->sysfs, first->sysfs, first); else - hashmap_remove(UNIT(d)->manager->devices_by_sysfs, d->sysfs); + hashmap_remove(devices, d->sysfs); free(d->sysfs); d->sysfs = NULL; @@ -73,10 +76,10 @@ static void device_init(Unit *u) { * indefinitely for plugged in devices, something which cannot * happen for the other units since their operations time out * anyway. */ - UNIT(d)->job_timeout = u->manager->default_timeout_start_usec; + u->job_timeout = u->manager->default_timeout_start_usec; - UNIT(d)->ignore_on_isolate = true; - UNIT(d)->ignore_on_snapshot = true; + u->ignore_on_isolate = true; + u->ignore_on_snapshot = true; } static void device_done(Unit *u) { @@ -190,7 +193,7 @@ static int device_update_unit(Manager *m, struct udev_device *dev, const char *p sysfs = udev_device_get_syspath(dev); if (!sysfs) - return -ENOMEM; + return 0; r = device_find_escape_name(m, path, &u); if (r < 0) @@ -204,7 +207,7 @@ static int device_update_unit(Manager *m, struct udev_device *dev, const char *p u = unit_new(m, sizeof(Device)); if (!u) - return -ENOMEM; + return log_oom(); r = device_add_escaped_name(u, path); if (r < 0) @@ -248,30 +251,10 @@ static int device_update_unit(Manager *m, struct udev_device *dev, const char *p goto fail; if (main) { + const char *wants; + /* The additional systemd udev properties we only * interpret for the main object */ - const char *wants, *alias; - - alias = udev_device_get_property_value(dev, "SYSTEMD_ALIAS"); - if (alias) { - char *state, *w; - size_t l; - - FOREACH_WORD_QUOTED(w, l, alias, state) { - _cleanup_free_ char *e; - - e = strndup(w, l); - if (!e) { - r = -ENOMEM; - goto fail; - } - - if (is_path(e)) - device_update_unit(m, dev, e, false); - else - log_warning("SYSTEMD_ALIAS for %s is not a path, ignoring: %s", sysfs, e); - } - } wants = udev_device_get_property_value(dev, m->running_as == SYSTEMD_USER ? "SYSTEMD_USER_WANTS" : "SYSTEMD_WANTS"); if (wants) { @@ -279,15 +262,13 @@ static int device_update_unit(Manager *m, struct udev_device *dev, const char *p size_t l; FOREACH_WORD_QUOTED(w, l, wants, state) { - _cleanup_free_ char *e, *n = NULL; + _cleanup_free_ char *n = NULL; + char e[l+1]; - e = strndup(w, l); - if (!e) { - r = -ENOMEM; - goto fail; - } + memcpy(e, w, l); + e[l] = 0; - n = unit_name_mangle(e); + n = unit_name_mangle(e, MANGLE_NOGLOB); if (!n) { r = -ENOMEM; goto fail; @@ -300,6 +281,9 @@ static int device_update_unit(Manager *m, struct udev_device *dev, const char *p } } + /* Note that this won't dispatch the load queue, the caller + * has to do that if needed and appropriate */ + unit_add_to_dbus_queue(u); return 0; @@ -312,8 +296,8 @@ fail: return r; } -static int device_process_new_device(Manager *m, struct udev_device *dev, bool update_state) { - const char *sysfs, *dn; +static int device_process_new_device(Manager *m, struct udev_device *dev) { + const char *sysfs, *dn, *alias; struct udev_list_entry *item = NULL, *first = NULL; int r; @@ -321,7 +305,7 @@ static int device_process_new_device(Manager *m, struct udev_device *dev, bool u sysfs = udev_device_get_syspath(dev); if (!sysfs) - return -ENOMEM; + return 0; /* Add the main unit named after the sysfs path */ r = device_update_unit(m, dev, sysfs, true); @@ -329,7 +313,8 @@ static int device_process_new_device(Manager *m, struct udev_device *dev, bool u return r; /* Add an additional unit for the device node */ - if ((dn = udev_device_get_devnode(dev))) + dn = udev_device_get_devnode(dev); + if (dn) device_update_unit(m, dev, dn, false); /* Add additional units for all symlinks */ @@ -360,35 +345,43 @@ static int device_process_new_device(Manager *m, struct udev_device *dev, bool u device_update_unit(m, dev, p, false); } - if (update_state) { - Device *d, *l; + /* Add additional units for all explicitly configured + * aliases */ + alias = udev_device_get_property_value(dev, "SYSTEMD_ALIAS"); + if (alias) { + char *state, *w; + size_t l; - manager_dispatch_load_queue(m); + FOREACH_WORD_QUOTED(w, l, alias, state) { + char e[l+1]; + + memcpy(e, w, l); + e[l] = 0; - l = hashmap_get(m->devices_by_sysfs, sysfs); - LIST_FOREACH(same_sysfs, d, l) - device_set_state(d, DEVICE_PLUGGED); + if (path_is_absolute(e)) + device_update_unit(m, dev, e, false); + else + log_warning("SYSTEMD_ALIAS for %s is not an absolute path, ignoring: %s", sysfs, e); + } } return 0; } -static int device_process_path(Manager *m, const char *path, bool update_state) { - int r; - struct udev_device *dev; +static void device_set_path_plugged(Manager *m, struct udev_device *dev) { + const char *sysfs; + Device *d, *l; assert(m); - assert(path); + assert(dev); - dev = udev_device_new_from_syspath(m->udev, path); - if (!dev) { - log_warning("Failed to get udev device object from udev for path %s.", path); - return -ENOMEM; - } + sysfs = udev_device_get_syspath(dev); + if (!sysfs) + return; - r = device_process_new_device(m, dev, update_state); - udev_device_unref(dev); - return r; + l = hashmap_get(m->devices_by_sysfs, sysfs); + LIST_FOREACH(same_sysfs, d, l) + device_set_state(d, DEVICE_PLUGGED); } static int device_process_removed_device(Manager *m, struct udev_device *dev) { @@ -398,7 +391,8 @@ static int device_process_removed_device(Manager *m, struct udev_device *dev) { assert(m); assert(dev); - if (!(sysfs = udev_device_get_syspath(dev))) + sysfs = udev_device_get_syspath(dev); + if (!sysfs) return -ENOMEM; /* Remove all units of this sysfs path */ @@ -410,6 +404,34 @@ static int device_process_removed_device(Manager *m, struct udev_device *dev) { return 0; } +static bool device_is_ready(struct udev_device *dev) { + const char *ready; + + assert(dev); + + ready = udev_device_get_property_value(dev, "SYSTEMD_READY"); + if (!ready) + return true; + + return parse_boolean(ready) != 0; +} + +static int device_process_new_path(Manager *m, const char *path) { + _cleanup_udev_device_unref_ struct udev_device *dev = NULL; + + assert(m); + assert(path); + + dev = udev_device_new_from_syspath(m->udev, path); + if (!dev) + return log_oom(); + + if (!device_is_ready(dev)) + return 0; + + return device_process_new_device(m, dev); +} + static Unit *device_following(Unit *u) { Device *d = DEVICE(u); Device *other, *first = NULL; @@ -434,36 +456,40 @@ static Unit *device_following(Unit *u) { return UNIT(first); } -static int device_following_set(Unit *u, Set **_s) { - Device *d = DEVICE(u); - Device *other; - Set *s; +static int device_following_set(Unit *u, Set **_set) { + Device *d = DEVICE(u), *other; + Set *set; int r; assert(d); - assert(_s); + assert(_set); - if (!d->same_sysfs_prev && !d->same_sysfs_next) { - *_s = NULL; + if (LIST_JUST_US(same_sysfs, d)) { + *_set = NULL; return 0; } - if (!(s = set_new(NULL, NULL))) + set = set_new(NULL, NULL); + if (!set) return -ENOMEM; - for (other = d->same_sysfs_next; other; other = other->same_sysfs_next) - if ((r = set_put(s, other)) < 0) + LIST_FOREACH_AFTER(same_sysfs, other, d) { + r = set_put(set, other); + if (r < 0) goto fail; + } - for (other = d->same_sysfs_prev; other; other = other->same_sysfs_prev) - if ((r = set_put(s, other)) < 0) + LIST_FOREACH_BEFORE(same_sysfs, other, d) { + r = set_put(set, other); + if (r < 0) goto fail; + } - *_s = s; + *_set = set; return 1; fail: - set_free(s); + set_free(set); return r; } @@ -477,11 +503,6 @@ static void device_shutdown(Manager *m) { m->udev_monitor = NULL; } - if (m->udev) { - udev_unref(m->udev); - m->udev = NULL; - } - hashmap_free(m->devices_by_sysfs); m->devices_by_sysfs = NULL; } @@ -493,11 +514,7 @@ static int device_enumerate(Manager *m) { assert(m); - if (!m->udev) { - m->udev = udev_new(); - if (!m->udev) - return -ENOMEM; - + if (!m->udev_monitor) { m->udev_monitor = udev_monitor_new_from_netlink(m->udev, "udev"); if (!m->udev_monitor) { r = -ENOMEM; @@ -532,13 +549,17 @@ static int device_enumerate(Manager *m) { if (r < 0) goto fail; + r = udev_enumerate_add_match_is_initialized(e); + if (r < 0) + goto fail; + r = udev_enumerate_scan_devices(e); if (r < 0) goto fail; first = udev_enumerate_get_list_entry(e); udev_list_entry_foreach(item, first) - device_process_path(m, udev_list_entry_get_name(item), false); + device_process_new_path(m, udev_list_entry_get_name(item)); return 0; @@ -549,8 +570,8 @@ fail: static int device_dispatch_io(sd_event_source *source, int fd, uint32_t revents, void *userdata) { _cleanup_udev_device_unref_ struct udev_device *dev = NULL; - const char *action, *ready; Manager *m = userdata; + const char *action; int r; assert(m); @@ -578,16 +599,27 @@ static int device_dispatch_io(sd_event_source *source, int fd, uint32_t revents, return 0; } - ready = udev_device_get_property_value(dev, "SYSTEMD_READY"); - - if (streq(action, "remove") || (ready && parse_boolean(ready) == 0)) { + if (streq(action, "remove") || !device_is_ready(dev)) { r = device_process_removed_device(m, dev); if (r < 0) - log_error("Failed to process udev device event: %s", strerror(-r)); + log_error("Failed to process device remove event: %s", strerror(-r)); + + r = swap_process_removed_device(m, dev); + if (r < 0) + log_error("Failed to process swap device remove event: %s", strerror(-r)); + } else { - r = device_process_new_device(m, dev, true); + r = device_process_new_device(m, dev); if (r < 0) - log_error("Failed to process udev device event: %s", strerror(-r)); + log_error("Failed to process device new event: %s", strerror(-r)); + + r = swap_process_new_device(m, dev); + if (r < 0) + log_error("Failed to process swap device new event: %s", strerror(-r)); + + manager_dispatch_load_queue(m); + + device_set_path_plugged(m, dev); } return 0; @@ -621,7 +653,6 @@ const UnitVTable device_vtable = { .sub_state_to_string = device_sub_state_to_string, .bus_interface = "org.freedesktop.systemd1.Device", - .bus_changing_properties = bus_device_changing_properties, .bus_vtable = bus_device_vtable, .following = device_following,