[PATCH] configfs+dlm: Rename config_group_find_obj and state semantics clearly

Configfs being based upon sysfs code, config_group_find_obj() is probably
so named because of the similar kset_find_obj() in sysfs. However,
"kobject"s in sysfs become "config_item"s in configfs, so let's call it
config_group_find_item() instead, for sake of uniformity, and make
corresponding change in the users of this function.

BTW a crucial difference between kset_find_obj and config_group_find_item
is in locking expectations. kset_find_obj does its locking by itself, but
config_group_find_item expects the *caller* to do the locking. The reason
for this: kset's have their own locks, config_group's don't but instead
rely on the subsystem mutex. And, subsystem needn't necessarily be around
when config_group_find_item() is called.

So let's state these locking semantics explicitly, and rectify the comment,
otherwise bugs could continue to occur in future, as they did in the past
(refer commit d82b8191e238 in gfs2-2.6-fixes.git).

[ I also took the opportunity to fix some bad whitespace and
double-empty lines. --Joel ]

[ Conflict in fs/dlm/config.c with commit
  3168b0780d06ace875696f8a648d04d6089654e5 manually resolved. --Mark ]

Signed-off-by: Satyam Sharma <ssatyam@cse.iitk.ac.in>
Cc: David Teigland <teigland@redhat.com>
Signed-off-by: Joel Becker <joel.becker@oracle.com>
Signed-off-by: Mark Fasheh <mark.fasheh@oracle.com>

2 files changed, 9 insertions, 11 deletions

diff --git a/fs/configfs/item.c b/fs/configfs/item.c
index b762bbeaa0be..76dc4c3e5d51 100644
--- a/fs/configfs/item.c
+++ b/fs/configfs/item.c
@@ -183,27 +183,25 @@ void config_group_init(struct config_group *group)
         INIT_LIST_HEAD(&group->cg_children);
 }
 
-
 /**
- *      config_group_find_obj - search for item in group.
+ *      config_group_find_item - search for item in group.
  *      @group: group we're looking in.
  *      @name:  item's name.
  *
- *      Lock group via @group->cg_subsys, and iterate over @group->cg_list,
- *      looking for a matching config_item. If matching item is found
- *      take a reference and return the item.
+ *      Iterate over @group->cg_list, looking for a matching config_item.
+ *      If matching item is found take a reference and return the item.
+ *      Caller must have locked group via @group->cg_subsys->su_mtx.
  */
-struct config_item *config_group_find_obj(struct config_group *group,
-                                          const char * name)
+struct config_item *config_group_find_item(struct config_group *group,
+                                           const char *name)
 {
         struct list_head * entry;
         struct config_item * ret = NULL;
 
-        /* XXX LOCKING! */
         list_for_each(entry,&group->cg_children) {
                 struct config_item * item = to_item(entry);
                 if (config_item_name(item) &&
-                    !strcmp(config_item_name(item), name)) {
+                    !strcmp(config_item_name(item), name)) {
                         ret = config_item_get(item);
                         break;
                 }
@@ -215,4 +213,4 @@ EXPORT_SYMBOL(config_item_init);
 EXPORT_SYMBOL(config_group_init);
 EXPORT_SYMBOL(config_item_get);
 EXPORT_SYMBOL(config_item_put);
-EXPORT_SYMBOL(config_group_find_obj);
+EXPORT_SYMBOL(config_group_find_item);
diff --git a/fs/dlm/config.c b/fs/dlm/config.c
index e47eb42406fa..4348cb42cf17 100644
--- a/fs/dlm/config.c
+++ b/fs/dlm/config.c
@@ -752,7 +752,7 @@ static struct space *get_space(char *name)
                 return NULL;
 
         down(&space_list->cg_subsys->su_sem);
-        i = config_group_find_obj(space_list, name);
+        i = config_group_find_item(space_list, name);
         up(&space_list->cg_subsys->su_sem);
 
         return to_space(i);