linux 2.6.16.38 w/ vs2.0.3-rc1
[linux-2.6.git] / lib / kobject.c
index a971b8e..efe67fa 100644 (file)
  * about using the kobject interface.
  */
 
-#undef DEBUG
-
 #include <linux/kobject.h>
 #include <linux/string.h>
 #include <linux/module.h>
 #include <linux/stat.h>
+#include <linux/slab.h>
 
 /**
  *     populate_dir - populate directory with attributes.
@@ -63,7 +62,7 @@ static inline struct kobject * to_kobj(struct list_head * entry)
        return container_of(entry,struct kobject,entry);
 }
 
-static int get_kobj_path_length(struct kset *kset, struct kobject *kobj)
+static int get_kobj_path_length(struct kobject *kobj)
 {
        int length = 1;
        struct kobject * parent = kobj;
@@ -73,13 +72,15 @@ static int get_kobj_path_length(struct kset *kset, struct kobject *kobj)
         * Add 1 to strlen for leading '/' of each level.
         */
        do {
+               if (kobject_name(parent) == NULL)
+                       return 0;
                length += strlen(kobject_name(parent)) + 1;
                parent = parent->parent;
        } while (parent);
        return length;
 }
 
-static void fill_kobj_path(struct kset *kset, struct kobject *kobj, char *path, int length)
+static void fill_kobj_path(struct kobject *kobj, char *path, int length)
 {
        struct kobject * parent;
 
@@ -99,153 +100,33 @@ static void fill_kobj_path(struct kset *kset, struct kobject *kobj, char *path,
  * kobject_get_path - generate and return the path associated with a given kobj
  * and kset pair.  The result must be freed by the caller with kfree().
  *
- * @kset:      kset in question, with which to build the path
  * @kobj:      kobject in question, with which to build the path
  * @gfp_mask:  the allocation type used to allocate the path
  */
-char * kobject_get_path(struct kset *kset, struct kobject *kobj, int gfp_mask)
+char *kobject_get_path(struct kobject *kobj, gfp_t gfp_mask)
 {
        char *path;
        int len;
 
-       len = get_kobj_path_length(kset, kobj);
+       len = get_kobj_path_length(kobj);
+       if (len == 0)
+               return NULL;
        path = kmalloc(len, gfp_mask);
        if (!path)
                return NULL;
        memset(path, 0x00, len);
-       fill_kobj_path(kset, kobj, path, len);
+       fill_kobj_path(kobj, path, len);
 
        return path;
 }
 
-#ifdef CONFIG_HOTPLUG
-
-#define BUFFER_SIZE    1024    /* should be enough memory for the env */
-#define NUM_ENVP       32      /* number of env pointers */
-static unsigned long sequence_num;
-static spinlock_t sequence_lock = SPIN_LOCK_UNLOCKED;
-
-static void kset_hotplug(const char *action, struct kset *kset,
-                        struct kobject *kobj)
-{
-       char *argv [3];
-       char **envp = NULL;
-       char *buffer = NULL;
-       char *scratch;
-       int i = 0;
-       int retval;
-       char *kobj_path = NULL;
-       char *name = NULL;
-       unsigned long seq;
-
-       /* If the kset has a filter operation, call it. If it returns
-          failure, no hotplug event is required. */
-       if (kset->hotplug_ops->filter) {
-               if (!kset->hotplug_ops->filter(kset, kobj))
-                       return;
-       }
-
-       pr_debug ("%s\n", __FUNCTION__);
-
-       if (!hotplug_path[0])
-               return;
-
-       envp = kmalloc(NUM_ENVP * sizeof (char *), GFP_KERNEL);
-       if (!envp)
-               return;
-       memset (envp, 0x00, NUM_ENVP * sizeof (char *));
-
-       buffer = kmalloc(BUFFER_SIZE, GFP_KERNEL);
-       if (!buffer)
-               goto exit;
-
-       if (kset->hotplug_ops->name)
-               name = kset->hotplug_ops->name(kset, kobj);
-       if (name == NULL)
-               name = kset->kobj.name;
-
-       argv [0] = hotplug_path;
-       argv [1] = name;
-       argv [2] = NULL;
-
-       /* minimal command environment */
-       envp [i++] = "HOME=/";
-       envp [i++] = "PATH=/sbin:/bin:/usr/sbin:/usr/bin";
-
-       scratch = buffer;
-
-       envp [i++] = scratch;
-       scratch += sprintf(scratch, "ACTION=%s", action) + 1;
-
-       spin_lock(&sequence_lock);
-       seq = sequence_num++;
-       spin_unlock(&sequence_lock);
-
-       envp [i++] = scratch;
-       scratch += sprintf(scratch, "SEQNUM=%ld", seq) + 1;
-
-       kobj_path = kobject_get_path(kset, kobj, GFP_KERNEL);
-       if (!kobj_path)
-               goto exit;
-
-       envp [i++] = scratch;
-       scratch += sprintf (scratch, "DEVPATH=%s", kobj_path) + 1;
-
-       if (kset->hotplug_ops->hotplug) {
-               /* have the kset specific function add its stuff */
-               retval = kset->hotplug_ops->hotplug (kset, kobj,
-                                 &envp[i], NUM_ENVP - i, scratch,
-                                 BUFFER_SIZE - (scratch - buffer));
-               if (retval) {
-                       pr_debug ("%s - hotplug() returned %d\n",
-                                 __FUNCTION__, retval);
-                       goto exit;
-               }
-       }
-
-       pr_debug ("%s: %s %s %s %s %s %s %s\n", __FUNCTION__, argv[0], argv[1],
-                 envp[0], envp[1], envp[2], envp[3], envp[4]);
-       retval = call_usermodehelper (argv[0], argv, envp, 0);
-       if (retval)
-               pr_debug ("%s - call_usermodehelper returned %d\n",
-                         __FUNCTION__, retval);
-
-exit:
-       kfree(kobj_path);
-       kfree(buffer);
-       kfree(envp);
-       return;
-}
-
-void kobject_hotplug(const char *action, struct kobject *kobj)
-{
-       struct kobject * top_kobj = kobj;
-
-       /* If this kobj does not belong to a kset,
-          try to find a parent that does. */
-       if (!top_kobj->kset && top_kobj->parent) {
-               do {
-                       top_kobj = top_kobj->parent;
-               } while (!top_kobj->kset && top_kobj->parent);
-       }
-
-       if (top_kobj->kset && top_kobj->kset->hotplug_ops)
-               kset_hotplug(action, top_kobj->kset, kobj);
-}
-#else
-void kobject_hotplug(const char *action, struct kobject *kobj)
-{
-       return;
-}
-#endif /* CONFIG_HOTPLUG */
-
 /**
  *     kobject_init - initialize object.
  *     @kobj:  object in question.
  */
 void kobject_init(struct kobject * kobj)
 {
-       kref_init(&kobj->kref);
+       kref_init(&kobj->kref);
        INIT_LIST_HEAD(&kobj->entry);
        kobj->kset = kset_get(kobj->kset);
 }
@@ -264,9 +145,9 @@ void kobject_init(struct kobject * kobj)
 static void unlink(struct kobject * kobj)
 {
        if (kobj->kset) {
-               down_write(&kobj->kset->subsys->rwsem);
+               spin_lock(&kobj->kset->list_lock);
                list_del_init(&kobj->entry);
-               up_write(&kobj->kset->subsys->rwsem);
+               spin_unlock(&kobj->kset->list_lock);
        }
        kobject_put(kobj);
 }
@@ -285,6 +166,11 @@ int kobject_add(struct kobject * kobj)
                return -ENOENT;
        if (!kobj->k_name)
                kobj->k_name = kobj->name;
+       if (!kobj->k_name) {
+               pr_debug("kobject attempted to be registered with no name!\n");
+               WARN_ON(1);
+               return -EINVAL;
+       }
        parent = kobject_get(kobj->parent);
 
        pr_debug("kobject %s: registering. parent: %s, set: %s\n",
@@ -292,23 +178,22 @@ int kobject_add(struct kobject * kobj)
                 kobj->kset ? kobj->kset->kobj.name : "<NULL>" );
 
        if (kobj->kset) {
-               down_write(&kobj->kset->subsys->rwsem);
+               spin_lock(&kobj->kset->list_lock);
 
                if (!parent)
                        parent = kobject_get(&kobj->kset->kobj);
 
                list_add_tail(&kobj->entry,&kobj->kset->list);
-               up_write(&kobj->kset->subsys->rwsem);
+               spin_unlock(&kobj->kset->list_lock);
        }
        kobj->parent = parent;
 
        error = create_dir(kobj);
        if (error) {
+               /* unlink does the kobject_put() for us */
                unlink(kobj);
                if (parent)
                        kobject_put(parent);
-       } else {
-               kobject_hotplug("add", kobj);
        }
 
        return error;
@@ -330,7 +215,8 @@ int kobject_register(struct kobject * kobj)
                        printk("kobject_register failed for %s (%d)\n",
                               kobject_name(kobj),error);
                        dump_stack();
-               }
+               } else
+                       kobject_uevent(kobj, KOBJ_ADD);
        } else
                error = -EINVAL;
        return error;
@@ -340,13 +226,12 @@ int kobject_register(struct kobject * kobj)
 /**
  *     kobject_set_name - Set the name of an object
  *     @kobj:  object.
- *     @name:  name. 
+ *     @fmt:   format string used to build the name
  *
  *     If strlen(name) >= KOBJ_NAME_LEN, then use a dynamically allocated
  *     string that @kobj->k_name points to. Otherwise, use the static 
  *     @kobj->name array.
  */
-
 int kobject_set_name(struct kobject * kobj, const char * fmt, ...)
 {
        int error = 0;
@@ -355,11 +240,12 @@ int kobject_set_name(struct kobject * kobj, const char * fmt, ...)
        va_list args;
        char * name;
 
-       va_start(args,fmt);
        /* 
         * First, try the static array 
         */
+       va_start(args,fmt);
        need = vsnprintf(kobj->name,limit,fmt,args);
+       va_end(args);
        if (need < limit) 
                name = kobj->name;
        else {
@@ -372,7 +258,9 @@ int kobject_set_name(struct kobject * kobj, const char * fmt, ...)
                        error = -ENOMEM;
                        goto Done;
                }
+               va_start(args,fmt);
                need = vsnprintf(name,limit,fmt,args);
+               va_end(args);
 
                /* Still? Give up. */
                if (need >= limit) {
@@ -389,7 +277,6 @@ int kobject_set_name(struct kobject * kobj, const char * fmt, ...)
        /* Now, set the new name */
        kobj->k_name = name;
  Done:
-       va_end(args);
        return error;
 }
 
@@ -402,7 +289,7 @@ EXPORT_SYMBOL(kobject_set_name);
  *     @new_name: object's new name
  */
 
-int kobject_rename(struct kobject * kobj, char *new_name)
+int kobject_rename(struct kobject * kobj, const char *new_name)
 {
        int error = 0;
 
@@ -422,7 +309,6 @@ int kobject_rename(struct kobject * kobj, char *new_name)
 
 void kobject_del(struct kobject * kobj)
 {
-       kobject_hotplug("remove", kobj);
        sysfs_remove_dir(kobj);
        unlink(kobj);
 }
@@ -435,6 +321,7 @@ void kobject_del(struct kobject * kobj)
 void kobject_unregister(struct kobject * kobj)
 {
        pr_debug("kobject %s: unregistering\n",kobject_name(kobj));
+       kobject_uevent(kobj, KOBJ_REMOVE);
        kobject_del(kobj);
        kobject_put(kobj);
 }
@@ -501,6 +388,7 @@ void kset_init(struct kset * k)
 {
        kobject_init(&k->kobj);
        INIT_LIST_HEAD(&k->list);
+       spin_lock_init(&k->list_lock);
 }
 
 
@@ -565,7 +453,7 @@ struct kobject * kset_find_obj(struct kset * kset, const char * name)
        struct list_head * entry;
        struct kobject * ret = NULL;
 
-       down_read(&kset->subsys->rwsem);
+       spin_lock(&kset->list_lock);
        list_for_each(entry,&kset->list) {
                struct kobject * k = to_kobj(entry);
                if (kobject_name(k) && !strcmp(kobject_name(k),name)) {
@@ -573,7 +461,7 @@ struct kobject * kset_find_obj(struct kset * kset, const char * name)
                        break;
                }
        }
-       up_read(&kset->subsys->rwsem);
+       spin_unlock(&kset->list_lock);
        return ret;
 }
 
@@ -645,7 +533,6 @@ void subsys_remove_file(struct subsystem * s, struct subsys_attribute * a)
        }
 }
 
-EXPORT_SYMBOL(kobject_get_path);
 EXPORT_SYMBOL(kobject_init);
 EXPORT_SYMBOL(kobject_register);
 EXPORT_SYMBOL(kobject_unregister);
@@ -653,8 +540,6 @@ EXPORT_SYMBOL(kobject_get);
 EXPORT_SYMBOL(kobject_put);
 EXPORT_SYMBOL(kobject_add);
 EXPORT_SYMBOL(kobject_del);
-EXPORT_SYMBOL(kobject_rename);
-EXPORT_SYMBOL(kobject_hotplug);
 
 EXPORT_SYMBOL(kset_register);
 EXPORT_SYMBOL(kset_unregister);