diff options
-rw-r--r-- | fs/sysfs/dir.c | 8 | ||||
-rw-r--r-- | fs/sysfs/file.c | 4 | ||||
-rw-r--r-- | fs/sysfs/sysfs.h | 1 |
3 files changed, 12 insertions, 1 deletions
diff --git a/fs/sysfs/dir.c b/fs/sysfs/dir.c index 1af963e66e3c..9fe83d23dc2c 100644 --- a/fs/sysfs/dir.c +++ b/fs/sysfs/dir.c | |||
@@ -15,6 +15,7 @@ | |||
15 | #include "sysfs.h" | 15 | #include "sysfs.h" |
16 | 16 | ||
17 | DEFINE_MUTEX(sysfs_mutex); | 17 | DEFINE_MUTEX(sysfs_mutex); |
18 | DEFINE_MUTEX(sysfs_rename_mutex); | ||
18 | spinlock_t sysfs_assoc_lock = SPIN_LOCK_UNLOCKED; | 19 | spinlock_t sysfs_assoc_lock = SPIN_LOCK_UNLOCKED; |
19 | 20 | ||
20 | static spinlock_t sysfs_ino_lock = SPIN_LOCK_UNLOCKED; | 21 | static spinlock_t sysfs_ino_lock = SPIN_LOCK_UNLOCKED; |
@@ -82,7 +83,7 @@ static void sysfs_unlink_sibling(struct sysfs_dirent *sd) | |||
82 | * down from there looking up dentry for each step. | 83 | * down from there looking up dentry for each step. |
83 | * | 84 | * |
84 | * LOCKING: | 85 | * LOCKING: |
85 | * Kernel thread context (may sleep) | 86 | * mutex_lock(sysfs_rename_mutex) |
86 | * | 87 | * |
87 | * RETURNS: | 88 | * RETURNS: |
88 | * Pointer to found dentry on success, ERR_PTR() value on error. | 89 | * Pointer to found dentry on success, ERR_PTR() value on error. |
@@ -858,6 +859,8 @@ int sysfs_rename_dir(struct kobject * kobj, const char *new_name) | |||
858 | const char *dup_name = NULL; | 859 | const char *dup_name = NULL; |
859 | int error; | 860 | int error; |
860 | 861 | ||
862 | mutex_lock(&sysfs_rename_mutex); | ||
863 | |||
861 | /* get the original dentry */ | 864 | /* get the original dentry */ |
862 | sd = kobj->sd; | 865 | sd = kobj->sd; |
863 | old_dentry = sysfs_get_dentry(sd); | 866 | old_dentry = sysfs_get_dentry(sd); |
@@ -915,6 +918,7 @@ int sysfs_rename_dir(struct kobject * kobj, const char *new_name) | |||
915 | kfree(dup_name); | 918 | kfree(dup_name); |
916 | dput(old_dentry); | 919 | dput(old_dentry); |
917 | dput(new_dentry); | 920 | dput(new_dentry); |
921 | mutex_unlock(&sysfs_rename_mutex); | ||
918 | return error; | 922 | return error; |
919 | } | 923 | } |
920 | 924 | ||
@@ -926,6 +930,7 @@ int sysfs_move_dir(struct kobject *kobj, struct kobject *new_parent_kobj) | |||
926 | struct dentry *old_dentry = NULL, *new_dentry = NULL; | 930 | struct dentry *old_dentry = NULL, *new_dentry = NULL; |
927 | int error; | 931 | int error; |
928 | 932 | ||
933 | mutex_lock(&sysfs_rename_mutex); | ||
929 | BUG_ON(!sd->s_parent); | 934 | BUG_ON(!sd->s_parent); |
930 | new_parent_sd = new_parent_kobj->sd ? new_parent_kobj->sd : &sysfs_root; | 935 | new_parent_sd = new_parent_kobj->sd ? new_parent_kobj->sd : &sysfs_root; |
931 | 936 | ||
@@ -982,6 +987,7 @@ again: | |||
982 | dput(new_parent); | 987 | dput(new_parent); |
983 | dput(old_dentry); | 988 | dput(old_dentry); |
984 | dput(new_dentry); | 989 | dput(new_dentry); |
990 | mutex_unlock(&sysfs_rename_mutex); | ||
985 | return error; | 991 | return error; |
986 | } | 992 | } |
987 | 993 | ||
diff --git a/fs/sysfs/file.c b/fs/sysfs/file.c index 16f39c30b091..ff93c92164b3 100644 --- a/fs/sysfs/file.c +++ b/fs/sysfs/file.c | |||
@@ -470,7 +470,9 @@ int sysfs_update_file(struct kobject * kobj, const struct attribute * attr) | |||
470 | if (!victim_sd) | 470 | if (!victim_sd) |
471 | goto out; | 471 | goto out; |
472 | 472 | ||
473 | mutex_lock(&sysfs_rename_mutex); | ||
473 | victim = sysfs_get_dentry(victim_sd); | 474 | victim = sysfs_get_dentry(victim_sd); |
475 | mutex_unlock(&sysfs_rename_mutex); | ||
474 | if (IS_ERR(victim)) { | 476 | if (IS_ERR(victim)) { |
475 | rc = PTR_ERR(victim); | 477 | rc = PTR_ERR(victim); |
476 | victim = NULL; | 478 | victim = NULL; |
@@ -509,7 +511,9 @@ int sysfs_chmod_file(struct kobject *kobj, struct attribute *attr, mode_t mode) | |||
509 | if (!victim_sd) | 511 | if (!victim_sd) |
510 | goto out; | 512 | goto out; |
511 | 513 | ||
514 | mutex_lock(&sysfs_rename_mutex); | ||
512 | victim = sysfs_get_dentry(victim_sd); | 515 | victim = sysfs_get_dentry(victim_sd); |
516 | mutex_unlock(&sysfs_rename_mutex); | ||
513 | if (IS_ERR(victim)) { | 517 | if (IS_ERR(victim)) { |
514 | rc = PTR_ERR(victim); | 518 | rc = PTR_ERR(victim); |
515 | victim = NULL; | 519 | victim = NULL; |
diff --git a/fs/sysfs/sysfs.h b/fs/sysfs/sysfs.h index 77253aabc4af..179e6a26ece1 100644 --- a/fs/sysfs/sysfs.h +++ b/fs/sysfs/sysfs.h | |||
@@ -90,6 +90,7 @@ extern int sysfs_setattr(struct dentry *dentry, struct iattr *iattr); | |||
90 | 90 | ||
91 | extern spinlock_t sysfs_assoc_lock; | 91 | extern spinlock_t sysfs_assoc_lock; |
92 | extern struct mutex sysfs_mutex; | 92 | extern struct mutex sysfs_mutex; |
93 | extern struct mutex sysfs_rename_mutex; | ||
93 | extern struct super_block * sysfs_sb; | 94 | extern struct super_block * sysfs_sb; |
94 | extern const struct file_operations sysfs_dir_operations; | 95 | extern const struct file_operations sysfs_dir_operations; |
95 | extern const struct file_operations sysfs_file_operations; | 96 | extern const struct file_operations sysfs_file_operations; |