aboutsummaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorEric Paris <eparis@redhat.com>2009-12-17 21:24:22 -0500
committerEric Paris <eparis@redhat.com>2010-07-28 09:58:50 -0400
commitffab83402f01555a5fa32efb48a4dd0ce8d12ef5 (patch)
tree4e02fa7422becb56e6ed4b8721f8b0fb3a867d44 /fs
parentcd7752ce7cac5184ca35aecebffafae9662570bc (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.c2
-rw-r--r--fs/notify/group.c10
-rw-r--r--fs/notify/inotify/inotify_user.c2
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 */
169struct fsnotify_group *fsnotify_obtain_group(__u32 mask, 167struct 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