]> nv-tegra.nvidia Code Review - linux-3.10.git/blobdiff - lib/kobject.c
misc: tegra-profiler: fix stopping the session
[linux-3.10.git] / lib / kobject.c
index c321f1910cc2bd944161046a58d7d68ef3b76c65..b7e29a6056d3eb583728f0f381eaab07a61cfd03 100644 (file)
 
 #include <linux/kobject.h>
 #include <linux/string.h>
-#include <linux/module.h>
+#include <linux/export.h>
 #include <linux/stat.h>
 #include <linux/slab.h>
 
-/**
- *     populate_dir - populate directory with attributes.
- *     @kobj:  object we're working on.
- *
- *     Most subsystems have a set of default attributes that 
- *     are associated with an object that registers with them.
- *     This is a helper called during object registration that 
- *     loops through the default attributes of the subsystem 
- *     and creates attributes files for them in sysfs.
+/*
+ * populate_dir - populate directory with attributes.
+ * @kobj: object we're working on.
  *
+ * Most subsystems have a set of default attributes that are associated
+ * with an object that registers with them.  This is a helper called during
+ * object registration that loops through the default attributes of the
+ * subsystem and creates attributes files for them in sysfs.
  */
-
-static int populate_dir(struct kobject * kobj)
+static int populate_dir(struct kobject *kobj)
 {
-       struct kobj_type * t = get_ktype(kobj);
-       struct attribute * attr;
+       struct kobj_type *t = get_ktype(kobj);
+       struct attribute *attr;
        int error = 0;
        int i;
-       
+
        if (t && t->default_attrs) {
                for (i = 0; (attr = t->default_attrs[i]) != NULL; i++) {
-                       if ((error = sysfs_create_file(kobj,attr)))
+                       error = sysfs_create_file(kobj, attr);
+                       if (error)
                                break;
                }
        }
        return error;
 }
 
-static int create_dir(struct kobject * kobj)
+static int create_dir(struct kobject *kobj)
 {
        int error = 0;
-       if (kobject_name(kobj)) {
-               error = sysfs_create_dir(kobj);
-               if (!error) {
-                       if ((error = populate_dir(kobj)))
-                               sysfs_remove_dir(kobj);
-               }
+       error = sysfs_create_dir(kobj);
+       if (!error) {
+               error = populate_dir(kobj);
+               if (error)
+                       sysfs_remove_dir(kobj);
        }
        return error;
 }
 
-static inline struct kobject * to_kobj(struct list_head * entry)
-{
-       return container_of(entry,struct kobject,entry);
-}
-
 static int get_kobj_path_length(struct kobject *kobj)
 {
        int length = 1;
-       struct kobject * parent = kobj;
+       struct kobject *parent = kobj;
 
-       /* walk up the ancestors until we hit the one pointing to the 
+       /* walk up the ancestors until we hit the one pointing to the
         * root.
         * Add 1 to strlen for leading '/' of each level.
         */
@@ -84,19 +76,19 @@ static int get_kobj_path_length(struct kobject *kobj)
 
 static void fill_kobj_path(struct kobject *kobj, char *path, int length)
 {
-       struct kobject * parent;
+       struct kobject *parent;
 
        --length;
        for (parent = kobj; parent; parent = parent->parent) {
                int cur = strlen(kobject_name(parent));
                /* back up enough to print this name with '/' */
                length -= cur;
-               strncpy (path + length, kobject_name(parent), cur);
+               strncpy(path + length, kobject_name(parent), cur);
                *(path + --length) = '/';
        }
 
        pr_debug("kobject: '%s' (%p): %s: path = '%s'\n", kobject_name(kobj),
-                kobj, __FUNCTION__,path);
+                kobj, __func__, path);
 }
 
 /**
@@ -124,97 +116,90 @@ char *kobject_get_path(struct kobject *kobj, gfp_t gfp_mask)
 }
 EXPORT_SYMBOL_GPL(kobject_get_path);
 
-static void kobject_init_internal(struct kobject * kobj)
+/* add the kobject to its kset's list */
+static void kobj_kset_join(struct kobject *kobj)
 {
-       if (!kobj)
+       if (!kobj->kset)
                return;
-       kref_init(&kobj->kref);
-       INIT_LIST_HEAD(&kobj->entry);
+
+       kset_get(kobj->kset);
+       spin_lock(&kobj->kset->list_lock);
+       list_add_tail(&kobj->entry, &kobj->kset->list);
+       spin_unlock(&kobj->kset->list_lock);
 }
 
+/* remove the kobject from its kset's list */
+static void kobj_kset_leave(struct kobject *kobj)
+{
+       if (!kobj->kset)
+               return;
 
-/**
- *     unlink - remove kobject from kset list.
- *     @kobj:  kobject.
- *
- *     Remove the kobject from the kset list and decrement
- *     its parent's refcount.
- *     This is separated out, so we can use it in both 
- *     kobject_del() and kobject_add_internal() on error.
- */
+       spin_lock(&kobj->kset->list_lock);
+       list_del_init(&kobj->entry);
+       spin_unlock(&kobj->kset->list_lock);
+       kset_put(kobj->kset);
+}
 
-static void unlink(struct kobject * kobj)
+static void kobject_init_internal(struct kobject *kobj)
 {
-       struct kobject *parent = kobj->parent;
-
-       if (kobj->kset) {
-               spin_lock(&kobj->kset->list_lock);
-               list_del_init(&kobj->entry);
-               spin_unlock(&kobj->kset->list_lock);
-       }
-       kobj->parent = NULL;
-       kobject_put(kobj);
-       kobject_put(parent);
+       if (!kobj)
+               return;
+       kref_init(&kobj->kref);
+       INIT_LIST_HEAD(&kobj->entry);
+       kobj->state_in_sysfs = 0;
+       kobj->state_add_uevent_sent = 0;
+       kobj->state_remove_uevent_sent = 0;
+       kobj->state_initialized = 1;
 }
 
+
 static int kobject_add_internal(struct kobject *kobj)
 {
        int error = 0;
-       struct kobject * parent;
+       struct kobject *parent;
 
-       if (!(kobj = kobject_get(kobj)))
+       if (!kobj)
                return -ENOENT;
-       if (!kobj->k_name)
-               kobject_set_name(kobj, "NO_NAME");
-       if (!*kobj->k_name) {
-               pr_debug("kobject (%p) attempted to be registered with no "
+
+       if (!kobj->name || !kobj->name[0]) {
+               WARN(1, "kobject: (%p): attempted to be registered with empty "
                         "name!\n", kobj);
-               WARN_ON(1);
-               kobject_put(kobj);
                return -EINVAL;
        }
-       parent = kobject_get(kobj->parent);
 
-       pr_debug("kobject: '%s' (%p): %s: parent: '%s', set: '%s'\n",
-                kobject_name(kobj), kobj, __FUNCTION__,
-                parent ? kobject_name(parent) : "<NULL>",
-                kobj->kset ? kobject_name(&kobj->kset->kobj) : "<NULL>" );
+       parent = kobject_get(kobj->parent);
 
+       /* join kset if set, use it as parent if we do not already have one */
        if (kobj->kset) {
-               kobj->kset = kset_get(kobj->kset);
-
-               if (!parent) {
+               if (!parent)
                        parent = kobject_get(&kobj->kset->kobj);
-                       /*
-                        * If the kset is our parent, get a second
-                        * reference, we drop both the kset and the
-                        * parent ref on cleanup
-                        */
-                       kobject_get(parent);
-               }
-
-               spin_lock(&kobj->kset->list_lock);
-               list_add_tail(&kobj->entry, &kobj->kset->list);
-               spin_unlock(&kobj->kset->list_lock);
+               kobj_kset_join(kobj);
                kobj->parent = parent;
        }
 
+       pr_debug("kobject: '%s' (%p): %s: parent: '%s', set: '%s'\n",
+                kobject_name(kobj), kobj, __func__,
+                parent ? kobject_name(parent) : "<NULL>",
+                kobj->kset ? kobject_name(&kobj->kset->kobj) : "<NULL>");
+
        error = create_dir(kobj);
        if (error) {
-               /* unlink does the kobject_put() for us */
-               unlink(kobj);
+               kobj_kset_leave(kobj);
+               kobject_put(parent);
+               kobj->parent = NULL;
 
                /* be noisy on error issues */
                if (error == -EEXIST)
-                       printk(KERN_ERR "%s failed for %s with "
-                              "-EEXIST, don't try to register things with "
-                              "the same name in the same directory.\n",
-                              __FUNCTION__, kobject_name(kobj));
+                       WARN(1, "%s failed for %s with "
+                            "-EEXIST, don't try to register things with "
+                            "the same name in the same directory.\n",
+                            __func__, kobject_name(kobj));
                else
-                       printk(KERN_ERR "%s failed for %s (%d)\n",
-                              __FUNCTION__, kobject_name(kobj), error);
-               dump_stack();
-       }
+                       WARN(1, "%s failed for %s (error: %d parent: %s)\n",
+                            __func__, kobject_name(kobj), error,
+                            parent ? kobject_name(parent) : "'none'");
+       } else
+               kobj->state_in_sysfs = 1;
 
        return error;
 }
@@ -225,25 +210,24 @@ static int kobject_add_internal(struct kobject *kobj)
  * @fmt: format string used to build the name
  * @vargs: vargs to format the string.
  */
-static int kobject_set_name_vargs(struct kobject *kobj, const char *fmt,
+int kobject_set_name_vargs(struct kobject *kobj, const char *fmt,
                                  va_list vargs)
 {
-       va_list aq;
-       char *name;
+       const char *old_name = kobj->name;
+       char *s;
 
-       va_copy(aq, vargs);
-       name = kvasprintf(GFP_KERNEL, fmt, vargs);
-       va_end(aq);
+       if (kobj->name && !fmt)
+               return 0;
 
-       if (!name)
+       kobj->name = kvasprintf(GFP_KERNEL, fmt, vargs);
+       if (!kobj->name)
                return -ENOMEM;
 
-       /* Free the old name, if necessary. */
-       kfree(kobj->k_name);
-
-       /* Now, set the new name */
-       kobj->k_name = name;
+       /* ewww... some of these buggers have '/' in the name ... */
+       while ((s = strchr(kobj->name, '/')))
+               s[0] = '!';
 
+       kfree(old_name);
        return 0;
 }
 
@@ -258,12 +242,12 @@ static int kobject_set_name_vargs(struct kobject *kobj, const char *fmt,
  */
 int kobject_set_name(struct kobject *kobj, const char *fmt, ...)
 {
-       va_list args;
+       va_list vargs;
        int retval;
 
-       va_start(args, fmt);
-       retval = kobject_set_name_vargs(kobj, fmt, args);
-       va_end(args);
+       va_start(vargs, fmt);
+       retval = kobject_set_name_vargs(kobj, fmt, vargs);
+       va_end(vargs);
 
        return retval;
 }
@@ -293,20 +277,19 @@ void kobject_init(struct kobject *kobj, struct kobj_type *ktype)
                err_str = "must have a ktype to be initialized properly!\n";
                goto error;
        }
-       if (atomic_read(&kobj->kref.refcount)) {
+       if (kobj->state_initialized) {
                /* do not error out as sometimes we can recover */
-               printk(KERN_ERR "kobject: reference count is already set, "
-                      "something is seriously wrong.\n");
+               printk(KERN_ERR "kobject (%p): tried to init an initialized "
+                      "object, something is seriously wrong.\n", kobj);
                dump_stack();
        }
 
-       kref_init(&kobj->kref);
-       INIT_LIST_HEAD(&kobj->entry);
+       kobject_init_internal(kobj);
        kobj->ktype = ktype;
        return;
 
 error:
-       printk(KERN_ERR "kobject: %s\n", err_str);
+       printk(KERN_ERR "kobject (%p): %s\n", kobj, err_str);
        dump_stack();
 }
 EXPORT_SYMBOL(kobject_init);
@@ -314,12 +297,9 @@ EXPORT_SYMBOL(kobject_init);
 static int kobject_add_varg(struct kobject *kobj, struct kobject *parent,
                            const char *fmt, va_list vargs)
 {
-       va_list aq;
        int retval;
 
-       va_copy(aq, vargs);
-       retval = kobject_set_name_vargs(kobj, fmt, aq);
-       va_end(aq);
+       retval = kobject_set_name_vargs(kobj, fmt, vargs);
        if (retval) {
                printk(KERN_ERR "kobject: can not set name properly!\n");
                return retval;
@@ -345,17 +325,10 @@ static int kobject_add_varg(struct kobject *kobj, struct kobject *parent,
  *
  * If this function returns an error, kobject_put() must be called to
  * properly clean up the memory associated with the object.
- *
- * If the function is successful, the only way to properly clean up the
- * memory is with a call to kobject_del(), in which case, a call to
- * kobject_put() is not necessary (kobject_del() does the final
- * kobject_put() to call the release function in the ktype's release
- * pointer.)
- *
  * Under no instance should the kobject that is passed to this function
  * be directly freed with a call to kfree(), that can leak memory.
  *
- * Note, no uevent will be created with this call, the caller should set
+ * Note, no "add" uevent will be created with this call, the caller should set
  * up all of the necessary sysfs files for the object and then call
  * kobject_uevent() with the UEVENT_ADD parameter to ensure that
  * userspace is properly notified of this kobject's creation.
@@ -369,6 +342,13 @@ int kobject_add(struct kobject *kobj, struct kobject *parent,
        if (!kobj)
                return -EINVAL;
 
+       if (!kobj->state_initialized) {
+               printk(KERN_ERR "kobject '%s' (%p): tried to add an "
+                      "uninitialized object, something is seriously wrong.\n",
+                      kobject_name(kobj), kobj);
+               dump_stack();
+               return -EINVAL;
+       }
        va_start(args, fmt);
        retval = kobject_add_varg(kobj, parent, fmt, args);
        va_end(args);
@@ -405,15 +385,20 @@ int kobject_init_and_add(struct kobject *kobj, struct kobj_type *ktype,
 EXPORT_SYMBOL_GPL(kobject_init_and_add);
 
 /**
- *     kobject_rename - change the name of an object
- *     @kobj:  object in question.
- *     @new_name: object's new name
+ * kobject_rename - change the name of an object
+ * @kobj: object in question.
+ * @new_name: object's new name
+ *
+ * It is the responsibility of the caller to provide mutual
+ * exclusion between two different calls of kobject_rename
+ * on the same kobject and to ensure that new_name is valid and
+ * won't conflict with other kobjects.
  */
-
-int kobject_rename(struct kobject * kobj, const char *new_name)
+int kobject_rename(struct kobject *kobj, const char *new_name)
 {
        int error = 0;
        const char *devpath = NULL;
+       const char *dup_name = NULL, *name;
        char *devpath_string = NULL;
        char *envp[2];
 
@@ -423,19 +408,6 @@ int kobject_rename(struct kobject * kobj, const char *new_name)
        if (!kobj->parent)
                return -EINVAL;
 
-       /* see if this name is already in use */
-       if (kobj->kset) {
-               struct kobject *temp_kobj;
-               temp_kobj = kset_find_obj(kobj->kset, new_name);
-               if (temp_kobj) {
-                       printk(KERN_WARNING "kobject '%s' cannot be renamed "
-                              "to '%s' as '%s' is already in existence.\n",
-                              kobject_name(kobj), new_name, new_name);
-                       kobject_put(temp_kobj);
-                       return -EINVAL;
-               }
-       }
-
        devpath = kobject_get_path(kobj, GFP_KERNEL);
        if (!devpath) {
                error = -ENOMEM;
@@ -450,28 +422,40 @@ int kobject_rename(struct kobject * kobj, const char *new_name)
        envp[0] = devpath_string;
        envp[1] = NULL;
 
+       name = dup_name = kstrdup(new_name, GFP_KERNEL);
+       if (!name) {
+               error = -ENOMEM;
+               goto out;
+       }
+
        error = sysfs_rename_dir(kobj, new_name);
+       if (error)
+               goto out;
+
+       /* Install the new kobject name */
+       dup_name = kobj->name;
+       kobj->name = name;
 
        /* This function is mostly/only used for network interface.
         * Some hotplug package track interfaces by their name and
         * therefore want to know when the name is changed by the user. */
-       if (!error)
-               kobject_uevent_env(kobj, KOBJ_MOVE, envp);
+       kobject_uevent_env(kobj, KOBJ_MOVE, envp);
 
 out:
+       kfree(dup_name);
        kfree(devpath_string);
        kfree(devpath);
        kobject_put(kobj);
 
        return error;
 }
+EXPORT_SYMBOL_GPL(kobject_rename);
 
 /**
- *     kobject_move - move object to another parent
- *     @kobj:  object in question.
- *     @new_parent: object's new parent (can be NULL)
+ * kobject_move - move object to another parent
+ * @kobj: object in question.
+ * @new_parent: object's new parent (can be NULL)
  */
-
 int kobject_move(struct kobject *kobj, struct kobject *new_parent)
 {
        int error;
@@ -519,43 +503,36 @@ out:
 }
 
 /**
- *     kobject_del - unlink kobject from hierarchy.
- *     @kobj:  object.
+ * kobject_del - unlink kobject from hierarchy.
+ * @kobj: object.
  */
-
-void kobject_del(struct kobject * kobj)
+void kobject_del(struct kobject *kobj)
 {
        if (!kobj)
                return;
+
        sysfs_remove_dir(kobj);
-       unlink(kobj);
+       kobj->state_in_sysfs = 0;
+       kobj_kset_leave(kobj);
+       kobject_put(kobj->parent);
+       kobj->parent = NULL;
 }
 
 /**
- *     kobject_unregister - remove object from hierarchy and decrement refcount.
- *     @kobj:  object going away.
+ * kobject_get - increment refcount for object.
+ * @kobj: object.
  */
-
-void kobject_unregister(struct kobject * kobj)
+struct kobject *kobject_get(struct kobject *kobj)
 {
-       if (!kobj)
-               return;
-       pr_debug("kobject: '%s' (%p): %s\n",
-                kobject_name(kobj), kobj, __FUNCTION__);
-       kobject_uevent(kobj, KOBJ_REMOVE);
-       kobject_del(kobj);
-       kobject_put(kobj);
+       if (kobj)
+               kref_get(&kobj->kref);
+       return kobj;
 }
 
-/**
- *     kobject_get - increment refcount for object.
- *     @kobj:  object.
- */
-
-struct kobject * kobject_get(struct kobject * kobj)
+static struct kobject * __must_check kobject_get_unless_zero(struct kobject *kobj)
 {
-       if (kobj)
-               kref_get(&kobj->kref);
+       if (!kref_get_unless_zero(&kobj->kref))
+               kobj = NULL;
        return kobj;
 }
 
@@ -565,21 +542,42 @@ struct kobject * kobject_get(struct kobject * kobj)
  */
 static void kobject_cleanup(struct kobject *kobj)
 {
-       struct kobj_type * t = get_ktype(kobj);
-       struct kset * s = kobj->kset;
-       const char *name = kobj->k_name;
+       struct kobj_type *t = get_ktype(kobj);
+       const char *name = kobj->name;
 
        pr_debug("kobject: '%s' (%p): %s\n",
-                kobject_name(kobj), kobj, __FUNCTION__);
+                kobject_name(kobj), kobj, __func__);
+
+       if (t && !t->release)
+               pr_debug("kobject: '%s' (%p): does not have a release() "
+                        "function, it is broken and must be fixed.\n",
+                        kobject_name(kobj), kobj);
+
+       /* send "remove" if the caller did not do it but sent "add" */
+       if (kobj->state_add_uevent_sent && !kobj->state_remove_uevent_sent) {
+               pr_debug("kobject: '%s' (%p): auto cleanup 'remove' event\n",
+                        kobject_name(kobj), kobj);
+               kobject_uevent(kobj, KOBJ_REMOVE);
+       }
+
+       /* remove from sysfs if the caller did not do it */
+       if (kobj->state_in_sysfs) {
+               pr_debug("kobject: '%s' (%p): auto cleanup kobject_del\n",
+                        kobject_name(kobj), kobj);
+               kobject_del(kobj);
+       }
+
        if (t && t->release) {
+               pr_debug("kobject: '%s' (%p): calling ktype release\n",
+                        kobject_name(kobj), kobj);
                t->release(kobj);
-               /* If we have a release function, we can guess that this was
-                * not a statically allocated kobject, so we should be safe to
-                * free the name */
+       }
+
+       /* free name if we allocated it */
+       if (name) {
+               pr_debug("kobject: '%s': free name\n", name);
                kfree(name);
        }
-       if (s)
-               kset_put(s);
 }
 
 static void kobject_release(struct kref *kref)
@@ -588,21 +586,25 @@ static void kobject_release(struct kref *kref)
 }
 
 /**
- *     kobject_put - decrement refcount for object.
- *     @kobj:  object.
+ * kobject_put - decrement refcount for object.
+ * @kobj: object.
  *
- *     Decrement the refcount, and if 0, call kobject_cleanup().
+ * Decrement the refcount, and if 0, call kobject_cleanup().
  */
-void kobject_put(struct kobject * kobj)
+void kobject_put(struct kobject *kobj)
 {
-       if (kobj)
+       if (kobj) {
+               if (!kobj->state_initialized)
+                       WARN(1, KERN_WARNING "kobject: '%s' (%p): is not "
+                              "initialized, yet kobject_put() is being "
+                              "called.\n", kobject_name(kobj), kobj);
                kref_put(&kobj->kref, kobject_release);
+       }
 }
 
 static void dynamic_kobj_release(struct kobject *kobj)
 {
-       pr_debug("kobject: '%s' (%p): %s\n",
-                kobject_name(kobj), kobj, __FUNCTION__);
+       pr_debug("kobject: (%p): %s\n", kobj, __func__);
        kfree(kobj);
 }
 
@@ -637,12 +639,12 @@ struct kobject *kobject_create(void)
 /**
  * kobject_create_and_add - create a struct kobject dynamically and register it with sysfs
  *
- * @name: the name for the kset
+ * @name: the name for the kobject
  * @parent: the parent kobject of this kobject, if any.
  *
- * This function creates a kset structure dynamically and registers it
+ * This function creates a kobject structure dynamically and registers it
  * with sysfs.  When you are finished with this structure, call
- * kobject_unregister() and the structure will be dynamically freed when
+ * kobject_put() and the structure will be dynamically freed when
  * it is no longer being used.
  *
  * If the kobject was not able to be created, NULL will be returned.
@@ -659,7 +661,7 @@ struct kobject *kobject_create_and_add(const char *name, struct kobject *parent)
        retval = kobject_add(kobj, parent, "%s", name);
        if (retval) {
                printk(KERN_WARNING "%s: kobject_add error: %d\n",
-                      __FUNCTION__, retval);
+                      __func__, retval);
                kobject_put(kobj);
                kobj = NULL;
        }
@@ -668,11 +670,10 @@ struct kobject *kobject_create_and_add(const char *name, struct kobject *parent)
 EXPORT_SYMBOL_GPL(kobject_create_and_add);
 
 /**
- *     kset_init - initialize a kset for use
- *     @k:     kset 
+ * kset_init - initialize a kset for use
+ * @k: kset
  */
-
-void kset_init(struct kset * k)
+void kset_init(struct kset *k)
 {
        kobject_init_internal(&k->kobj);
        INIT_LIST_HEAD(&k->list);
@@ -704,17 +705,16 @@ static ssize_t kobj_attr_store(struct kobject *kobj, struct attribute *attr,
        return ret;
 }
 
-struct sysfs_ops kobj_sysfs_ops = {
+const struct sysfs_ops kobj_sysfs_ops = {
        .show   = kobj_attr_show,
        .store  = kobj_attr_store,
 };
 
 /**
- *     kset_register - initialize and add a kset.
- *     @k:     kset.
+ * kset_register - initialize and add a kset.
+ * @k: kset.
  */
-
-int kset_register(struct kset * k)
+int kset_register(struct kset *k)
 {
        int err;
 
@@ -729,43 +729,40 @@ int kset_register(struct kset * k)
        return 0;
 }
 
-
 /**
- *     kset_unregister - remove a kset.
- *     @k:     kset.
+ * kset_unregister - remove a kset.
+ * @k: kset.
  */
-
-void kset_unregister(struct kset * k)
+void kset_unregister(struct kset *k)
 {
        if (!k)
                return;
-       kobject_unregister(&k->kobj);
+       kobject_put(&k->kobj);
 }
 
-
 /**
- *     kset_find_obj - search for object in kset.
- *     @kset:  kset we're looking in.
- *     @name:  object's name.
+ * kset_find_obj - search for object in kset.
+ * @kset: kset we're looking in.
+ * @name: object's name.
  *
- *     Lock kset via @kset->subsys, and iterate over @kset->list,
- *     looking for a matching kobject. If matching object is found
- *     take a reference and return the object.
+ * Lock kset via @kset->subsys, and iterate over @kset->list,
+ * looking for a matching kobject. If matching object is found
+ * take a reference and return the object.
  */
-
-struct kobject * kset_find_obj(struct kset * kset, const char * name)
+struct kobject *kset_find_obj(struct kset *kset, const char *name)
 {
-       struct list_head * entry;
-       struct kobject * ret = NULL;
+       struct kobject *k;
+       struct kobject *ret = NULL;
 
        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)) {
-                       ret = kobject_get(k);
+
+       list_for_each_entry(k, &kset->list, entry) {
+               if (kobject_name(k) && !strcmp(kobject_name(k), name)) {
+                       ret = kobject_get_unless_zero(k);
                        break;
                }
        }
+
        spin_unlock(&kset->list_lock);
        return ret;
 }
@@ -774,7 +771,7 @@ static void kset_release(struct kobject *kobj)
 {
        struct kset *kset = container_of(kobj, struct kset, kobj);
        pr_debug("kobject: '%s' (%p): %s\n",
-                kobject_name(kobj), kobj, __FUNCTION__);
+                kobject_name(kobj), kobj, __func__);
        kfree(kset);
 }
 
@@ -799,15 +796,20 @@ static struct kobj_type kset_ktype = {
  * If the kset was not able to be created, NULL will be returned.
  */
 static struct kset *kset_create(const char *name,
-                               struct kset_uevent_ops *uevent_ops,
+                               const struct kset_uevent_ops *uevent_ops,
                                struct kobject *parent_kobj)
 {
        struct kset *kset;
+       int retval;
 
        kset = kzalloc(sizeof(*kset), GFP_KERNEL);
        if (!kset)
                return NULL;
-       kobject_set_name(&kset->kobj, name);
+       retval = kobject_set_name(&kset->kobj, name);
+       if (retval) {
+               kfree(kset);
+               return NULL;
+       }
        kset->uevent_ops = uevent_ops;
        kset->kobj.parent = parent_kobj;
 
@@ -837,7 +839,7 @@ static struct kset *kset_create(const char *name,
  * If the kset was not able to be created, NULL will be returned.
  */
 struct kset *kset_create_and_add(const char *name,
-                                struct kset_uevent_ops *uevent_ops,
+                                const struct kset_uevent_ops *uevent_ops,
                                 struct kobject *parent_kobj)
 {
        struct kset *kset;
@@ -855,7 +857,113 @@ struct kset *kset_create_and_add(const char *name,
 }
 EXPORT_SYMBOL_GPL(kset_create_and_add);
 
-EXPORT_SYMBOL(kobject_unregister);
+
+static DEFINE_SPINLOCK(kobj_ns_type_lock);
+static const struct kobj_ns_type_operations *kobj_ns_ops_tbl[KOBJ_NS_TYPES];
+
+int kobj_ns_type_register(const struct kobj_ns_type_operations *ops)
+{
+       enum kobj_ns_type type = ops->type;
+       int error;
+
+       spin_lock(&kobj_ns_type_lock);
+
+       error = -EINVAL;
+       if (type >= KOBJ_NS_TYPES)
+               goto out;
+
+       error = -EINVAL;
+       if (type <= KOBJ_NS_TYPE_NONE)
+               goto out;
+
+       error = -EBUSY;
+       if (kobj_ns_ops_tbl[type])
+               goto out;
+
+       error = 0;
+       kobj_ns_ops_tbl[type] = ops;
+
+out:
+       spin_unlock(&kobj_ns_type_lock);
+       return error;
+}
+
+int kobj_ns_type_registered(enum kobj_ns_type type)
+{
+       int registered = 0;
+
+       spin_lock(&kobj_ns_type_lock);
+       if ((type > KOBJ_NS_TYPE_NONE) && (type < KOBJ_NS_TYPES))
+               registered = kobj_ns_ops_tbl[type] != NULL;
+       spin_unlock(&kobj_ns_type_lock);
+
+       return registered;
+}
+
+const struct kobj_ns_type_operations *kobj_child_ns_ops(struct kobject *parent)
+{
+       const struct kobj_ns_type_operations *ops = NULL;
+
+       if (parent && parent->ktype->child_ns_type)
+               ops = parent->ktype->child_ns_type(parent);
+
+       return ops;
+}
+
+const struct kobj_ns_type_operations *kobj_ns_ops(struct kobject *kobj)
+{
+       return kobj_child_ns_ops(kobj->parent);
+}
+
+
+void *kobj_ns_grab_current(enum kobj_ns_type type)
+{
+       void *ns = NULL;
+
+       spin_lock(&kobj_ns_type_lock);
+       if ((type > KOBJ_NS_TYPE_NONE) && (type < KOBJ_NS_TYPES) &&
+           kobj_ns_ops_tbl[type])
+               ns = kobj_ns_ops_tbl[type]->grab_current_ns();
+       spin_unlock(&kobj_ns_type_lock);
+
+       return ns;
+}
+
+const void *kobj_ns_netlink(enum kobj_ns_type type, struct sock *sk)
+{
+       const void *ns = NULL;
+
+       spin_lock(&kobj_ns_type_lock);
+       if ((type > KOBJ_NS_TYPE_NONE) && (type < KOBJ_NS_TYPES) &&
+           kobj_ns_ops_tbl[type])
+               ns = kobj_ns_ops_tbl[type]->netlink_ns(sk);
+       spin_unlock(&kobj_ns_type_lock);
+
+       return ns;
+}
+
+const void *kobj_ns_initial(enum kobj_ns_type type)
+{
+       const void *ns = NULL;
+
+       spin_lock(&kobj_ns_type_lock);
+       if ((type > KOBJ_NS_TYPE_NONE) && (type < KOBJ_NS_TYPES) &&
+           kobj_ns_ops_tbl[type])
+               ns = kobj_ns_ops_tbl[type]->initial_ns();
+       spin_unlock(&kobj_ns_type_lock);
+
+       return ns;
+}
+
+void kobj_ns_drop(enum kobj_ns_type type, void *ns)
+{
+       spin_lock(&kobj_ns_type_lock);
+       if ((type > KOBJ_NS_TYPE_NONE) && (type < KOBJ_NS_TYPES) &&
+           kobj_ns_ops_tbl[type] && kobj_ns_ops_tbl[type]->drop_ns)
+               kobj_ns_ops_tbl[type]->drop_ns(ns);
+       spin_unlock(&kobj_ns_type_lock);
+}
+
 EXPORT_SYMBOL(kobject_get);
 EXPORT_SYMBOL(kobject_put);
 EXPORT_SYMBOL(kobject_del);