X-Git-Url: http://git.ozlabs.org/?p=petitboot;a=blobdiff_plain;f=discover%2Fudev.c;h=6cc718ea4614f9da6e7110fd9901144c5f37fb7c;hp=b1a25b3e639925ee753b53ad4830c9db75e2a7ba;hb=70907c3fc4b344d00ce900e6a536e9cc16a35216;hpb=c70e102425ecc261dc9fa740ce7e2296c5d39f4e diff --git a/discover/udev.c b/discover/udev.c index b1a25b3..6cc718e 100644 --- a/discover/udev.c +++ b/discover/udev.c @@ -25,6 +25,12 @@ #include "pb-discover.h" #include "device-handler.h" #include "cdrom.h" +#include "devmapper.h" + +/* We set a default monitor buffer size, as we may not process monitor + * events while performing device discvoery. systemd uses a 128M buffer, so + * we'll do the same here */ +static const int monitor_bufsize = 128 * 1024 * 1024; struct pb_udev { struct udev *udev; @@ -36,11 +42,15 @@ static int udev_destructor(void *p) { struct pb_udev *udev = p; - udev_monitor_unref(udev->monitor); - udev->monitor = NULL; + if (udev->monitor) { + udev_monitor_unref(udev->monitor); + udev->monitor = NULL; + } - udev_unref(udev->udev); - udev->udev = NULL; + if (udev->udev) { + udev_unref(udev->udev); + udev->udev = NULL; + } return 0; } @@ -64,13 +74,21 @@ static int udev_handle_block_add(struct pb_udev *udev, struct udev_device *dev, const char *name) { struct discover_device *ddev; + unsigned int i = 0; const char *typestr; const char *uuid; const char *path; const char *node; const char *prop; const char *type; - bool cdrom; + const char *devname; + const char *ignored_types[] = { + "linux_raid_member", + "swap", + "LVM2_member", + NULL, + }; + bool cdrom, usb; typestr = udev_device_get_devtype(dev); if (!typestr) { @@ -85,12 +103,16 @@ static int udev_handle_block_add(struct pb_udev *udev, struct udev_device *dev, node = udev_device_get_devnode(dev); path = udev_device_get_devpath(dev); - if (path && (strstr(path, "virtual/block/loop") - || strstr(path, "virtual/block/ram"))) { + if (path && strstr(path, "virtual/block/loop")) { pb_log("SKIP: %s: ignored (path=%s)\n", name, path); return 0; } + if (path && strstr(path, "virtual/block/ram")) { + device_handler_add_ramdisk(udev->handler, node); + return 0; + } + cdrom = node && !!udev_device_get_property_value(dev, "ID_CDROM"); if (cdrom) { /* CDROMs require a little initialisation, to get @@ -102,12 +124,28 @@ static int udev_handle_block_add(struct pb_udev *udev, struct udev_device *dev, } } + /* If our environment's udev can recognise them explictly skip any + * device mapper devices we encounter */ + devname = udev_device_get_property_value(dev, "DM_NAME"); + if (devname) { + pb_debug("SKIP: dm-device %s\n", devname); + return 0; + } + type = udev_device_get_property_value(dev, "ID_FS_TYPE"); if (!type) { pb_log("SKIP: %s: no ID_FS_TYPE property\n", name); return 0; } + while (ignored_types[i]) { + if (!strncmp(type, ignored_types[i], strlen(ignored_types[i]))) { + pb_log("SKIP: %s: ignore '%s' filesystem\n", name, type); + return 0; + } + i++; + } + /* We may see multipath devices; they'll have the same uuid as an * existing device, so only parse the first. */ uuid = udev_device_get_property_value(dev, "ID_FS_UUID"); @@ -116,7 +154,7 @@ static int udev_handle_block_add(struct pb_udev *udev, struct udev_device *dev, if (ddev) { pb_log("SKIP: %s UUID [%s] already present (as %s)\n", name, uuid, ddev->device->id); - return -1; + return 0; } } @@ -129,10 +167,21 @@ static int udev_handle_block_add(struct pb_udev *udev, struct udev_device *dev, prop = udev_device_get_property_value(dev, "ID_FS_LABEL"); if (prop) ddev->label = talloc_strdup(ddev, prop); - ddev->device->type = cdrom ? DEVICE_TYPE_OPTICAL : DEVICE_TYPE_DISK; + + usb = !!udev_device_get_property_value(dev, "ID_USB_DRIVER"); + if (cdrom) + ddev->device->type = DEVICE_TYPE_OPTICAL; + else + ddev->device->type = usb ? DEVICE_TYPE_USB : DEVICE_TYPE_DISK; udev_setup_device_params(dev, ddev); + /* Create a snapshot for all disks, unless it is an assembled RAID array */ + if ((ddev->device->type == DEVICE_TYPE_DISK || + ddev->device->type == DEVICE_TYPE_USB) && + !udev_device_get_property_value(dev, "MD_LEVEL")) + devmapper_init_snapshot(udev->handler, ddev); + device_handler_discover(udev->handler, ddev); return 0; @@ -258,23 +307,18 @@ static int udev_handle_dev_change(struct pb_udev *udev, struct udev_device *dev) static int udev_handle_dev_action(struct udev_device *dev, const char *action) { struct pb_udev *udev = udev_get_userdata(udev_device_get_udev(dev)); + struct udev_list_entry *list; + const char *name; -#ifdef DEBUG - { - struct udev_list_entry *list; - const char *name; - - list = udev_device_get_properties_list_entry(dev); - name = udev_device_get_sysname(dev); + list = udev_device_get_properties_list_entry(dev); + name = udev_device_get_sysname(dev); - pb_debug("%s: action %s, device %s\n", __func__, action, name); - pb_debug("%s properties:\n", __func__); + pb_debug("udev: action %s, device %s\n", action, name); + pb_debug("udev: properties:\n"); - for (; list; list = udev_list_entry_get_next(list)) - pb_log("\t%-20s: %s\n", udev_list_entry_get_name(list), - udev_list_entry_get_value(list)); - } while (0); -#endif + for (; list; list = udev_list_entry_get_next(list)) + pb_debug("\t%-20s: %s\n", udev_list_entry_get_name(list), + udev_list_entry_get_value(list)); if (!strcmp(action, "add")) return udev_handle_dev_add(udev, dev); @@ -317,11 +361,6 @@ static int udev_enumerate(struct udev *udev) list = udev_enumerate_get_list_entry(enumerate); - if (!list) { - pb_log("udev_enumerate_get_list_entry failed\n"); - goto fail; - } - udev_list_entry_foreach(entry, list) { const char *syspath; struct udev_device *dev; @@ -355,6 +394,12 @@ static int udev_setup_monitor(struct udev *udev, struct udev_monitor **monitor) goto out_err; } + result = udev_monitor_set_receive_buffer_size(m, monitor_bufsize); + if (result) { + pb_log("udev_monitor_set_rx_bufsize(%d) failed\n", + monitor_bufsize); + } + result = udev_monitor_filter_add_match_subsystem_devtype(m, "block", NULL); @@ -387,10 +432,8 @@ static int udev_process(void *arg) struct udev_monitor *monitor = arg; struct udev_device *dev; const char *action; - int result; dev = udev_monitor_receive_device(monitor); - if (!dev) { pb_log("udev_monitor_receive_device failed\n"); return -1; @@ -400,17 +443,12 @@ static int udev_process(void *arg) if (!action) { pb_log("udev_device_get_action failed\n"); - goto fail; + } else { + udev_handle_dev_action(dev, action); } - result = udev_handle_dev_action(dev, action); - udev_device_unref(dev); - return result; - -fail: - udev_device_unref(dev); - return -1; + return 0; } static void udev_log_fn(struct udev __attribute__((unused)) *udev, @@ -421,12 +459,13 @@ static void udev_log_fn(struct udev __attribute__((unused)) *udev, vfprintf(pb_log_get_stream(), format, args); } -struct pb_udev *udev_init(struct waitset *waitset, - struct device_handler *handler) +struct pb_udev *udev_init(struct device_handler *handler, + struct waitset *waitset) { + struct pb_udev *udev; int result; - struct pb_udev *udev = talloc(NULL, struct pb_udev); + udev = talloc_zero(handler, struct pb_udev); talloc_set_destructor(udev, udev_destructor); udev->handler = handler; @@ -434,7 +473,7 @@ struct pb_udev *udev_init(struct waitset *waitset, if (!udev->udev) { pb_log("udev_new failed\n"); - goto fail_new; + goto fail; } udev_set_userdata(udev->udev, udev); @@ -443,11 +482,11 @@ struct pb_udev *udev_init(struct waitset *waitset, result = udev_setup_monitor(udev->udev, &udev->monitor); if (result) - goto fail_monitor; + goto fail; result = udev_enumerate(udev->udev); if (result) - goto fail_enumerate; + goto fail; waiter_register_io(waitset, udev_monitor_get_fd(udev->monitor), WAIT_IN, udev_process, udev->monitor); @@ -456,16 +495,13 @@ struct pb_udev *udev_init(struct waitset *waitset, return udev; -fail_monitor: - udev_monitor_unref(udev->monitor); -fail_enumerate: - udev_unref(udev->udev); -fail_new: +fail: talloc_free(udev); return NULL; } -void udev_destroy(struct pb_udev *udev) +void udev_reinit(struct pb_udev *udev) { - talloc_free(udev); + pb_log("udev: reinit requested, starting enumeration\n"); + udev_enumerate(udev->udev); }