Whamcloud - gitweb
LU-8066 obd_type: discard obd_types linked list.
[fs/lustre-release.git] / lustre / obdclass / genops.c
index 0a840c9..e622041 100644 (file)
 #include <lustre_disk.h>
 #include <lustre_kernelcomm.h>
 
-static DEFINE_SPINLOCK(obd_types_lock);
-static LIST_HEAD(obd_types);
 DEFINE_RWLOCK(obd_dev_lock);
 static struct obd_device *obd_devs[MAX_OBD_DEVICES];
 
 static struct kmem_cache *obd_device_cachep;
-
+static struct kobj_type class_ktype;
 static struct workqueue_struct *zombie_wq;
 
 static void obd_zombie_export_add(struct obd_export *exp);
@@ -98,30 +96,26 @@ static void obd_device_free(struct obd_device *obd)
 
 struct obd_type *class_search_type(const char *name)
 {
-       struct list_head *tmp;
-       struct obd_type *type;
+       struct kobject *kobj = kset_find_obj(lustre_kset, name);
 
-       spin_lock(&obd_types_lock);
-       list_for_each(tmp, &obd_types) {
-               type = list_entry(tmp, struct obd_type, typ_chain);
-               if (strcmp(type->typ_name, name) == 0) {
-                       spin_unlock(&obd_types_lock);
-                       return type;
-               }
-       }
-       spin_unlock(&obd_types_lock);
+       if (kobj && kobj->ktype == &class_ktype)
+               return container_of(kobj, struct obd_type, typ_kobj);
+
+       kobject_put(kobj);
        return NULL;
 }
 EXPORT_SYMBOL(class_search_type);
 
 struct obd_type *class_get_type(const char *name)
 {
-        struct obd_type *type = class_search_type(name);
+       struct obd_type *type;
 
+       type = class_search_type(name);
 #ifdef HAVE_MODULE_LOADING_SUPPORT
         if (!type) {
                 const char *modname = name;
 
+#ifdef HAVE_SERVER_SUPPORT
                if (strcmp(modname, "obdfilter") == 0)
                        modname = "ofd";
 
@@ -130,6 +124,7 @@ struct obd_type *class_get_type(const char *name)
 
                if (!strncmp(modname, LUSTRE_MDS_NAME, strlen(LUSTRE_MDS_NAME)))
                        modname = LUSTRE_MDT_NAME;
+#endif /* HAVE_SERVER_SUPPORT */
 
                if (!request_module("%s", modname)) {
                        CDEBUG(D_INFO, "Loaded module '%s'\n", modname);
@@ -145,6 +140,11 @@ struct obd_type *class_get_type(const char *name)
                type->typ_refcnt++;
                try_module_get(type->typ_dt_ops->o_owner);
                spin_unlock(&type->obd_type_lock);
+               /* class_search_type() returned a counted reference,
+                * but we don't need that count any more as
+                * we have one through typ_refcnt.
+                */
+               kobject_put(&type->typ_kobj);
        }
        return type;
 }
@@ -160,7 +160,24 @@ void class_put_type(struct obd_type *type)
 
 static void class_sysfs_release(struct kobject *kobj)
 {
-       OBD_FREE(kobj, sizeof(*kobj));
+       struct obd_type *type = container_of(kobj, struct obd_type, typ_kobj);
+
+       debugfs_remove_recursive(type->typ_debugfs_entry);
+       type->typ_debugfs_entry = NULL;
+
+       if (type->typ_lu)
+               lu_device_type_fini(type->typ_lu);
+
+#ifdef CONFIG_PROC_FS
+       if (type->typ_name && type->typ_procroot)
+               remove_proc_subtree(type->typ_name, proc_lustre_root);
+#endif
+       if (type->typ_md_ops)
+               OBD_FREE_PTR(type->typ_md_ops);
+       if (type->typ_dt_ops)
+               OBD_FREE_PTR(type->typ_dt_ops);
+
+       OBD_FREE(type, sizeof(*type));
 }
 
 static struct kobj_type class_ktype = {
@@ -168,30 +185,52 @@ static struct kobj_type class_ktype = {
        .release        = class_sysfs_release,
 };
 
-struct kobject *class_setup_tunables(const char *name)
+#ifdef HAVE_SERVER_SUPPORT
+struct obd_type *class_add_symlinks(const char *name, bool enable_proc)
 {
-       struct kobject *kobj;
+       struct dentry *symlink;
+       struct obd_type *type;
        int rc;
 
-#ifdef HAVE_SERVER_SUPPORT
-       kobj = kset_find_obj(lustre_kset, name);
-       if (kobj)
-               return kobj;
-#endif
-       OBD_ALLOC(kobj, sizeof(*kobj));
-       if (!kobj)
+       type = class_search_type(name);
+       if (type) {
+               kobject_put(&type->typ_kobj);
+               return ERR_PTR(-EEXIST);
+       }
+
+       OBD_ALLOC(type, sizeof(*type));
+       if (!type)
                return ERR_PTR(-ENOMEM);
 
-       kobj->kset = lustre_kset;
-       kobject_init(kobj, &class_ktype);
-       rc = kobject_add(kobj, &lustre_kset->kobj, "%s", name);
-       if (rc) {
-               kobject_put(kobj);
+       type->typ_kobj.kset = lustre_kset;
+       rc = kobject_init_and_add(&type->typ_kobj, &class_ktype,
+                                 &lustre_kset->kobj, "%s", name);
+       if (rc)
+               return ERR_PTR(rc);
+
+       symlink = debugfs_create_dir(name, debugfs_lustre_root);
+       if (IS_ERR_OR_NULL(symlink)) {
+               rc = symlink ? PTR_ERR(symlink) : -ENOMEM;
+               kobject_put(&type->typ_kobj);
                return ERR_PTR(rc);
        }
-       return kobj;
+       type->typ_debugfs_entry = symlink;
+       type->typ_sym_filter = true;
+
+       if (enable_proc) {
+               type->typ_procroot = lprocfs_register(name, proc_lustre_root,
+                                                     NULL, NULL);
+               if (IS_ERR(type->typ_procroot)) {
+                       CERROR("%s: can't create compat proc entry: %d\n",
+                              name, (int)PTR_ERR(type->typ_procroot));
+                       type->typ_procroot = NULL;
+               }
+       }
+
+       return type;
 }
-EXPORT_SYMBOL(class_setup_tunables);
+EXPORT_SYMBOL(class_add_symlinks);
+#endif /* HAVE_SERVER_SUPPORT */
 
 #define CLASS_MAX_NAME 1024
 
@@ -200,44 +239,55 @@ int class_register_type(struct obd_ops *dt_ops, struct md_ops *md_ops,
                        const char *name, struct lu_device_type *ldt)
 {
        struct obd_type *type;
-#ifdef HAVE_SERVER_SUPPORT
-       struct qstr dname;
-#endif /* HAVE_SERVER_SUPPORT */
-       int rc = 0;
+       int rc;
 
        ENTRY;
        /* sanity check */
        LASSERT(strnlen(name, CLASS_MAX_NAME) < CLASS_MAX_NAME);
 
-        if (class_search_type(name)) {
+       type = class_search_type(name);
+       if (type) {
+#ifdef HAVE_SERVER_SUPPORT
+               if (type->typ_sym_filter)
+                       goto dir_exist;
+#endif /* HAVE_SERVER_SUPPORT */
+               kobject_put(&type->typ_kobj);
                 CDEBUG(D_IOCTL, "Type %s already registered\n", name);
                 RETURN(-EEXIST);
         }
 
-        rc = -ENOMEM;
         OBD_ALLOC(type, sizeof(*type));
         if (type == NULL)
-                RETURN(rc);
+               RETURN(-ENOMEM);
 
+       type->typ_kobj.kset = lustre_kset;
+       kobject_init(&type->typ_kobj, &class_ktype);
+#ifdef HAVE_SERVER_SUPPORT
+dir_exist:
+#endif /* HAVE_SERVER_SUPPORT */
         OBD_ALLOC_PTR(type->typ_dt_ops);
         OBD_ALLOC_PTR(type->typ_md_ops);
-        OBD_ALLOC(type->typ_name, strlen(name) + 1);
 
         if (type->typ_dt_ops == NULL ||
-            type->typ_md_ops == NULL ||
-            type->typ_name == NULL)
-                GOTO (failed, rc);
+           type->typ_md_ops == NULL)
+               GOTO (failed, rc = -ENOMEM);
 
         *(type->typ_dt_ops) = *dt_ops;
         /* md_ops is optional */
         if (md_ops)
                 *(type->typ_md_ops) = *md_ops;
-        strcpy(type->typ_name, name);
        spin_lock_init(&type->obd_type_lock);
 
+#ifdef HAVE_SERVER_SUPPORT
+       if (type->typ_sym_filter) {
+               type->typ_sym_filter = false;
+               kobject_put(&type->typ_kobj);
+               goto setup_ldt;
+       }
+#endif
 #ifdef CONFIG_PROC_FS
-       if (enable_proc) {
-               type->typ_procroot = lprocfs_register(type->typ_name,
+       if (enable_proc && !type->typ_procroot) {
+               type->typ_procroot = lprocfs_register(name,
                                                      proc_lustre_root,
                                                      NULL, type);
                if (IS_ERR(type->typ_procroot)) {
@@ -247,21 +297,7 @@ int class_register_type(struct obd_ops *dt_ops, struct md_ops *md_ops,
                }
        }
 #endif
-#ifdef HAVE_SERVER_SUPPORT
-       dname.name = name;
-       dname.len = strlen(dname.name);
-       dname.hash = ll_full_name_hash(debugfs_lustre_root, dname.name,
-                                      dname.len);
-       type->typ_debugfs_entry = d_lookup(debugfs_lustre_root, &dname);
-       if (type->typ_debugfs_entry) {
-               dput(type->typ_debugfs_entry);
-               type->typ_sym_filter = true;
-               goto dir_exist;
-       }
-#endif /* HAVE_SERVER_SUPPORT */
-
-       type->typ_debugfs_entry = ldebugfs_register(type->typ_name,
-                                                   debugfs_lustre_root,
+       type->typ_debugfs_entry = ldebugfs_register(name, debugfs_lustre_root,
                                                    vars, type);
        if (IS_ERR_OR_NULL(type->typ_debugfs_entry)) {
                rc = type->typ_debugfs_entry ? PTR_ERR(type->typ_debugfs_entry)
@@ -269,54 +305,33 @@ int class_register_type(struct obd_ops *dt_ops, struct md_ops *md_ops,
                type->typ_debugfs_entry = NULL;
                GOTO(failed, rc);
        }
+
+       rc = kobject_add(&type->typ_kobj, &lustre_kset->kobj, "%s", name);
+       if (rc)
+               GOTO(failed, rc);
 #ifdef HAVE_SERVER_SUPPORT
-dir_exist:
+setup_ldt:
 #endif
-       type->typ_kobj = class_setup_tunables(type->typ_name);
-       if (IS_ERR(type->typ_kobj))
-               GOTO(failed, rc = PTR_ERR(type->typ_kobj));
-
        if (ldt) {
                type->typ_lu = ldt;
                rc = lu_device_type_init(ldt);
-               if (rc) {
-                       kobject_put(type->typ_kobj);
+               if (rc)
                        GOTO(failed, rc);
-               }
        }
 
-       spin_lock(&obd_types_lock);
-       list_add(&type->typ_chain, &obd_types);
-       spin_unlock(&obd_types_lock);
-
        RETURN(0);
 
 failed:
-#ifdef HAVE_SERVER_SUPPORT
-       if (type->typ_sym_filter)
-               type->typ_debugfs_entry = NULL;
-#endif
-       if (!IS_ERR_OR_NULL(type->typ_debugfs_entry))
-               ldebugfs_remove(&type->typ_debugfs_entry);
-       if (type->typ_name != NULL) {
-#ifdef CONFIG_PROC_FS
-               if (type->typ_procroot != NULL)
-                       remove_proc_subtree(type->typ_name, proc_lustre_root);
-#endif
-                OBD_FREE(type->typ_name, strlen(name) + 1);
-       }
-        if (type->typ_md_ops != NULL)
-                OBD_FREE_PTR(type->typ_md_ops);
-        if (type->typ_dt_ops != NULL)
-                OBD_FREE_PTR(type->typ_dt_ops);
-        OBD_FREE(type, sizeof(*type));
-        RETURN(rc);
+       kobject_put(&type->typ_kobj);
+
+       RETURN(rc);
 }
 EXPORT_SYMBOL(class_register_type);
 
 int class_unregister_type(const char *name)
 {
         struct obd_type *type = class_search_type(name);
+       int rc = 0;
         ENTRY;
 
         if (!type) {
@@ -330,41 +345,16 @@ int class_unregister_type(const char *name)
                 /* Remove ops, but leave the name for debugging */
                 OBD_FREE_PTR(type->typ_dt_ops);
                 OBD_FREE_PTR(type->typ_md_ops);
-                RETURN(-EBUSY);
+               GOTO(out_put, rc = -EBUSY);
         }
 
-       kobject_put(type->typ_kobj);
-
-       /* we do not use type->typ_procroot as for compatibility purposes
-        * other modules can share names (i.e. lod can use lov entry). so
-        * we can't reference pointer as it can get invalided when another
-        * module removes the entry */
-#ifdef CONFIG_PROC_FS
-       if (type->typ_procroot != NULL)
-               remove_proc_subtree(type->typ_name, proc_lustre_root);
-       if (type->typ_procsym != NULL)
-               lprocfs_remove(&type->typ_procsym);
-#endif
-#ifdef HAVE_SERVER_SUPPORT
-       if (type->typ_sym_filter)
-               type->typ_debugfs_entry = NULL;
-#endif
-       if (!IS_ERR_OR_NULL(type->typ_debugfs_entry))
-               ldebugfs_remove(&type->typ_debugfs_entry);
-
-        if (type->typ_lu)
-                lu_device_type_fini(type->typ_lu);
+       /* Put the final ref */
+       kobject_put(&type->typ_kobj);
+out_put:
+       /* Put the ref returned by class_search_type() */
+       kobject_put(&type->typ_kobj);
 
-       spin_lock(&obd_types_lock);
-       list_del(&type->typ_chain);
-       spin_unlock(&obd_types_lock);
-        OBD_FREE(type->typ_name, strlen(name) + 1);
-        if (type->typ_dt_ops != NULL)
-                OBD_FREE_PTR(type->typ_dt_ops);
-        if (type->typ_md_ops != NULL)
-                OBD_FREE_PTR(type->typ_md_ops);
-        OBD_FREE(type, sizeof(*type));
-        RETURN(0);
+       RETURN(rc);
 } /* class_unregister_type */
 EXPORT_SYMBOL(class_unregister_type);
 
@@ -784,7 +774,7 @@ void class_obd_list(void)
    specified, then only the client with that uuid is returned,
    otherwise any client connected to the tgt is returned. */
 struct obd_device * class_find_client_obd(struct obd_uuid *tgt_uuid,
-                                          const char * typ_name,
+                                         const char *type_name,
                                           struct obd_uuid *grp_uuid)
 {
         int i;
@@ -795,8 +785,8 @@ struct obd_device * class_find_client_obd(struct obd_uuid *tgt_uuid,
 
                 if (obd == NULL)
                         continue;
-                if ((strncmp(obd->obd_type->typ_name, typ_name,
-                             strlen(typ_name)) == 0)) {
+               if ((strncmp(obd->obd_type->typ_name, type_name,
+                            strlen(type_name)) == 0)) {
                         if (obd_uuid_equals(tgt_uuid,
                                             &obd->u.cli.cl_target_uuid) &&
                             ((grp_uuid)? obd_uuid_equals(grp_uuid,
@@ -910,9 +900,9 @@ int obd_init_caches(void)
        ENTRY;
 
        LASSERT(obd_device_cachep == NULL);
-       obd_device_cachep = kmem_cache_create("ll_obd_dev_cache",
-                                             sizeof(struct obd_device),
-                                             0, 0, NULL);
+       obd_device_cachep = kmem_cache_create_usercopy("ll_obd_dev_cache",
+                               sizeof(struct obd_device),
+                               0, 0, 0, sizeof(struct obd_device), NULL);
        if (!obd_device_cachep)
                GOTO(out, rc = -ENOMEM);
 
@@ -922,6 +912,8 @@ out:
        RETURN(rc);
 }
 
+static struct portals_handle_ops export_handle_ops;
+
 /* map connection to client */
 struct obd_export *class_conn2export(struct lustre_handle *conn)
 {
@@ -939,7 +931,7 @@ struct obd_export *class_conn2export(struct lustre_handle *conn)
         }
 
        CDEBUG(D_INFO, "looking for export cookie %#llx\n", conn->cookie);
-       export = class_handle2object(conn->cookie, NULL);
+       export = class_handle2object(conn->cookie, &export_handle_ops);
        RETURN(export);
 }
 EXPORT_SYMBOL(class_conn2export);
@@ -1315,7 +1307,7 @@ struct obd_import *class_new_import(struct obd_device *obd)
        imp->imp_last_success_conn = 0;
        imp->imp_state = LUSTRE_IMP_NEW;
        imp->imp_obd = class_incref(obd, "import", imp);
-       mutex_init(&imp->imp_sec_mutex);
+       rwlock_init(&imp->imp_sec_lock);
        init_waitqueue_head(&imp->imp_recovery_waitq);
        INIT_WORK(&imp->imp_zombie_work, obd_zombie_imp_cull);
 
@@ -2123,14 +2115,14 @@ int obd_set_max_rpcs_in_flight(struct client_obd *cli, __u32 max)
        __u32                           old;
        int                             diff;
        int                             i;
-       char                            *typ_name;
+       const char *type_name;
        int                             rc;
 
        if (max > OBD_MAX_RIF_MAX || max < 1)
                return -ERANGE;
 
-       typ_name = cli->cl_import->imp_obd->obd_type->typ_name;
-       if (strcmp(typ_name, LUSTRE_MDC_NAME) == 0) {
+       type_name = cli->cl_import->imp_obd->obd_type->typ_name;
+       if (strcmp(type_name, LUSTRE_MDC_NAME) == 0) {
                /* adjust max_mod_rpcs_in_flight to ensure it is always
                 * strictly lower that max_rpcs_in_flight */
                if (max < 2) {
@@ -2349,6 +2341,12 @@ __u16 obd_get_mod_rpc_slot(struct client_obd *cli, __u32 opc,
                        LASSERT(!test_and_set_bit(i, cli->cl_mod_tag_bitmap));
                        spin_unlock(&cli->cl_mod_rpcs_lock);
                        /* tag 0 is reserved for non-modify RPCs */
+
+                       CDEBUG(D_RPCTRACE, "%s: modify RPC slot %u is allocated"
+                              "opc %u, max %hu\n",
+                              cli->cl_import->imp_obd->obd_name,
+                              i + 1, opc, max);
+
                        return i + 1;
                }
                spin_unlock(&cli->cl_mod_rpcs_lock);