Revert "sysfs: handle duplicate removal attempts in sysfs_remove_group()"
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 27 Nov 2013 17:44:55 +0000 (09:44 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 27 Nov 2013 17:44:55 +0000 (09:44 -0800)
This reverts commit 54d71145a4548330313ca664a4a009772fe8b7dd.

The root cause of these "inverted" sysfs removals have now been found,
so there is no need for this patch.  Keep this functionality around so
that this type of error doesn't show up in driver code again.

Cc: Mika Westerberg <mika.westerberg@linux.intel.com>
Cc: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Cc: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/sysfs/group.c

index 3796afdff40c0f6214b701177bfbf697b9d1b334..1898a10e38ce8ccef26a687882c637f349ac5b29 100644 (file)
@@ -206,15 +206,6 @@ void sysfs_remove_group(struct kobject *kobj,
        struct sysfs_dirent *dir_sd = kobj->sd;
        struct sysfs_dirent *sd;
 
-       /*
-        * Sysfs directories are now removed recursively by
-        * sysfs_remove_dir(). This means that the function can be called
-        * for a group whose sysfs entry is already removed. In that case
-        * all its groups are guaranteed to be already removed.
-        */
-       if (dir_sd->s_flags & SYSFS_FLAG_REMOVED)
-               return;
-
        if (grp->name) {
                sd = sysfs_get_dirent(dir_sd, grp->name);
                if (!sd) {