aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric W. Biederman <ebiederm@xmission.com>2009-11-08 02:27:02 -0500
committerGreg Kroah-Hartman <gregkh@suse.de>2009-12-11 14:24:53 -0500
commit4c6974f51a981d14f13e36049d6307d3bcda550e (patch)
tree608016b1dcdbf094732f06a2688a8af3d4d20d62
parente8f077c8831528e2ec1ea6c8ba090e405fdcd0b7 (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.c10
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);