Whamcloud - gitweb
LU-6142 fld: Fix style issues for fld_handler.c
[fs/lustre-release.git] / lustre / fld / fld_handler.c
index fbf0369..9b84b62 100644 (file)
  *
  * You should have received a copy of the GNU General Public License
  * version 2 along with this program; If not, see
- * http://www.sun.com/software/products/lustre/docs/GPLv2.pdf
- *
- * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
- * CA 95054 USA or visit www.sun.com if you need additional information or
- * have any questions.
+ * http://www.gnu.org/licenses/gpl-2.0.html
  *
  * GPL HEADER END
  */
@@ -27,7 +23,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, 2015, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -51,7 +47,6 @@
 #include <obd_support.h>
 #include <lustre_fid.h>
 #include <lustre_fld.h>
-#include <md_object.h>
 #include <lustre_req_layout.h>
 #include <lprocfs_status.h>
 #include "fld_internal.h"
@@ -108,15 +103,16 @@ EXPORT_SYMBOL(fld_server_create);
 /**
  * Extract index information from fld name like srv-fsname-MDT0000
  **/
-int fld_name_to_index(const char *name, __u32 *index)
+int fld_name_to_index(const char *name, u32 *index)
 {
        char *dash;
        int rc;
+
        ENTRY;
 
        CDEBUG(D_INFO, "get index from %s\n", name);
        dash = strrchr(name, '-');
-       if (dash == NULL)
+       if (!dash)
                RETURN(-EINVAL);
        dash++;
        rc = target_name2index(dash, index, NULL);
@@ -129,17 +125,20 @@ int fld_name_to_index(const char *name, __u32 *index)
 int fld_update_from_controller(const struct lu_env *env,
                               struct lu_server_fld *fld)
 {
-       struct fld_thread_info    *info;
-       struct lu_seq_range       *range;
+       struct fld_thread_info *info;
+       struct lu_seq_range *range;
        struct lu_seq_range_array *lsra;
-       __u32                     index;
-       struct ptlrpc_request     *req;
-       int                       rc;
-       int                       i;
+       u32 index;
+       struct ptlrpc_request *req;
+       int rc;
+       int i;
+
        ENTRY;
 
-       /* Update only happens during initalization, i.e. local FLDB
-        * does not exist yet */
+       /*
+        * Update only happens during initalization, i.e. local FLDB
+        * does not exist yet
+        */
        if (!fld->lsf_new)
                RETURN(0);
 
@@ -167,7 +166,7 @@ int fld_update_from_controller(const struct lu_env *env,
                LASSERT(req != NULL);
                lsra = (struct lu_seq_range_array *)req_capsule_server_get(
                                          &req->rq_pill, &RMF_GENERIC_DATA);
-               if (lsra == NULL)
+               if (!lsra)
                        GOTO(out, rc = -EPROTO);
 
                range_array_le_to_cpu(lsra, lsra);
@@ -180,7 +179,10 @@ int fld_update_from_controller(const struct lu_env *env,
                        if (lsra->lsra_lsr[i].lsr_index != index)
                                GOTO(out, rc = -EINVAL);
 
+                       mutex_lock(&fld->lsf_lock);
                        rc1 = fld_insert_entry(env, fld, &lsra->lsra_lsr[i]);
+                       mutex_unlock(&fld->lsf_lock);
+
                        if (rc1 != 0)
                                GOTO(out, rc = rc1);
                }
@@ -190,7 +192,7 @@ int fld_update_from_controller(const struct lu_env *env,
 
        fld->lsf_new = 1;
 out:
-       if (req != NULL)
+       if (req)
                ptlrpc_req_finished(req);
 
        RETURN(rc);
@@ -201,11 +203,12 @@ EXPORT_SYMBOL(fld_update_from_controller);
  * Lookup sequece in local cache/fldb.
  **/
 int fld_local_lookup(const struct lu_env *env, struct lu_server_fld *fld,
-                    seqno_t seq, struct lu_seq_range *range)
+                    u64 seq, struct lu_seq_range *range)
 {
        struct lu_seq_range *erange;
        struct fld_thread_info *info;
        int rc;
+
        ENTRY;
 
        info = lu_context_key_get(&env->le_ctx, &fld_thread_key);
@@ -217,9 +220,9 @@ int fld_local_lookup(const struct lu_env *env, struct lu_server_fld *fld,
        if (rc == 0) {
                if (unlikely(fld_range_type(erange) != fld_range_type(range) &&
                             !fld_range_is_any(range))) {
-                       CERROR("%s: FLD cache range "DRANGE" does not match"
-                              "requested flag %x: rc = %d\n", fld->lsf_name,
-                              PRANGE(erange), range->lsr_flags, -EIO);
+                       CERROR("%s: FLD cache range "DRANGE" does not match requested flag %x: rc = %d\n",
+                              fld->lsf_name, PRANGE(erange), range->lsr_flags,
+                              -EIO);
                        RETURN(-EIO);
                }
                *range = *erange;
@@ -237,10 +240,11 @@ EXPORT_SYMBOL(fld_local_lookup);
  *  cache fld entries, but this cache is not persistent.
  */
 int fld_server_lookup(const struct lu_env *env, struct lu_server_fld *fld,
-                     seqno_t seq, struct lu_seq_range *range)
+                     u64 seq, struct lu_seq_range *range)
 {
-       __u32 index;
+       u32 index;
        int rc;
+
        ENTRY;
 
        rc = fld_local_lookup(env, fld, seq, range);
@@ -252,18 +256,21 @@ int fld_server_lookup(const struct lu_env *env, struct lu_server_fld *fld,
                RETURN(rc);
 
        if (index == 0 && rc == LDD_F_SV_TYPE_MDT) {
-               /* On server side, all entries should be in cache.
-                * If we can not find it in cache, just return error */
-               CERROR("%s: Cannot find sequence "LPX64": rc = %d\n",
+               /*
+                * On server side, all entries should be in cache.
+                * If we can not find it in cache, just return error
+                */
+               CERROR("%s: Cannot find sequence %#llx: rc = %d\n",
                       fld->lsf_name, seq, -ENOENT);
                RETURN(-ENOENT);
        } else {
-               if (fld->lsf_control_exp == NULL) {
-                       CERROR("%s: lookup "LPX64", but not connects to MDT0"
-                              "yet: rc = %d.\n", fld->lsf_name, seq, -EIO);
+               if (!fld->lsf_control_exp) {
+                       CERROR("%s: lookup %#llx, but not connects to MDT0 yet: rc = %d.\n",
+                              fld->lsf_name, seq, -EIO);
                        RETURN(-EIO);
                }
-               /* send request to mdt0 i.e. super seq. controller.
+               /*
+                * send request to mdt0 i.e. super seq. controller.
                 * This is temporary solution, long term solution is fld
                 * replication on all mdt servers.
                 */
@@ -283,17 +290,17 @@ EXPORT_SYMBOL(fld_server_lookup);
  */
 static int fld_handle_lookup(struct tgt_session_info *tsi)
 {
-       struct obd_export       *exp = tsi->tsi_exp;
-       struct lu_site          *site = exp->exp_obd->obd_lu_dev->ld_site;
-       struct lu_server_fld    *fld;
-       struct lu_seq_range     *in;
-       struct lu_seq_range     *out;
-       int                     rc;
+       struct obd_export *exp = tsi->tsi_exp;
+       struct lu_site *site = exp->exp_obd->obd_lu_dev->ld_site;
+       struct lu_server_fld *fld;
+       struct lu_seq_range *in;
+       struct lu_seq_range *out;
+       int rc;
 
        ENTRY;
 
        in = req_capsule_client_get(tsi->tsi_pill, &RMF_FLD_MDFLD);
-       if (in == NULL)
+       if (!in)
                RETURN(err_serious(-EPROTO));
 
        rc = req_capsule_server_pack(tsi->tsi_pill);
@@ -301,7 +308,7 @@ static int fld_handle_lookup(struct tgt_session_info *tsi)
                RETURN(err_serious(rc));
 
        out = req_capsule_server_get(tsi->tsi_pill, &RMF_FLD_MDFLD);
-       if (out == NULL)
+       if (!out)
                RETURN(err_serious(-EPROTO));
        *out = *in;
 
@@ -317,22 +324,22 @@ static int fld_handle_lookup(struct tgt_session_info *tsi)
 
 static int fld_handle_read(struct tgt_session_info *tsi)
 {
-       struct obd_export       *exp = tsi->tsi_exp;
-       struct lu_site          *site = exp->exp_obd->obd_lu_dev->ld_site;
-       struct lu_seq_range     *in;
-       void                    *data;
-       int                     rc;
+       struct obd_export *exp = tsi->tsi_exp;
+       struct lu_site *site = exp->exp_obd->obd_lu_dev->ld_site;
+       struct lu_seq_range *in;
+       void *data;
+       int rc;
 
        ENTRY;
 
        req_capsule_set(tsi->tsi_pill, &RQF_FLD_READ);
 
        in = req_capsule_client_get(tsi->tsi_pill, &RMF_FLD_MDFLD);
-       if (in == NULL)
+       if (!in)
                RETURN(err_serious(-EPROTO));
 
        req_capsule_set_size(tsi->tsi_pill, &RMF_GENERIC_DATA, RCL_SERVER,
-                            PAGE_CACHE_SIZE);
+                            PAGE_SIZE);
 
        rc = req_capsule_server_pack(tsi->tsi_pill);
        if (unlikely(rc != 0))
@@ -341,7 +348,7 @@ static int fld_handle_read(struct tgt_session_info *tsi)
        data = req_capsule_server_get(tsi->tsi_pill, &RMF_GENERIC_DATA);
 
        rc = fld_server_read(tsi->tsi_env, lu_site2seq(site)->ss_server_fld,
-                            in, data, PAGE_CACHE_SIZE);
+                            in, data, PAGE_SIZE);
        RETURN(rc);
 }
 
@@ -367,12 +374,13 @@ static int fld_handle_query(struct tgt_session_info *tsi)
  * fid_is_local() is supposed to be used in assertion checks only.
  */
 int fid_is_local(const struct lu_env *env,
-                 struct lu_site *site, const struct lu_fid *fid)
+                struct lu_site *site, const struct lu_fid *fid)
 {
        int result;
        struct seq_server_site *ss_site;
        struct lu_seq_range *range;
        struct fld_thread_info *info;
+
        ENTRY;
 
        info = lu_context_key_get(&env->le_ctx, &fld_thread_key);
@@ -380,7 +388,7 @@ int fid_is_local(const struct lu_env *env,
 
        result = 1; /* conservatively assume fid is local */
        ss_site = lu_site2seq(site);
-       if (ss_site->ss_client_fld != NULL) {
+       if (ss_site->ss_client_fld) {
                int rc;
 
                rc = fld_cache_lookup(ss_site->ss_client_fld->lcf_cache,
@@ -390,56 +398,36 @@ int fid_is_local(const struct lu_env *env,
        }
        return result;
 }
-EXPORT_SYMBOL(fid_is_local);
 
-static void fld_server_proc_fini(struct lu_server_fld *fld);
+static void fld_server_debugfs_fini(struct lu_server_fld *fld)
+{
+       if (!IS_ERR_OR_NULL(fld->lsf_debugfs_entry))
+               ldebugfs_remove(&fld->lsf_debugfs_entry);
+}
 
-#ifdef LPROCFS
-static int fld_server_proc_init(struct lu_server_fld *fld)
+static int fld_server_debugfs_init(struct lu_server_fld *fld)
 {
-        int rc = 0;
-        ENTRY;
-
-       fld->lsf_proc_dir = lprocfs_seq_register(fld->lsf_name,
-                                                fld_type_proc_dir,
-                                                fld_server_proc_list, fld);
-       if (IS_ERR(fld->lsf_proc_dir)) {
-               rc = PTR_ERR(fld->lsf_proc_dir);
+       int rc = 0;
+
+       ENTRY;
+       fld->lsf_debugfs_entry = ldebugfs_register(fld->lsf_name,
+                                                  fld_debugfs_dir,
+                                                  NULL, NULL);
+       if (IS_ERR_OR_NULL(fld->lsf_debugfs_entry)) {
+               rc = fld->lsf_debugfs_entry ? PTR_ERR(fld->lsf_debugfs_entry)
+                                           : -ENOMEM;
+               fld->lsf_debugfs_entry = NULL;
                RETURN(rc);
        }
 
-       rc = lprocfs_seq_create(fld->lsf_proc_dir, "fldb", 0444,
-                               &fld_proc_seq_fops, fld);
-       if (rc) {
-               lprocfs_remove(&fld->lsf_proc_dir);
-               fld->lsf_proc_dir = NULL;
-       }
+       rc = ldebugfs_seq_create(fld->lsf_debugfs_entry, "fldb", 0444,
+                                &fld_debugfs_seq_fops, fld);
+       if (rc)
+               ldebugfs_remove(&fld->lsf_debugfs_entry);
 
        RETURN(rc);
 }
 
-static void fld_server_proc_fini(struct lu_server_fld *fld)
-{
-        ENTRY;
-        if (fld->lsf_proc_dir != NULL) {
-                if (!IS_ERR(fld->lsf_proc_dir))
-                        lprocfs_remove(&fld->lsf_proc_dir);
-                fld->lsf_proc_dir = NULL;
-        }
-        EXIT;
-}
-#else
-static int fld_server_proc_init(struct lu_server_fld *fld)
-{
-        return 0;
-}
-
-static void fld_server_proc_fini(struct lu_server_fld *fld)
-{
-        return;
-}
-#endif
-
 int fld_server_init(const struct lu_env *env, struct lu_server_fld *fld,
                    struct dt_device *dt, const char *prefix, int type)
 {
@@ -467,12 +455,14 @@ int fld_server_init(const struct lu_env *env, struct lu_server_fld *fld,
        if (rc)
                GOTO(out_cache, rc);
 
-       rc = fld_server_proc_init(fld);
+       rc = fld_server_debugfs_init(fld);
        if (rc)
                GOTO(out_index, rc);
 
        fld->lsf_control_exp = NULL;
+       fld->lsf_seq_lookup = fld_server_lookup;
 
+       fld->lsf_seq_lookup = fld_server_lookup;
        RETURN(0);
 out_index:
        fld_index_fini(env, fld);
@@ -486,10 +476,10 @@ void fld_server_fini(const struct lu_env *env, struct lu_server_fld *fld)
 {
        ENTRY;
 
-       fld_server_proc_fini(fld);
+       fld_server_debugfs_fini(fld);
        fld_index_fini(env, fld);
 
-       if (fld->lsf_cache != NULL) {
+       if (fld->lsf_cache) {
                if (!IS_ERR(fld->lsf_cache))
                        fld_cache_fini(fld->lsf_cache);
                fld->lsf_cache = NULL;