Commit 13b3086d authored by Tejun Heo's avatar Tejun Heo Committed by Greg Kroah-Hartman

sysfs: add sysfs_dirent->s_parent

Add sysfs_dirent->s_parent.  With this patch, each sd points to and
holds a reference to its parent.  This allows walking sysfs tree
without referencing sd->s_dentry which can go away anytime if the user
doesn't control when it's deleted.

sd->s_parent is initialized and parent is referenced in
sysfs_attach_dirent().  Reference to parent is released when the sd is
released, so as long as reference to a sd is held, s_parent can be
followed.

dentry walk in sysfs_readdir() is convereted to s_parent walk.

This will be used to reimplement symlink such that it uses only
sysfs_dirent tree.
Signed-off-by: default avatarTejun Heo <htejun@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent a26cd722
...@@ -47,6 +47,11 @@ static void sysfs_free_ino(ino_t ino) ...@@ -47,6 +47,11 @@ static void sysfs_free_ino(ino_t ino)
void release_sysfs_dirent(struct sysfs_dirent * sd) void release_sysfs_dirent(struct sysfs_dirent * sd)
{ {
struct sysfs_dirent *parent_sd;
repeat:
parent_sd = sd->s_parent;
if (sd->s_type & SYSFS_KOBJ_LINK) { if (sd->s_type & SYSFS_KOBJ_LINK) {
struct sysfs_symlink * sl = sd->s_element; struct sysfs_symlink * sl = sd->s_element;
kfree(sl->link_name); kfree(sl->link_name);
...@@ -56,6 +61,10 @@ void release_sysfs_dirent(struct sysfs_dirent * sd) ...@@ -56,6 +61,10 @@ void release_sysfs_dirent(struct sysfs_dirent * sd)
kfree(sd->s_iattr); kfree(sd->s_iattr);
sysfs_free_ino(sd->s_ino); sysfs_free_ino(sd->s_ino);
kmem_cache_free(sysfs_dir_cachep, sd); kmem_cache_free(sysfs_dir_cachep, sd);
sd = parent_sd;
if (sd && atomic_dec_and_test(&sd->s_count))
goto repeat;
} }
static void sysfs_d_iput(struct dentry * dentry, struct inode * inode) static void sysfs_d_iput(struct dentry * dentry, struct inode * inode)
...@@ -119,8 +128,10 @@ void sysfs_attach_dirent(struct sysfs_dirent *sd, ...@@ -119,8 +128,10 @@ void sysfs_attach_dirent(struct sysfs_dirent *sd,
dentry->d_op = &sysfs_dentry_ops; dentry->d_op = &sysfs_dentry_ops;
} }
if (parent_sd) if (parent_sd) {
sd->s_parent = sysfs_get(parent_sd);
list_add(&sd->s_sibling, &parent_sd->s_children); list_add(&sd->s_sibling, &parent_sd->s_children);
}
} }
/* /*
...@@ -571,7 +582,10 @@ static int sysfs_readdir(struct file * filp, void * dirent, filldir_t filldir) ...@@ -571,7 +582,10 @@ static int sysfs_readdir(struct file * filp, void * dirent, filldir_t filldir)
i++; i++;
/* fallthrough */ /* fallthrough */
case 1: case 1:
ino = parent_ino(dentry); if (parent_sd->s_parent)
ino = parent_sd->s_parent->s_ino;
else
ino = parent_sd->s_ino;
if (filldir(dirent, "..", 2, i, ino, DT_DIR) < 0) if (filldir(dirent, "..", 2, i, ino, DT_DIR) < 0)
break; break;
filp->f_pos++; filp->f_pos++;
...@@ -688,13 +702,13 @@ int sysfs_make_shadowed_dir(struct kobject *kobj, ...@@ -688,13 +702,13 @@ int sysfs_make_shadowed_dir(struct kobject *kobj,
struct dentry *sysfs_create_shadow_dir(struct kobject *kobj) struct dentry *sysfs_create_shadow_dir(struct kobject *kobj)
{ {
struct dentry *dir = kobj->dentry;
struct inode *inode = dir->d_inode;
struct dentry *parent = dir->d_parent;
struct sysfs_dirent *parent_sd = parent->d_fsdata;
struct dentry *shadow;
struct sysfs_dirent *sd; struct sysfs_dirent *sd;
struct dentry *parent, *dir, *shadow;
struct inode *inode;
dir = kobj->dentry;
inode = dir->d_inode;
parent = dir->d_parent;
shadow = ERR_PTR(-EINVAL); shadow = ERR_PTR(-EINVAL);
if (!sysfs_is_shadowed_inode(inode)) if (!sysfs_is_shadowed_inode(inode))
goto out; goto out;
...@@ -706,6 +720,8 @@ struct dentry *sysfs_create_shadow_dir(struct kobject *kobj) ...@@ -706,6 +720,8 @@ struct dentry *sysfs_create_shadow_dir(struct kobject *kobj)
sd = sysfs_new_dirent(kobj, inode->i_mode, SYSFS_DIR); sd = sysfs_new_dirent(kobj, inode->i_mode, SYSFS_DIR);
if (!sd) if (!sd)
goto nomem; goto nomem;
/* point to parent_sd but don't attach to it */
sd->s_parent = sysfs_get(parent_sd);
sysfs_attach_dirent(sd, NULL, shadow); sysfs_attach_dirent(sd, NULL, shadow);
d_instantiate(shadow, igrab(inode)); d_instantiate(shadow, igrab(inode));
......
...@@ -28,6 +28,7 @@ static const struct super_operations sysfs_ops = { ...@@ -28,6 +28,7 @@ static const struct super_operations sysfs_ops = {
}; };
static struct sysfs_dirent sysfs_root = { static struct sysfs_dirent sysfs_root = {
.s_count = ATOMIC_INIT(1),
.s_sibling = LIST_HEAD_INIT(sysfs_root.s_sibling), .s_sibling = LIST_HEAD_INIT(sysfs_root.s_sibling),
.s_children = LIST_HEAD_INIT(sysfs_root.s_children), .s_children = LIST_HEAD_INIT(sysfs_root.s_children),
.s_element = NULL, .s_element = NULL,
......
struct sysfs_dirent { struct sysfs_dirent {
atomic_t s_count; atomic_t s_count;
struct sysfs_dirent * s_parent;
struct list_head s_sibling; struct list_head s_sibling;
struct list_head s_children; struct list_head s_children;
void * s_element; void * s_element;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment