LU-5396 lod: (and mdt, mgs) make some symbols static

[fs/lustre-release.git] / lustre / mgs / mgs_fs.c

diff --git a/lustre/mgs/mgs_fs.c b/lustre/mgs/mgs_fs.c
index 5d2b99b..0ffa587 100644 (file)
--- a/lustre/mgs/mgs_fs.c
+++ b/lustre/mgs/mgs_fs.c
@@ -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, 2012, Intel Corporation.
+ * Copyright (c) 2011, 2014, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -46,44 +46,54 @@
 #include <lustre_fid.h>
 #include "mgs_internal.h"
 
+/**
+ * Initialize MGS per-export statistics.
+ *
+ * This function sets up procfs entries for various MGS export counters. These
+ * counters are for per-client statistics tracked on the server.
+ *
+ * \param[in] obd      OBD device
+ * \param[in] exp      OBD export
+ * \param[in] localdata        NID of client
+ *
+ * \retval             0 if successful
+ * \retval             negative value on error
+ */
 int mgs_export_stats_init(struct obd_device *obd, struct obd_export *exp,
                          void *localdata)
 {
        lnet_nid_t *client_nid = localdata;
-       struct nid_stat *tmp;
-       int rc, is_new_nid;
+       struct nid_stat *stats;
+       int rc;
        ENTRY;
 
-       rc = lprocfs_exp_setup(exp, client_nid, &is_new_nid);
-       if (rc != 0) {
-               /* Mask error for already created /proc entries */
-               if (rc == -EALREADY)
-                       rc = 0;
-               GOTO(out, rc = 0);
-        }
-
-       if (!is_new_nid)
-               GOTO(out, rc = 0);
-
-       tmp = exp->exp_nid_stats;
-       tmp->nid_stats = lprocfs_alloc_stats(NUM_OBD_STATS + LPROC_MGS_LAST,
-                                            LPROCFS_STATS_FLAG_NOPERCPU);
-       if (tmp->nid_stats == NULL)
-               GOTO(out, rc = -ENOMEM);
-
-       lprocfs_init_ops_stats(LPROC_MGS_LAST, tmp->nid_stats);
-       mgs_stats_counter_init(tmp->nid_stats);
-       rc = lprocfs_register_stats(tmp->nid_proc, "stats", tmp->nid_stats);
+       rc = lprocfs_exp_setup(exp, client_nid);
        if (rc != 0)
+               /* Mask error for already created /proc entries */
+               RETURN(rc == -EALREADY ? 0 : rc);
+
+       stats = exp->exp_nid_stats;
+       stats->nid_stats = lprocfs_alloc_stats(NUM_OBD_STATS + LPROC_MGS_LAST,
+                                               LPROCFS_STATS_FLAG_NOPERCPU);
+       if (stats->nid_stats == NULL)
+               RETURN(-ENOMEM);
+
+       lprocfs_init_ops_stats(LPROC_MGS_LAST, stats->nid_stats);
+
+       mgs_stats_counter_init(stats->nid_stats);
+
+       rc = lprocfs_register_stats(stats->nid_proc, "stats", stats->nid_stats);
+       if (rc != 0) {
+               lprocfs_free_stats(&stats->nid_stats);
                GOTO(out, rc);
+       }
 
-       rc = lprocfs_nid_ldlm_stats_init(tmp);
+       rc = lprocfs_nid_ldlm_stats_init(stats);
        if (rc != 0)
                GOTO(out, rc);
 
-       RETURN(0);
 out:
-       return rc;
+       RETURN(rc);
 }
 
 /**
@@ -92,7 +102,7 @@ out:
  * crash all connections are treated as new connections.
  */
 int mgs_client_add(struct obd_device *obd, struct obd_export *exp,
-                   void *localdata)
+                         void *localdata)
 {
         return 0;
 }
@@ -113,11 +123,6 @@ int mgs_fs_setup(const struct lu_env *env, struct mgs_device *mgs)
 
        ENTRY;
 
-       /* FIXME what's this?  Do I need it? */
-       rc = cfs_cleanup_group_info();
-       if (rc)
-               RETURN(rc);
-
        OBD_SET_CTXT_MAGIC(&mgs->mgs_obd->obd_lvfs_ctxt);
        mgs->mgs_obd->obd_lvfs_ctxt.dt = mgs->mgs_bottom;