[PATCH] make struct config_device contain a struct device_attr instead of duplicating...

[elogind.git] / namedev.c

diff --git a/namedev.c b/namedev.c
index 0271ba55a2ad4091f382c506c87a559aa456dd57..8b7edb7fb1a5ce1f43cabd99959c803e60eab4cb 100644 (file)
--- a/namedev.c
+++ b/namedev.c
@@ -34,50 +34,13 @@
 #include "udev.h"
 #include "udev_version.h"
 #include "namedev.h"
 #include "udev.h"
 #include "udev_version.h"
 #include "namedev.h"

+#include "libsysfs/libsysfs.h"

 
 #define TYPE_LABEL     "LABEL"
 #define TYPE_NUMBER    "NUMBER"
 #define TYPE_TOPOLOGY  "TOPOLOGY"
 #define TYPE_REPLACE   "REPLACE"
 
 
 #define TYPE_LABEL     "LABEL"
 #define TYPE_NUMBER    "NUMBER"
 #define TYPE_TOPOLOGY  "TOPOLOGY"
 #define TYPE_REPLACE   "REPLACE"
 

-enum config_type {
-       KERNEL_NAME     = 0,    /* must be 0 to let memset() default to this value */
-       LABEL           = 1,
-       NUMBER          = 2,
-       TOPOLOGY        = 3,
-       REPLACE         = 4,
-};
-
-#define BUS_SIZE       30
-#define FILE_SIZE      50
-#define VALUE_SIZE     100
-#define ID_SIZE                50
-#define PLACE_SIZE     50
-#define NAME_SIZE      100
-#define OWNER_SIZE     30
-#define GROUP_SIZE     30
-
-
-struct config_device {
-       struct list_head node;
-
-       enum config_type type;
-
-       char bus[BUS_SIZE];
-       char sysfs_file[FILE_SIZE];
-       char sysfs_value[VALUE_SIZE];
-       char id[ID_SIZE];
-       char place[PLACE_SIZE];
-       char kernel_name[NAME_SIZE];
-       
-       /* what to set the device to */
-       int mode;
-       char name[NAME_SIZE];
-       char owner[OWNER_SIZE];
-       char group[GROUP_SIZE];
-};
-
-

 static LIST_HEAD(config_device_list);
 
 static void dump_dev(struct config_device *dev)
 static LIST_HEAD(config_device_list);
 
 static void dump_dev(struct config_device *dev)


@@ -85,33 +48,33 @@ static void dump_dev(struct config_device *dev)
        switch (dev->type) {
        case KERNEL_NAME:
                dbg("KERNEL name ='%s'"
        switch (dev->type) {
        case KERNEL_NAME:
                dbg("KERNEL name ='%s'"

-                       " owner = '%s', group = '%s', mode = '%d'", 
-                       dev->name, 
-                       dev->owner, dev->group, dev->mode);
+                       " owner = '%s', group = '%s', mode = '%#o'",
+                       dev->attr.name, 
+                       dev->attr.owner, dev->attr.group, dev->attr.mode);

                break;
        case LABEL:
                dbg("LABEL name = '%s', bus = '%s', sysfs_file = '%s', sysfs_value = '%s'"
                break;
        case LABEL:
                dbg("LABEL name = '%s', bus = '%s', sysfs_file = '%s', sysfs_value = '%s'"

-                       " owner = '%s', group = '%s', mode = '%d'",
-                       dev->name, dev->bus, dev->sysfs_file, dev->sysfs_value,
-                       dev->owner, dev->group, dev->mode);
+                       " owner = '%s', group = '%s', mode = '%#o'",
+                       dev->attr.name, dev->bus, dev->sysfs_file, dev->sysfs_value,
+                       dev->attr.owner, dev->attr.group, dev->attr.mode);

                break;
        case NUMBER:
                dbg("NUMBER name = '%s', bus = '%s', id = '%s'"
                break;
        case NUMBER:
                dbg("NUMBER name = '%s', bus = '%s', id = '%s'"

-                       " owner = '%s', group = '%s', mode = '%d'",
-                       dev->name, dev->bus, dev->id,
-                       dev->owner, dev->group, dev->mode);
+                       " owner = '%s', group = '%s', mode = '%#o'",
+                       dev->attr.name, dev->bus, dev->id,
+                       dev->attr.owner, dev->attr.group, dev->attr.mode);

                break;
        case TOPOLOGY:
                dbg("TOPOLOGY name = '%s', bus = '%s', place = '%s'"
                break;
        case TOPOLOGY:
                dbg("TOPOLOGY name = '%s', bus = '%s', place = '%s'"

-                       " owner = '%s', group = '%s', mode = '%d'",
-                       dev->name, dev->bus, dev->place,
-                       dev->owner, dev->group, dev->mode);
+                       " owner = '%s', group = '%s', mode = '%#o'",
+                       dev->attr.name, dev->bus, dev->place,
+                       dev->attr.owner, dev->attr.group, dev->attr.mode);

                break;
        case REPLACE:
                dbg("REPLACE name = %s, kernel_name = %s"
                break;
        case REPLACE:
                dbg("REPLACE name = %s, kernel_name = %s"

-                       " owner = '%s', group = '%s', mode = '%d'",
-                       dev->name, dev->kernel_name,
-                       dev->owner, dev->group, dev->mode);
+                       " owner = '%s', group = '%s', mode = '%#o'",
+                       dev->attr.name, dev->kernel_name,
+                       dev->attr.owner, dev->attr.group, dev->attr.mode);

                break;
        default:
                dbg("Unknown type of device!");
                break;
        default:
                dbg("Unknown type of device!");


@@ -135,18 +98,18 @@ static int add_dev(struct config_device *new_dev)
         * this one... */
        list_for_each(tmp, &config_device_list) {
                struct config_device *dev = list_entry(tmp, struct config_device, node);
         * this one... */
        list_for_each(tmp, &config_device_list) {
                struct config_device *dev = list_entry(tmp, struct config_device, node);

-               if (strcmp(dev->name, new_dev->name) == 0) {
+               if (strcmp(dev->attr.name, new_dev->attr.name) == 0) {

                        /* the same, copy the new info into this structure */
                        copy_var(dev, new_dev, type);
                        /* the same, copy the new info into this structure */
                        copy_var(dev, new_dev, type);

-                       copy_var(dev, new_dev, mode);
+                       copy_var(dev, new_dev, attr.mode);

                        copy_string(dev, new_dev, bus);
                        copy_string(dev, new_dev, sysfs_file);
                        copy_string(dev, new_dev, sysfs_value);
                        copy_string(dev, new_dev, id);
                        copy_string(dev, new_dev, place);
                        copy_string(dev, new_dev, kernel_name);
                        copy_string(dev, new_dev, bus);
                        copy_string(dev, new_dev, sysfs_file);
                        copy_string(dev, new_dev, sysfs_value);
                        copy_string(dev, new_dev, id);
                        copy_string(dev, new_dev, place);
                        copy_string(dev, new_dev, kernel_name);

-                       copy_string(dev, new_dev, owner);
-                       copy_string(dev, new_dev, group);
+                       copy_string(dev, new_dev, attr.owner);
+                       copy_string(dev, new_dev, attr.group);

                        return 0;
                }
        }
                        return 0;
                }
        }


@@ -288,9 +251,9 @@ static int namedev_init_config(void)
                        retval = get_value("NAME", &temp, &temp3);
                        if (retval)
                                continue;
                        retval = get_value("NAME", &temp, &temp3);
                        if (retval)
                                continue;

-                       strcpy(dev.name, temp3);
+                       strcpy(dev.attr.name, temp3);

 
 

-                       dbg("LABEL name = '%s', bus = '%s', sysfs_file = '%s', sysfs_value = '%s'", dev.name, dev.bus, dev.sysfs_file, dev.sysfs_value);
+                       dbg("LABEL name = '%s', bus = '%s', sysfs_file = '%s', sysfs_value = '%s'", dev.attr.name, dev.bus, dev.sysfs_file, dev.sysfs_value);

                }
 
                if (strcasecmp(temp2, TYPE_NUMBER) == 0) {
                }
 
                if (strcasecmp(temp2, TYPE_NUMBER) == 0) {


@@ -315,9 +278,9 @@ static int namedev_init_config(void)
                        retval = get_value("NAME", &temp, &temp3);
                        if (retval)
                                continue;
                        retval = get_value("NAME", &temp, &temp3);
                        if (retval)
                                continue;

-                       strcpy(dev.name, temp3);
+                       strcpy(dev.attr.name, temp3);

 
 

-                       dbg("NUMBER name = '%s', bus = '%s', id = '%s'", dev.name, dev.bus, dev.id);
+                       dbg("NUMBER name = '%s', bus = '%s', id = '%s'", dev.attr.name, dev.bus, dev.id);

                }
 
                if (strcasecmp(temp2, TYPE_TOPOLOGY) == 0) {
                }
 
                if (strcasecmp(temp2, TYPE_TOPOLOGY) == 0) {


@@ -342,9 +305,9 @@ static int namedev_init_config(void)
                        retval = get_value("NAME", &temp, &temp3);
                        if (retval)
                                continue;
                        retval = get_value("NAME", &temp, &temp3);
                        if (retval)
                                continue;

-                       strcpy(dev.name, temp3);
+                       strcpy(dev.attr.name, temp3);

 
 

-                       dbg("TOPOLOGY name = '%s', bus = '%s', place = '%s'", dev.name, dev.bus, dev.place);
+                       dbg("TOPOLOGY name = '%s', bus = '%s', place = '%s'", dev.attr.name, dev.bus, dev.place);

                }
 
                if (strcasecmp(temp2, TYPE_REPLACE) == 0) {
                }
 
                if (strcasecmp(temp2, TYPE_REPLACE) == 0) {


@@ -362,8 +325,8 @@ static int namedev_init_config(void)
                        retval = get_value("NAME", &temp, &temp3);
                        if (retval)
                                continue;
                        retval = get_value("NAME", &temp, &temp3);
                        if (retval)
                                continue;

-                       strcpy(dev.name, temp3);
-                       dbg("REPLACE name = %s, kernel_name = %s", dev.name, dev.kernel_name);
+                       strcpy(dev.attr.name, temp3);
+                       dbg("REPLACE name = %s, kernel_name = %s", dev.attr.name, dev.kernel_name);

                }
 
                retval = add_dev(&dev);
                }
 
                retval = add_dev(&dev);


@@ -422,17 +385,17 @@ static int namedev_init_permissions(void)
 
                /* parse the line */
                temp2 = strsep(&temp, ":");
 
                /* parse the line */
                temp2 = strsep(&temp, ":");

-               strncpy(dev.name, temp2, sizeof(dev.name));
+               strncpy(dev.attr.name, temp2, sizeof(dev.attr.name));

 
                temp2 = strsep(&temp, ":");
 
                temp2 = strsep(&temp, ":");

-               strncpy(dev.owner, temp2, sizeof(dev.owner));
+               strncpy(dev.attr.owner, temp2, sizeof(dev.attr.owner));

 
                temp2 = strsep(&temp, ":");
 
                temp2 = strsep(&temp, ":");

-               strncpy(dev.group, temp2, sizeof(dev.owner));
+               strncpy(dev.attr.group, temp2, sizeof(dev.attr.owner));

 
 

-               dev.mode = strtol(temp, NULL, 8);
+               dev.attr.mode = strtol(temp, NULL, 8);

 
 

-               dbg("name = %s, owner = %s, group = %s, mode = %x", dev.name, dev.owner, dev.group, dev.mode);
+               dbg("name = %s, owner = %s, group = %s, mode = %#o", dev.attr.name, dev.attr.owner, dev.attr.group, dev.attr.mode);

                retval = add_dev(&dev);
                if (retval) {
                        dbg("add_dev returned with error %d", retval);
                retval = add_dev(&dev);
                if (retval) {
                        dbg("add_dev returned with error %d", retval);


@@ -445,12 +408,53 @@ exit:
        return retval;
 }      
 
        return retval;
 }      
 

+static int get_default_mode(struct sysfs_class_device *class_dev)
+{
+       /* just default everyone to rw for the world! */
+       return 0666;
+}

 
 
 
 

-int namedev_init(void)
+static int get_attr(struct sysfs_class_device *class_dev, struct device_attr *attr)
+{
+       struct list_head *tmp;
+       int retval = 0;
+
+       list_for_each(tmp, &config_device_list) {
+               struct config_device *dev = list_entry(tmp, struct config_device, node);
+               if (strcmp(dev->attr.name, class_dev->name) == 0) {
+                       attr->mode = dev->attr.mode;
+                       strcpy(attr->owner, dev->attr.owner);
+                       strcpy(attr->group, dev->attr.group);
+                       /* FIXME  put the proper name here!!! */
+                       strcpy(attr->name, dev->attr.name);
+                       dbg("%s - owner = %s, group = %s, mode = %#o", dev->attr.name, dev->attr.owner, dev->attr.group, dev->attr.mode);
+                       goto exit;
+               }
+       }
+       attr->mode = get_default_mode(class_dev);
+       attr->owner[0] = 0x00;
+       attr->group[0] = 0x00;
+       strcpy(attr->name, class_dev->name);
+exit:
+       return retval;
+}
+
+int namedev_name_device(struct sysfs_class_device *class_dev, struct device_attr *attr)

 {
        int retval;
 
 {
        int retval;
 

+       retval = get_attr(class_dev, attr);
+       if (retval)
+               dbg("get_attr failed");
+
+       return retval;
+}
+
+int namedev_init(void)
+{
+       int retval;
+       

        retval = namedev_init_config();
        if (retval)
                return retval;
        retval = namedev_init_config();
        if (retval)
                return retval;


@@ -463,3 +467,4 @@ int namedev_init(void)
        return retval;
 }
 
        return retval;
 }
 

+