[PATCH] move kernel name/number evaluation into udev_init_device()

diff --git a/namedev.c b/namedev.c
index a5c711bdf65dd282d95ca56f74d5bca7f9856ab9..0463d4df0b6625dc03b4570eae8c1e71fbe96070 100644 (file)
--- a/namedev.c
+++ b/namedev.c
@@ -343,19 +343,6 @@ static void apply_format(struct udevice *udev, char *string, size_t maxsize,
        }
 }
 
        }
 }
 

-static void fix_kernel_name(struct udevice *udev)
-{
-       char *temp = udev->kernel_name;
-
-       while (*temp != 0x00) {
-               /* Some block devices have a ! in their name, 
-                * we need to change that to / */
-               if (*temp == '!')
-                       *temp = '/';
-               ++temp;
-       }
-}
-

 static int execute_program(struct udevice *udev, const char *path, char *value, int len)
 {
        int retval;
 static int execute_program(struct udevice *udev, const char *path, char *value, int len)
 {
        int retval;


@@ -549,8 +536,8 @@ static int match_id(struct config_device *dev, struct sysfs_class_device *class_
        dbg("search '%s' in '%s', path='%s'", dev->id, temp, path);
        if (strcmp_pattern(dev->id, temp) != 0)
                return -ENODEV;
        dbg("search '%s' in '%s', path='%s'", dev->id, temp, path);
        if (strcmp_pattern(dev->id, temp) != 0)
                return -ENODEV;

-       else
-               return 0;
+
+       return 0;

 }
 
 static int match_place(struct config_device *dev, struct sysfs_class_device *class_dev, struct sysfs_device *sysfs_device)
 }
 
 static int match_place(struct config_device *dev, struct sysfs_class_device *class_dev, struct sysfs_device *sysfs_device)


@@ -582,7 +569,6 @@ static int match_place(struct config_device *dev, struct sysfs_class_device *cla
 static int match_rule(struct udevice *udev, struct config_device *dev,
                      struct sysfs_class_device *class_dev, struct sysfs_device *sysfs_device)
 {
 static int match_rule(struct udevice *udev, struct config_device *dev,
                      struct sysfs_class_device *class_dev, struct sysfs_device *sysfs_device)
 {

-       /* check for matching kernel name */

        if (dev->kernel[0] != '\0') {
                dbg("check for " FIELD_KERNEL " dev->kernel='%s' class_dev->name='%s'",
                    dev->kernel, class_dev->name);
        if (dev->kernel[0] != '\0') {
                dbg("check for " FIELD_KERNEL " dev->kernel='%s' class_dev->name='%s'",
                    dev->kernel, class_dev->name);


@@ -593,7 +579,6 @@ static int match_rule(struct udevice *udev, struct config_device *dev,
                dbg(FIELD_KERNEL " matches");
        }
 
                dbg(FIELD_KERNEL " matches");
        }
 

-       /* check for matching subsystem */

        if (dev->subsystem[0] != '\0') {
                dbg("check for " FIELD_SUBSYSTEM " dev->subsystem='%s' class_dev->name='%s'",
                    dev->subsystem, class_dev->name);
        if (dev->subsystem[0] != '\0') {
                dbg("check for " FIELD_SUBSYSTEM " dev->subsystem='%s' class_dev->name='%s'",
                    dev->subsystem, class_dev->name);


@@ -710,7 +695,6 @@ int namedev_name_device(struct udevice *udev, struct sysfs_class_device *class_d
        struct sysfs_class_device *class_dev_parent;
        struct sysfs_device *sysfs_device = NULL;
        struct config_device *dev;
        struct sysfs_class_device *class_dev_parent;
        struct sysfs_device *sysfs_device = NULL;
        struct config_device *dev;

-       char *pos;

 
        dbg("class_dev->name='%s'", class_dev->name);
 
 
        dbg("class_dev->name='%s'", class_dev->name);
 


@@ -733,16 +717,7 @@ int namedev_name_device(struct udevice *udev, struct sysfs_class_device *class_d
                strfieldcpy(udev->bus_id, sysfs_device->bus_id);
        }
 
                strfieldcpy(udev->bus_id, sysfs_device->bus_id);
        }
 

-       strfieldcpy(udev->kernel_name, class_dev->name);
-       fix_kernel_name(udev);
-       dbg("udev->kernel_name = '%s'", udev->kernel_name);
-
-       /* get kernel number */
-       pos = class_dev->name + strlen(class_dev->name);
-       while (isdigit(*(pos-1)))
-               pos--;
-       strfieldcpy(udev->kernel_number, pos);
-       dbg("kernel_number='%s'", udev->kernel_number);
+       dbg("udev->kernel_name='%s'", udev->kernel_name);

 
        /* look for a matching rule to apply */
        list_for_each_entry(dev, &config_device_list, node) {
 
        /* look for a matching rule to apply */
        list_for_each_entry(dev, &config_device_list, node) {


@@ -781,7 +756,7 @@ int namedev_name_device(struct udevice *udev, struct sysfs_class_device *class_d
                                dbg("applied group='%s' to '%s'", udev->group, udev->kernel_name);
                        }
 
                                dbg("applied group='%s' to '%s'", udev->group, udev->kernel_name);
                        }
 

-                       /* collect symlinks for this or the final matching rule */
+                       /* collect symlinks */

                        if (dev->symlink[0] != '\0') {
                                char temp[NAME_SIZE];
 
                        if (dev->symlink[0] != '\0') {
                                char temp[NAME_SIZE];
 

diff --git a/namedev.h b/namedev.h
index a929e527561ba5ee0a5dd049187824c32a728654..3a161441696eeb575c103afc82c75795cffad3af 100644 (file)
--- a/namedev.h
+++ b/namedev.h
@@ -36,14 +36,14 @@ struct sysfs_class_device;
 #define DRIVER_SIZE            64
 #define PROGRAM_SIZE           128
 
 #define DRIVER_SIZE            64
 #define PROGRAM_SIZE           128
 

+#define FIELD_KERNEL           "KERNEL"
+#define FIELD_SUBSYSTEM                "SUBSYSTEM"

 #define FIELD_BUS              "BUS"
 #define FIELD_SYSFS            "SYSFS"
 #define FIELD_ID               "ID"
 #define FIELD_PLACE            "PLACE"
 #define FIELD_PROGRAM          "PROGRAM"
 #define FIELD_RESULT           "RESULT"
 #define FIELD_BUS              "BUS"
 #define FIELD_SYSFS            "SYSFS"
 #define FIELD_ID               "ID"
 #define FIELD_PLACE            "PLACE"
 #define FIELD_PROGRAM          "PROGRAM"
 #define FIELD_RESULT           "RESULT"

-#define FIELD_KERNEL           "KERNEL"
-#define FIELD_SUBSYSTEM                "SUBSYSTEM"

 #define FIELD_DRIVER           "DRIVER"
 #define FIELD_NAME             "NAME"
 #define FIELD_SYMLINK          "SYMLINK"
 #define FIELD_DRIVER           "DRIVER"
 #define FIELD_NAME             "NAME"
 #define FIELD_SYMLINK          "SYMLINK"

diff --git a/udev.h b/udev.h
index 17ff4031c90dde49b00031d1529156598fb450d9..d6dbd7c3325dcd00dbd47562fd22eeac52b47a3f 100644 (file)
--- a/udev.h
+++ b/udev.h
@@ -26,6 +26,7 @@
 #include <sys/types.h>
 #include <sys/param.h>
 #include "libsysfs/sysfs/libsysfs.h"
 #include <sys/types.h>
 #include <sys/param.h>
 #include "libsysfs/sysfs/libsysfs.h"

+#include "list.h"

 
 #define ALARM_TIMEOUT                  120
 #define COMMENT_CHARACTER              '#'
 
 #define ALARM_TIMEOUT                  120
 #define COMMENT_CHARACTER              '#'

diff --git a/udev_utils.c b/udev_utils.c
index e6580d2c0acdca0404e17b2cddb1eee19ec0ad6d..ea4aa25c90310a3e49d4d1d16b10e053bbce200c 100644 (file)
--- a/udev_utils.c
+++ b/udev_utils.c
@@ -37,12 +37,16 @@
 #include "list.h"
 
 
 #include "list.h"
 
 

-void udev_init_device(struct udevice *udev, const char* devpath, const char *subsystem)
+int udev_init_device(struct udevice *udev, const char* devpath, const char *subsystem)

 {
 {

+       char *pos;
+

        memset(udev, 0x00, sizeof(struct udevice));
 
        memset(udev, 0x00, sizeof(struct udevice));
 

-       if (devpath)
+       if (devpath) {

                strfieldcpy(udev->devpath, devpath);
                strfieldcpy(udev->devpath, devpath);

+               no_trailing_slash(udev->devpath);
+       }

        if (subsystem)
                strfieldcpy(udev->subsystem, subsystem);
 
        if (subsystem)
                strfieldcpy(udev->subsystem, subsystem);
 


@@ -60,6 +64,30 @@ void udev_init_device(struct udevice *udev, const char* devpath, const char *sub
        udev->mode = 0660;
        strcpy(udev->owner, "root");
        strcpy(udev->group, "root");
        udev->mode = 0660;
        strcpy(udev->owner, "root");
        strcpy(udev->group, "root");

+
+       /* get kernel name */
+       pos = strrchr(udev->devpath, '/');
+       if (pos == NULL)
+               return -1;
+       strfieldcpy(udev->kernel_name, &pos[1]);
+
+       /* get kernel number */
+       pos = &udev->kernel_name[strlen(udev->kernel_name)];
+       while (isdigit(pos[-1]))
+               pos--;
+       strfieldcpy(udev->kernel_number, pos);
+       dbg("kernel_number='%s'", udev->kernel_number);
+
+       /* Some block devices have '!' in their name, change that to '/' */
+       pos = udev->kernel_name;
+       while (pos[0] != '\0') {
+               if (pos[0] == '!')
+                       pos[0] = '/';
+               pos++;
+       }
+
+       dbg("kernel_name='%s'", udev->kernel_name);
+       return 0;

 }
 
 int kernel_release_satisfactory(unsigned int version, unsigned int patchlevel, unsigned int sublevel)
 }
 
 int kernel_release_satisfactory(unsigned int version, unsigned int patchlevel, unsigned int sublevel)


@@ -232,31 +260,35 @@ void no_trailing_slash(char *path)
                path[--len] = '\0';
 }
 
                path[--len] = '\0';
 }
 

-struct files {
-       struct list_head list;
+struct name_entry {
+       struct list_head node;

        char name[NAME_SIZE];
 };
 
 /* sort files in lexical order */
        char name[NAME_SIZE];
 };
 
 /* sort files in lexical order */

-static int file_list_insert(char *filename, struct list_head *file_list)
+static int name_list_add(struct list_head *name_list, const char *name, int sort)

 {
 {

-       struct files *loop_file;
-       struct files *new_file;
-
-       list_for_each_entry(loop_file, file_list, list) {
-               if (strcmp(loop_file->name, filename) > 0) {
-                       break;
+       struct name_entry *loop_name;
+       struct name_entry *new_name;
+
+       list_for_each_entry(loop_name, name_list, node) {
+               /* avoid doubles */
+               if (strcmp(loop_name->name, name) == 0) {
+                       dbg("'%s' is already in the list", name);
+                       return 0;

                }
                }

+               if (sort && strcmp(loop_name->name, name) > 0)
+                       break;

        }
 
        }
 

-       new_file = malloc(sizeof(struct files));
-       if (new_file == NULL) {
+       new_name = malloc(sizeof(struct name_entry));
+       if (new_name == NULL) {

                dbg("error malloc");
                return -ENOMEM;
        }
 
                dbg("error malloc");
                return -ENOMEM;
        }
 

-       strfieldcpy(new_file->name, filename);
-       list_add_tail(&new_file->list, &loop_file->list);
+       strfieldcpy(new_name->name, name);
+       list_add_tail(&new_name->node, &loop_name->node);

        return 0;
 }
 
        return 0;
 }
 


@@ -267,8 +299,8 @@ int call_foreach_file(file_fnct_t fnct, const char *dirname,
        struct dirent *ent;
        DIR *dir;
        char *ext;
        struct dirent *ent;
        DIR *dir;
        char *ext;

-       struct files *loop_file;
-       struct files *tmp_file;
+       struct name_entry *loop_file;
+       struct name_entry *tmp_file;

        LIST_HEAD(file_list);
 
        dbg("open directory '%s'", dirname);
        LIST_HEAD(file_list);
 
        dbg("open directory '%s'", dirname);


@@ -295,11 +327,11 @@ int call_foreach_file(file_fnct_t fnct, const char *dirname,
                        continue;
 
                dbg("put file '%s/%s' in list", dirname, ent->d_name);
                        continue;
 
                dbg("put file '%s/%s' in list", dirname, ent->d_name);

-               file_list_insert(ent->d_name, &file_list);
+               name_list_add(&file_list, ent->d_name, 1);

        }
 
        /* call function for every file in the list */
        }
 
        /* call function for every file in the list */

-       list_for_each_entry_safe(loop_file, tmp_file, &file_list, list) {
+       list_for_each_entry_safe(loop_file, tmp_file, &file_list, node) {

                char filename[NAME_SIZE];
 
                snprintf(filename, NAME_SIZE, "%s/%s", dirname, loop_file->name);
                char filename[NAME_SIZE];
 
                snprintf(filename, NAME_SIZE, "%s/%s", dirname, loop_file->name);


@@ -307,7 +339,7 @@ int call_foreach_file(file_fnct_t fnct, const char *dirname,
 
                fnct(filename, data);
 
 
                fnct(filename, data);
 

-               list_del(&loop_file->list);
+               list_del(&loop_file->node);

                free(loop_file);
        }
 
                free(loop_file);
        }
 

diff --git a/udev_utils.h b/udev_utils.h
index 19061f16f659c2ef48cb48fff8e44cda73021e07..268c2075de9b9286274676d7378d2d49efe38924 100644 (file)
--- a/udev_utils.h
+++ b/udev_utils.h
@@ -76,7 +76,7 @@ do { \
 # define asmlinkage    /* nothing */
 #endif
 
 # define asmlinkage    /* nothing */
 #endif
 

-extern void udev_init_device(struct udevice *udev, const char* devpath, const char *subsystem);
+extern int udev_init_device(struct udevice *udev, const char* devpath, const char *subsystem);

 extern int kernel_release_satisfactory(unsigned int version, unsigned int patchlevel, unsigned int sublevel);
 extern int create_path(const char *path);
 extern int parse_get_pair(char **orig_string, char **left, char **right);
 extern int kernel_release_satisfactory(unsigned int version, unsigned int patchlevel, unsigned int sublevel);
 extern int create_path(const char *path);
 extern int parse_get_pair(char **orig_string, char **left, char **right);