Whamcloud - gitweb
LU-6070 libcfs: provide separate buffers for libcfs_*2str()
[fs/lustre-release.git] / lustre / mgs / lproc_mgs.c
index 732e362..2e4adf4 100644 (file)
@@ -27,7 +27,7 @@
  * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2011, 2013, Intel Corporation.
+ * Copyright (c) 2011, 2014, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
 #include <lustre_param.h>
 #include "mgs_internal.h"
 
-#ifdef LPROCFS
+#ifdef CONFIG_PROC_FS
 
 static int mgs_fs_seq_show(struct seq_file *seq, void *v)
 {
-        struct obd_device *obd = seq->private;
-       struct mgs_device *mgs;
-       cfs_list_t list;
-       struct mgs_direntry *dirent, *n;
-       struct lu_env env;
-        int rc, len;
-        ENTRY;
-
-        LASSERT(obd != NULL);
+       struct obd_device       *obd = seq->private;
+       struct mgs_device       *mgs;
+       struct list_head         list;
+       struct mgs_direntry     *dirent, *n;
+       struct lu_env            env;
+       int rc, len;
+
+       ENTRY;
+
+       LASSERT(obd != NULL);
        LASSERT(obd->obd_lu_dev != NULL);
        mgs = lu2mgs_dev(obd->obd_lu_dev);
 
@@ -64,22 +65,21 @@ static int mgs_fs_seq_show(struct seq_file *seq, void *v)
                RETURN(rc);
 
        rc = class_dentry_readdir(&env, mgs, &list);
-        if (rc) {
-                CERROR("Can't read config dir\n");
+       if (rc)
                GOTO(out, rc);
-        }
-       cfs_list_for_each_entry_safe(dirent, n, &list, list) {
-               cfs_list_del(&dirent->list);
-               len = strlen(dirent->name);
+
+       list_for_each_entry_safe(dirent, n, &list, mde_list) {
+               list_del_init(&dirent->mde_list);
+               len = strlen(dirent->mde_name);
                if (len > 7 &&
-                   strncmp(dirent->name + len - 7, "-client", len) == 0)
-                       seq_printf(seq, "%.*s\n", len - 7, dirent->name);
+                   strncmp(dirent->mde_name + len - 7, "-client", len) == 0)
+                       seq_printf(seq, "%.*s\n", len - 7, dirent->mde_name);
                mgs_direntry_free(dirent);
-        }
+       }
 
 out:
        lu_env_fini(&env);
-        RETURN(0);
+       RETURN(0);
 }
 
 LPROC_SEQ_FOPS_RO(mgs_fs);
@@ -90,16 +90,14 @@ static void seq_show_srpc_rules(struct seq_file *seq, const char *tgtname,
         struct sptlrpc_rule    *r;
         char                    dirbuf[10];
         char                    flvrbuf[40];
-        char                   *net;
+       char                    net[LNET_NIDSTR_SIZE] = "default";
         int                     i;
 
         for (i = 0; i < rset->srs_nrule; i++) {
                 r = &rset->srs_rules[i];
 
-                if (r->sr_netid == LNET_NIDNET(LNET_NID_ANY))
-                        net = "default";
-                else
-                        net = libcfs_net2str(r->sr_netid);
+               if (r->sr_netid != LNET_NIDNET(LNET_NID_ANY))
+                       libcfs_net2str_r(r->sr_netid, net, sizeof(net));
 
                 if (r->sr_from == LUSTRE_SP_ANY && r->sr_to == LUSTRE_SP_ANY)
                         dirbuf[0] = '\0';
@@ -116,34 +114,110 @@ static void seq_show_srpc_rules(struct seq_file *seq, const char *tgtname,
 
 static int mgsself_srpc_seq_show(struct seq_file *seq, void *v)
 {
-        struct obd_device *obd = seq->private;
-       struct mgs_device *mgs;
-        struct fs_db      *fsdb;
-       struct lu_env      env;
-        int                rc;
+       struct obd_device       *obd = seq->private;
+       struct mgs_device       *mgs = lu2mgs_dev(obd->obd_lu_dev);
+       struct lu_target        *tgt = &mgs->mgs_lut;
 
-       LASSERT(obd != NULL);
-       LASSERT(obd->obd_lu_dev != NULL);
-       mgs = lu2mgs_dev(obd->obd_lu_dev);
+       read_lock(&tgt->lut_sptlrpc_lock);
+       seq_show_srpc_rules(seq, MGSSELF_NAME, &tgt->lut_sptlrpc_rset);
+       read_unlock(&tgt->lut_sptlrpc_lock);
 
-       rc = lu_env_init(&env, LCT_MG_THREAD);
-       if (rc)
-               return rc;
+        return 0;
+}
+LPROC_SEQ_FOPS_RO(mgsself_srpc);
 
-       rc = mgs_find_or_make_fsdb(&env, mgs, MGSSELF_NAME, &fsdb);
-        if (rc)
-               goto out;
+static int mgs_live_seq_show(struct seq_file *seq, void *v)
+{
+       struct fs_db             *fsdb = seq->private;
+       struct mgs_tgt_srpc_conf *srpc_tgt;
+       int i;
 
        mutex_lock(&fsdb->fsdb_mutex);
-        seq_show_srpc_rules(seq, fsdb->fsdb_name, &fsdb->fsdb_srpc_gen);
+
+       seq_printf(seq, "fsname: %s\n", fsdb->fsdb_name);
+       seq_printf(seq, "flags: %#lx     gen: %d\n",
+                  fsdb->fsdb_flags, fsdb->fsdb_gen);
+       for (i = 0; i < INDEX_MAP_SIZE * 8; i++)
+               if (test_bit(i, fsdb->fsdb_mdt_index_map))
+                        seq_printf(seq, "%s-MDT%04x\n", fsdb->fsdb_name, i);
+       for (i = 0; i < INDEX_MAP_SIZE * 8; i++)
+               if (test_bit(i, fsdb->fsdb_ost_index_map))
+                        seq_printf(seq, "%s-OST%04x\n", fsdb->fsdb_name, i);
+
+       seq_printf(seq, "\nSecure RPC Config Rules:\n");
+#if 0
+       seq_printf(seq, "%s.%s=%s\n", fsdb->fsdb_name,
+                  PARAM_SRPC_UDESC, fsdb->fsdb_srpc_fl_udesc ? "yes" : "no");
+#endif
+       for (srpc_tgt = fsdb->fsdb_srpc_tgt; srpc_tgt;
+            srpc_tgt = srpc_tgt->mtsc_next) {
+               seq_show_srpc_rules(seq, srpc_tgt->mtsc_tgt,
+                                   &srpc_tgt->mtsc_rset);
+       }
+       seq_show_srpc_rules(seq, fsdb->fsdb_name, &fsdb->fsdb_srpc_gen);
+
+       lprocfs_rd_ir_state(seq, fsdb);
+
        mutex_unlock(&fsdb->fsdb_mutex);
+       return 0;
+}
 
-out:
-       lu_env_fini(&env);
-        return 0;
+static ssize_t mgs_live_seq_write(struct file *file, const char *buf,
+                                 size_t len, loff_t *off)
+{
+       struct seq_file *seq  = file->private_data;
+       struct fs_db    *fsdb = seq->private;
+       ssize_t rc;
+
+       rc = lprocfs_wr_ir_state(file, buf, len, fsdb);
+       if (rc >= 0)
+               rc = len;
+       return rc;
 }
+LPROC_SEQ_FOPS(mgs_live);
 
-LPROC_SEQ_FOPS_RO(mgsself_srpc);
+int lproc_mgs_add_live(struct mgs_device *mgs, struct fs_db *fsdb)
+{
+       int rc;
+
+       if (!mgs->mgs_proc_live)
+               return 0;
+       rc = lprocfs_seq_create(mgs->mgs_proc_live, fsdb->fsdb_name, 0644,
+                               &mgs_live_fops, fsdb);
+
+       return 0;
+}
+
+int lproc_mgs_del_live(struct mgs_device *mgs, struct fs_db *fsdb)
+{
+       if (!mgs->mgs_proc_live)
+               return 0;
+
+       /* didn't create the proc file for MGSSELF_NAME */
+       if (!test_bit(FSDB_MGS_SELF, &fsdb->fsdb_flags))
+               lprocfs_remove_proc_entry(fsdb->fsdb_name, mgs->mgs_proc_live);
+       return 0;
+}
+
+LPROC_SEQ_FOPS_RO_TYPE(mgs, uuid);
+LPROC_SEQ_FOPS_RO_TYPE(mgs, num_exports);
+LPROC_SEQ_FOPS_RO_TYPE(mgs, hash);
+LPROC_SEQ_FOPS_WO_TYPE(mgs, evict_client);
+LPROC_SEQ_FOPS_RW_TYPE(mgs, ir_timeout);
+
+static struct lprocfs_vars lprocfs_mgs_obd_vars[] = {
+       { .name =       "uuid",
+         .fops =       &mgs_uuid_fops          },
+       { .name =       "num_exports",
+         .fops =       &mgs_num_exports_fops   },
+       { .name =       "hash_stats",
+         .fops =       &mgs_hash_fops          },
+       { .name =       "evict_client",
+         .fops =       &mgs_evict_client_fops  },
+       { .name =       "ir_timeout",
+         .fops =       &mgs_ir_timeout_fops    },
+       { NULL }
+};
 
 int lproc_mgs_setup(struct mgs_device *mgs, const char *osd_name)
 {
@@ -151,10 +225,9 @@ int lproc_mgs_setup(struct mgs_device *mgs, const char *osd_name)
        struct obd_device *osd_obd = mgs->mgs_bottom->dd_lu_dev.ld_obd;
        int                osd_len = strlen(osd_name) - strlen("-osd");
        int                rc;
-       struct lprocfs_static_vars lvars;
 
-       lprocfs_mgs_init_vars(&lvars);
-       rc = lprocfs_obd_setup(obd, lvars.obd_vars);
+       obd->obd_vars = lprocfs_mgs_obd_vars;
+       rc = lprocfs_obd_setup(obd);
        if (rc != 0)
                GOTO(out, rc);
 
@@ -168,17 +241,17 @@ int lproc_mgs_setup(struct mgs_device *mgs, const char *osd_name)
        if (rc != 0)
                GOTO(out, rc);
 
-        mgs->mgs_proc_live = lprocfs_register("live", obd->obd_proc_entry,
-                                              NULL, NULL);
+       mgs->mgs_proc_live = lprocfs_register("live", obd->obd_proc_entry,
+                                             NULL, NULL);
         if (IS_ERR(mgs->mgs_proc_live)) {
                 rc = PTR_ERR(mgs->mgs_proc_live);
                 mgs->mgs_proc_live = NULL;
                GOTO(out, rc);
         }
 
-        obd->obd_proc_exports_entry = lprocfs_register("exports",
-                                                       obd->obd_proc_entry,
-                                                       NULL, NULL);
+       obd->obd_proc_exports_entry = lprocfs_register("exports",
+                                                      obd->obd_proc_entry,
+                                                      NULL, NULL);
         if (IS_ERR(obd->obd_proc_exports_entry)) {
                 rc = PTR_ERR(obd->obd_proc_exports_entry);
                 obd->obd_proc_exports_entry = NULL;
@@ -231,7 +304,6 @@ void lproc_mgs_cleanup(struct mgs_device *mgs)
 
        if (mgs->mgs_proc_live != NULL) {
                /* Should be no live entries */
-               LASSERT(mgs->mgs_proc_live->subdir == NULL);
                lprocfs_remove(&mgs->mgs_proc_live);
                mgs->mgs_proc_live = NULL;
        }
@@ -242,92 +314,6 @@ void lproc_mgs_cleanup(struct mgs_device *mgs)
         lprocfs_free_md_stats(obd);
 }
 
-static int mgs_live_seq_show(struct seq_file *seq, void *v)
-{
-        struct fs_db             *fsdb = seq->private;
-        struct mgs_tgt_srpc_conf *srpc_tgt;
-        int i;
-
-       mutex_lock(&fsdb->fsdb_mutex);
-
-        seq_printf(seq, "fsname: %s\n", fsdb->fsdb_name);
-        seq_printf(seq, "flags: %#lx     gen: %d\n",
-                   fsdb->fsdb_flags, fsdb->fsdb_gen);
-        for (i = 0; i < INDEX_MAP_SIZE * 8; i++)
-               if (test_bit(i, fsdb->fsdb_mdt_index_map))
-                         seq_printf(seq, "%s-MDT%04x\n", fsdb->fsdb_name, i);
-        for (i = 0; i < INDEX_MAP_SIZE * 8; i++)
-               if (test_bit(i, fsdb->fsdb_ost_index_map))
-                         seq_printf(seq, "%s-OST%04x\n", fsdb->fsdb_name, i);
-
-        seq_printf(seq, "\nSecure RPC Config Rules:\n");
-#if 0
-        seq_printf(seq, "%s.%s=%s\n", fsdb->fsdb_name,
-                   PARAM_SRPC_UDESC, fsdb->fsdb_srpc_fl_udesc ? "yes" : "no");
-#endif
-        for (srpc_tgt = fsdb->fsdb_srpc_tgt; srpc_tgt;
-             srpc_tgt = srpc_tgt->mtsc_next) {
-                seq_show_srpc_rules(seq, srpc_tgt->mtsc_tgt,
-                                    &srpc_tgt->mtsc_rset);
-        }
-        seq_show_srpc_rules(seq, fsdb->fsdb_name, &fsdb->fsdb_srpc_gen);
-
-        lprocfs_rd_ir_state(seq, fsdb);
-
-       mutex_unlock(&fsdb->fsdb_mutex);
-        return 0;
-}
-
-static ssize_t mgs_live_seq_write(struct file *file, const char *buf,
-                                  size_t len, loff_t *off)
-{
-        struct seq_file *seq  = file->private_data;
-        struct fs_db    *fsdb = seq->private;
-        ssize_t rc;
-
-        rc = lprocfs_wr_ir_state(file, buf, len, fsdb);
-        if (rc >= 0)
-                rc = len;
-        return rc;
-}
-LPROC_SEQ_FOPS(mgs_live);
-
-int lproc_mgs_add_live(struct mgs_device *mgs, struct fs_db *fsdb)
-{
-        int rc;
-
-        if (!mgs->mgs_proc_live)
-                return 0;
-       rc = lprocfs_seq_create(mgs->mgs_proc_live, fsdb->fsdb_name, 0644,
-                               &mgs_live_fops, fsdb);
-
-        return 0;
-}
-
-int lproc_mgs_del_live(struct mgs_device *mgs, struct fs_db *fsdb)
-{
-        if (!mgs->mgs_proc_live)
-                return 0;
-
-       /* didn't create the proc file for MGSSELF_NAME */
-       if (!test_bit(FSDB_MGS_SELF, &fsdb->fsdb_flags))
-               lprocfs_remove_proc_entry(fsdb->fsdb_name, mgs->mgs_proc_live);
-        return 0;
-}
-
-struct lprocfs_vars lprocfs_mgs_obd_vars[] = {
-        { "uuid",            lprocfs_rd_uuid,        0, 0 },
-        { "num_exports",     lprocfs_rd_num_exports, 0, 0 },
-        { "hash_stats",      lprocfs_obd_rd_hash,    0, 0 },
-        { "evict_client",    0, lprocfs_wr_evict_client, 0 },
-        { "ir_timeout",      lprocfs_rd_ir_timeout, lprocfs_wr_ir_timeout, 0 },
-        { 0 }
-};
-
-struct lprocfs_vars lprocfs_mgs_module_vars[] = {
-        { 0 }
-};
-
 void mgs_counter_incr(struct obd_export *exp, int opcode)
 {
         lprocfs_counter_incr(exp->exp_obd->obd_stats, opcode);
@@ -345,10 +331,4 @@ void mgs_stats_counter_init(struct lprocfs_stats *stats)
         lprocfs_counter_init(stats, LPROC_MGS_TARGET_REG, 0, "tgtreg", "reqs");
         lprocfs_counter_init(stats, LPROC_MGS_TARGET_DEL, 0, "tgtdel", "reqs");
 }
-
-void lprocfs_mgs_init_vars(struct lprocfs_static_vars *lvars)
-{
-    lvars->module_vars  = lprocfs_mgs_module_vars;
-    lvars->obd_vars     = lprocfs_mgs_obd_vars;
-}
 #endif