summaryrefslogtreecommitdiffstats
path: root/fs/sysfs
diff options
context:
space:
mode:
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>2014-01-13 14:05:13 -0800
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2014-01-13 14:05:13 -0800
commita9f138b0e537de55933335d580ebd38c2bc53c47 (patch)
treef54235f920d392519bde975d42aa4b4534f9561a /fs/sysfs
parent8634c422c1b7e50ca8e346f65afc140d93a3212c (diff)
downloadlinux-a9f138b0e537de55933335d580ebd38c2bc53c47.tar.gz
linux-a9f138b0e537de55933335d580ebd38c2bc53c47.tar.bz2
linux-a9f138b0e537de55933335d580ebd38c2bc53c47.zip
Revert "kernfs, sysfs, driver-core: implement kernfs_remove_self() and its wrappers"
This reverts commit 1ae06819c77cff1ea2833c94f8c093fe8a5c79db. Tejun writes: I'm sorry but can you please revert the whole series? get_active() waiting while a node is deactivated has potential to lead to deadlock and that deactivate/reactivate interface is something fundamentally flawed and that cgroup will have to work with the remove_self() like everybody else. IOW, I think the first posting was correct. Cc: Tejun Heo <tj@kernel.org> Cc: Alan Stern <stern@rowland.harvard.edu> Cc: kbuild test robot <fengguang.wu@intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs/sysfs')
-rw-r--r--fs/sysfs/file.c23
1 files changed, 0 insertions, 23 deletions
diff --git a/fs/sysfs/file.c b/fs/sysfs/file.c
index 1b8b91b67fdb..810cf6e613e5 100644
--- a/fs/sysfs/file.c
+++ b/fs/sysfs/file.c
@@ -372,29 +372,6 @@ void sysfs_remove_file_ns(struct kobject *kobj, const struct attribute *attr,
}
EXPORT_SYMBOL_GPL(sysfs_remove_file_ns);
-/**
- * sysfs_remove_file_self - remove an object attribute from its own method
- * @kobj: object we're acting for
- * @attr: attribute descriptor
- *
- * See kernfs_remove_self() for details.
- */
-bool sysfs_remove_file_self(struct kobject *kobj, const struct attribute *attr)
-{
- struct kernfs_node *parent = kobj->sd;
- struct kernfs_node *kn;
- bool ret;
-
- kn = kernfs_find_and_get(parent, attr->name);
- if (WARN_ON_ONCE(!kn))
- return false;
-
- ret = kernfs_remove_self(kn);
-
- kernfs_put(kn);
- return ret;
-}
-
void sysfs_remove_files(struct kobject *kobj, const struct attribute **ptr)
{
int i;