diff options
author | Eric Paris <eparis@redhat.com> | 2009-12-17 21:24:22 -0500 |
---|---|---|
committer | Eric Paris <eparis@redhat.com> | 2010-07-28 09:58:50 -0400 |
commit | ffab83402f01555a5fa32efb48a4dd0ce8d12ef5 (patch) | |
tree | 4e02fa7422becb56e6ed4b8721f8b0fb3a867d44 /fs | |
parent | cd7752ce7cac5184ca35aecebffafae9662570bc (diff) |
fsnotify: fsnotify_obtain_group should be fsnotify_alloc_group
fsnotify_obtain_group was intended to be able to find an already existing
group. Nothing uses that functionality. This just renames it to
fsnotify_alloc_group so it is clear what it is doing.
Signed-off-by: Eric Paris <eparis@redhat.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/notify/dnotify/dnotify.c | 2 | ||||
-rw-r--r-- | fs/notify/group.c | 10 | ||||
-rw-r--r-- | fs/notify/inotify/inotify_user.c | 2 |
3 files changed, 5 insertions, 9 deletions
diff --git a/fs/notify/dnotify/dnotify.c b/fs/notify/dnotify/dnotify.c index 1f46aeac3387..51e4fe33d6bb 100644 --- a/fs/notify/dnotify/dnotify.c +++ b/fs/notify/dnotify/dnotify.c | |||
@@ -433,7 +433,7 @@ static int __init dnotify_init(void) | |||
433 | dnotify_struct_cache = KMEM_CACHE(dnotify_struct, SLAB_PANIC); | 433 | dnotify_struct_cache = KMEM_CACHE(dnotify_struct, SLAB_PANIC); |
434 | dnotify_mark_entry_cache = KMEM_CACHE(dnotify_mark_entry, SLAB_PANIC); | 434 | dnotify_mark_entry_cache = KMEM_CACHE(dnotify_mark_entry, SLAB_PANIC); |
435 | 435 | ||
436 | dnotify_group = fsnotify_obtain_group(0, &dnotify_fsnotify_ops); | 436 | dnotify_group = fsnotify_alloc_group(0, &dnotify_fsnotify_ops); |
437 | if (IS_ERR(dnotify_group)) | 437 | if (IS_ERR(dnotify_group)) |
438 | panic("unable to allocate fsnotify group for dnotify\n"); | 438 | panic("unable to allocate fsnotify group for dnotify\n"); |
439 | return 0; | 439 | return 0; |
diff --git a/fs/notify/group.c b/fs/notify/group.c index 934860e98095..1d20d26d5fee 100644 --- a/fs/notify/group.c +++ b/fs/notify/group.c | |||
@@ -162,16 +162,13 @@ void fsnotify_put_group(struct fsnotify_group *group) | |||
162 | } | 162 | } |
163 | 163 | ||
164 | /* | 164 | /* |
165 | * Either finds an existing group which matches the group_num, mask, and ops or | 165 | * Create a new fsnotify_group and hold a reference for the group returned. |
166 | * creates a new group and adds it to the global group list. In either case we | ||
167 | * take a reference for the group returned. | ||
168 | */ | 166 | */ |
169 | struct fsnotify_group *fsnotify_obtain_group(__u32 mask, | 167 | struct fsnotify_group *fsnotify_alloc_group(__u32 mask, |
170 | const struct fsnotify_ops *ops) | 168 | const struct fsnotify_ops *ops) |
171 | { | 169 | { |
172 | struct fsnotify_group *group; | 170 | struct fsnotify_group *group; |
173 | 171 | ||
174 | /* very low use, simpler locking if we just always alloc */ | ||
175 | group = kzalloc(sizeof(struct fsnotify_group), GFP_KERNEL); | 172 | group = kzalloc(sizeof(struct fsnotify_group), GFP_KERNEL); |
176 | if (!group) | 173 | if (!group) |
177 | return ERR_PTR(-ENOMEM); | 174 | return ERR_PTR(-ENOMEM); |
@@ -192,7 +189,6 @@ struct fsnotify_group *fsnotify_obtain_group(__u32 mask, | |||
192 | 189 | ||
193 | mutex_lock(&fsnotify_grp_mutex); | 190 | mutex_lock(&fsnotify_grp_mutex); |
194 | 191 | ||
195 | /* group not found, add a new one */ | ||
196 | list_add_rcu(&group->group_list, &fsnotify_groups); | 192 | list_add_rcu(&group->group_list, &fsnotify_groups); |
197 | group->on_group_list = 1; | 193 | group->on_group_list = 1; |
198 | /* being on the fsnotify_groups list holds one num_marks */ | 194 | /* being on the fsnotify_groups list holds one num_marks */ |
diff --git a/fs/notify/inotify/inotify_user.c b/fs/notify/inotify/inotify_user.c index cae317f5bd9d..25a2854186e9 100644 --- a/fs/notify/inotify/inotify_user.c +++ b/fs/notify/inotify/inotify_user.c | |||
@@ -695,7 +695,7 @@ static struct fsnotify_group *inotify_new_group(struct user_struct *user, unsign | |||
695 | { | 695 | { |
696 | struct fsnotify_group *group; | 696 | struct fsnotify_group *group; |
697 | 697 | ||
698 | group = fsnotify_obtain_group(0, &inotify_fsnotify_ops); | 698 | group = fsnotify_alloc_group(0, &inotify_fsnotify_ops); |
699 | if (IS_ERR(group)) | 699 | if (IS_ERR(group)) |
700 | return group; | 700 | return group; |
701 | 701 | ||