From: kay.sievers@vrfy.org Date: Wed, 9 Feb 2005 07:43:18 +0000 (+0100) Subject: [PATCH] create /block/*/range count of partitons for all_partitions X-Git-Tag: 052~6 X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=6d56416665863dc2b4fb107dbdc779e4791bdf9e;ds=sidebyside [PATCH] create /block/*/range count of partitons for all_partitions --- diff --git a/namedev.h b/namedev.h index 8a3569ec7..43381bfbd 100644 --- a/namedev.h +++ b/namedev.h @@ -53,7 +53,6 @@ struct sysfs_class_device; #define ATTR_PARTITIONS "all_partitions" #define ATTR_IGNORE_REMOVE "ignore_remove" -#define PARTITIONS_COUNT 15 #define MAX_SYSFS_PAIRS 5 @@ -91,6 +90,7 @@ extern struct list_head config_device_list; extern int namedev_init(void); extern int namedev_name_device(struct udevice *udev, struct sysfs_class_device *class_dev); +extern void namedev_close(void); extern void dump_config_dev(struct config_device *dev); extern void dump_config_dev_list(void); diff --git a/namedev_parse.c b/namedev_parse.c index 5bd59d83f..7b5b96302 100644 --- a/namedev_parse.c +++ b/namedev_parse.c @@ -250,7 +250,7 @@ static int namedev_parse(const char *filename, void *data) if (attr != NULL) { if (strstr(attr, ATTR_PARTITIONS) != NULL) { dbg_parse("creation of partition nodes requested"); - dev.partitions = PARTITIONS_COUNT; + dev.partitions = DEFAULT_PARTITIONS_COUNT; } if (strstr(attr, ATTR_IGNORE_REMOVE) != NULL) { dbg_parse("remove event should be ignored"); @@ -338,3 +338,14 @@ int namedev_init(void) return retval; } + +void namedev_close(void) +{ + struct config_device *dev; + + list_for_each_entry(dev, &config_device_list, node) { + list_del(&dev->node); + free(dev); + } +} + diff --git a/udev.h b/udev.h index 5beec6455..c2f9859c5 100644 --- a/udev.h +++ b/udev.h @@ -46,6 +46,8 @@ #define HOTPLUGD_DIR "/etc/hotplug.d" #define HOTPLUG_SUFFIX ".hotplug" +#define DEFAULT_PARTITIONS_COUNT 15 + struct udevice { char devpath[DEVPATH_SIZE]; char subsystem[SUBSYSTEM_SIZE]; diff --git a/udev_add.c b/udev_add.c index a495902ab..c566571c1 100644 --- a/udev_add.c +++ b/udev_add.c @@ -117,16 +117,16 @@ exit: return retval; } -static int create_node(struct udevice *udev) +static int create_node(struct udevice *udev, struct sysfs_class_device *class_dev) { char filename[NAME_SIZE]; char partitionname[NAME_SIZE]; uid_t uid = 0; gid_t gid = 0; - int i; int tail; char *pos; int len; + int i; snprintf(filename, NAME_SIZE, "%s/%s", udev_root, udev->name); filename[NAME_SIZE-1] = '\0'; @@ -154,6 +154,7 @@ static int create_node(struct udevice *udev) if (udev->owner[0] != '\0') { char *endptr; unsigned long id = strtoul(udev->owner, &endptr, 10); + if (endptr[0] == '\0') uid = (uid_t) id; else { @@ -170,6 +171,7 @@ static int create_node(struct udevice *udev) if (udev->group[0] != '\0') { char *endptr; unsigned long id = strtoul(udev->group, &endptr, 10); + if (endptr[0] == '\0') gid = (gid_t) id; else { @@ -193,6 +195,16 @@ static int create_node(struct udevice *udev) /* create all_partitions if requested */ if (udev->partitions > 0) { + struct sysfs_attribute *attr; + int range; + + /* take the maximum registered minor range */ + attr = sysfs_get_classdev_attr(class_dev, "range"); + if (attr) { + range = atoi(attr->value); + if (range > 1) + udev->partitions = range-1; + } info("creating device partition nodes '%s[1-%i]'", filename, udev->partitions); if (!udev->test_run) { for (i = 1; i <= udev->partitions; i++) { @@ -298,7 +310,7 @@ int udev_add_device(struct udevice *udev, struct sysfs_class_device *class_dev) selinux_init(); if (udev->type == 'b' || udev->type == 'c') { - retval = create_node(udev); + retval = create_node(udev, class_dev); if (retval != 0) goto exit; diff --git a/udev_remove.c b/udev_remove.c index e1af3dbef..172ec1269 100644 --- a/udev_remove.c +++ b/udev_remove.c @@ -89,7 +89,7 @@ static int delete_node(struct udevice *udev) num = udev->partitions; if (num > 0) { info("removing all_partitions '%s[1-%i]'", filename, num); - if (num > PARTITIONS_COUNT) { + if (num > 255) { info("garbage from udev database, skip all_partitions removal"); return -1; }