diff options
| author | Eric W. Biederman <ebiederm@xmission.com> | 2009-11-08 02:27:02 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-12-11 14:24:53 -0500 |
| commit | 4c6974f51a981d14f13e36049d6307d3bcda550e (patch) | |
| tree | 608016b1dcdbf094732f06a2688a8af3d4d20d62 | |
| parent | e8f077c8831528e2ec1ea6c8ba090e405fdcd0b7 (diff) | |
sysfs: Simplify sysfs_chmod_file semantics
Currently every caller of sysfs_chmod_file happens at either
file creation time to set a non-default mode or in response
to a specific user requested space change in policy. Making
timestamps of when the chmod happens and notification of
a file changing mode uninteresting.
Remove the unnecessary time stamp and filesystem change
notification, and removes the last of the explicit inotify
and donitfy support from sysfs.
Acked-by: Tejun Heo <tj@kernel.org>
Acked-by: Serge Hallyn <serue@us.ibm.com>
Signed-off-by: Eric W. Biederman <ebiederm@aristanetworks.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | fs/sysfs/file.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/fs/sysfs/file.c b/fs/sysfs/file.c index f5ea4680f15f..faa1a803caa9 100644 --- a/fs/sysfs/file.c +++ b/fs/sysfs/file.c | |||
| @@ -604,17 +604,9 @@ int sysfs_chmod_file(struct kobject *kobj, struct attribute *attr, mode_t mode) | |||
| 604 | mutex_lock(&inode->i_mutex); | 604 | mutex_lock(&inode->i_mutex); |
| 605 | 605 | ||
| 606 | newattrs.ia_mode = (mode & S_IALLUGO) | (inode->i_mode & ~S_IALLUGO); | 606 | newattrs.ia_mode = (mode & S_IALLUGO) | (inode->i_mode & ~S_IALLUGO); |
| 607 | newattrs.ia_valid = ATTR_MODE | ATTR_CTIME; | 607 | newattrs.ia_valid = ATTR_MODE; |
| 608 | newattrs.ia_ctime = current_fs_time(inode->i_sb); | ||
| 609 | rc = sysfs_setattr(victim, &newattrs); | 608 | rc = sysfs_setattr(victim, &newattrs); |
| 610 | 609 | ||
| 611 | if (rc == 0) { | ||
| 612 | fsnotify_change(victim, newattrs.ia_valid); | ||
| 613 | mutex_lock(&sysfs_mutex); | ||
| 614 | victim_sd->s_mode = newattrs.ia_mode; | ||
| 615 | mutex_unlock(&sysfs_mutex); | ||
| 616 | } | ||
| 617 | |||
| 618 | mutex_unlock(&inode->i_mutex); | 610 | mutex_unlock(&inode->i_mutex); |
| 619 | out: | 611 | out: |
| 620 | dput(victim); | 612 | dput(victim); |
