Commit 6fa5c828 authored by Maneesh Soni's avatar Maneesh Soni Committed by Greg Kroah-Hartman

[PATCH] sysfs-iattr: attach sysfs_dirent before new inode

o The following patch makes sure to attach sysfs_dirent to the dentry before
  allocation a new inode through sysfs_create(). This change is done as
  preparatory work for implementing ->i_op->setattr() functionality for
  sysfs objects.
Signed-off-by: default avatarManeesh Soni <maneesh@in.ibm.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 050480f1
...@@ -101,18 +101,19 @@ static int create_dir(struct kobject * k, struct dentry * p, ...@@ -101,18 +101,19 @@ static int create_dir(struct kobject * k, struct dentry * p,
down(&p->d_inode->i_sem); down(&p->d_inode->i_sem);
*d = sysfs_get_dentry(p,n); *d = sysfs_get_dentry(p,n);
if (!IS_ERR(*d)) { if (!IS_ERR(*d)) {
error = sysfs_create(*d, mode, init_dir); error = sysfs_make_dirent(p->d_fsdata, *d, k, mode, SYSFS_DIR);
if (!error) { if (!error) {
error = sysfs_make_dirent(p->d_fsdata, *d, k, mode, error = sysfs_create(*d, mode, init_dir);
SYSFS_DIR);
if (!error) { if (!error) {
p->d_inode->i_nlink++; p->d_inode->i_nlink++;
(*d)->d_op = &sysfs_dentry_ops; (*d)->d_op = &sysfs_dentry_ops;
d_rehash(*d); d_rehash(*d);
} }
} }
if (error && (error != -EEXIST)) if (error && (error != -EEXIST)) {
sysfs_put((*d)->d_fsdata);
d_drop(*d); d_drop(*d);
}
dput(*d); dput(*d);
} else } else
error = PTR_ERR(*d); error = PTR_ERR(*d);
...@@ -171,17 +172,19 @@ static int sysfs_attach_attr(struct sysfs_dirent * sd, struct dentry * dentry) ...@@ -171,17 +172,19 @@ static int sysfs_attach_attr(struct sysfs_dirent * sd, struct dentry * dentry)
init = init_file; init = init_file;
} }
dentry->d_fsdata = sysfs_get(sd);
sd->s_dentry = dentry;
error = sysfs_create(dentry, (attr->mode & S_IALLUGO) | S_IFREG, init); error = sysfs_create(dentry, (attr->mode & S_IALLUGO) | S_IFREG, init);
if (error) if (error) {
sysfs_put(sd);
return error; return error;
}
if (bin_attr) { if (bin_attr) {
dentry->d_inode->i_size = bin_attr->size; dentry->d_inode->i_size = bin_attr->size;
dentry->d_inode->i_fop = &bin_fops; dentry->d_inode->i_fop = &bin_fops;
} }
dentry->d_op = &sysfs_dentry_ops; dentry->d_op = &sysfs_dentry_ops;
dentry->d_fsdata = sysfs_get(sd);
sd->s_dentry = dentry;
d_rehash(dentry); d_rehash(dentry);
return 0; return 0;
...@@ -191,13 +194,15 @@ static int sysfs_attach_link(struct sysfs_dirent * sd, struct dentry * dentry) ...@@ -191,13 +194,15 @@ static int sysfs_attach_link(struct sysfs_dirent * sd, struct dentry * dentry)
{ {
int err = 0; int err = 0;
dentry->d_fsdata = sysfs_get(sd);
sd->s_dentry = dentry;
err = sysfs_create(dentry, S_IFLNK|S_IRWXUGO, init_symlink); err = sysfs_create(dentry, S_IFLNK|S_IRWXUGO, init_symlink);
if (!err) { if (!err) {
dentry->d_op = &sysfs_dentry_ops; dentry->d_op = &sysfs_dentry_ops;
dentry->d_fsdata = sysfs_get(sd);
sd->s_dentry = dentry;
d_rehash(dentry); d_rehash(dentry);
} } else
sysfs_put(sd);
return err; return err;
} }
......
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