Whamcloud - gitweb
LU-10850 fld: handle empty ranges in fld_index_init()
[fs/lustre-release.git] / lustre / fld / fld_index.c
index a6f9be5..7399206 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, 2017, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
 
 #define DEBUG_SUBSYSTEM S_FLD
 
-#ifdef __KERNEL__
-# include <libcfs/libcfs.h>
-# include <linux/module.h>
-# include <linux/jbd.h>
-#else /* __KERNEL__ */
-# include <liblustre.h>
-#endif
-
-#include <obd.h>
-#include <obd_class.h>
-#include <lustre_ver.h>
+#include <libcfs/libcfs.h>
+#include <linux/module.h>
 #include <obd_support.h>
-#include <lprocfs_status.h>
-
 #include <dt_object.h>
-#include <md_object.h>
-#include <lustre_mdc.h>
 #include <lustre_fid.h>
 #include <lustre_fld.h>
 #include "fld_internal.h"
 
-const char fld_index_name[] = "fld";
+static const char fld_index_name[] = "fld";
 
 static const struct lu_seq_range IGIF_FLD_RANGE = {
        .lsr_start = FID_SEQ_IGIF,
@@ -71,6 +54,13 @@ static const struct lu_seq_range IGIF_FLD_RANGE = {
        .lsr_flags = LU_SEQ_RANGE_MDT
 };
 
+static const struct lu_seq_range DOT_LUSTRE_FLD_RANGE = {
+       .lsr_start = FID_SEQ_DOT_LUSTRE,
+       .lsr_end   = FID_SEQ_DOT_LUSTRE + 1,
+       .lsr_index = 0,
+       .lsr_flags = LU_SEQ_RANGE_MDT
+};
+
 static const struct lu_seq_range ROOT_FLD_RANGE = {
        .lsr_start = FID_SEQ_ROOT,
        .lsr_end   = FID_SEQ_ROOT + 1,
@@ -78,10 +68,10 @@ static const struct lu_seq_range ROOT_FLD_RANGE = {
        .lsr_flags = LU_SEQ_RANGE_MDT
 };
 
-const struct dt_index_features fld_index_features = {
+static const struct dt_index_features fld_index_features = {
        .dif_flags       = DT_IND_UPDATE,
-       .dif_keysize_min = sizeof(seqno_t),
-       .dif_keysize_max = sizeof(seqno_t),
+       .dif_keysize_min = sizeof(u64),
+       .dif_keysize_max = sizeof(u64),
        .dif_recsize_min = sizeof(struct lu_seq_range),
        .dif_recsize_max = sizeof(struct lu_seq_range),
        .dif_ptrsize     = 4
@@ -109,7 +99,7 @@ int fld_declare_index_create(const struct lu_env *env,
        rc = fld_index_lookup(env, fld, new_range->lsr_start, range);
        if (rc == 0) {
                /* In case of duplicate entry, the location must be same */
-               LASSERT((range_compare_loc(new_range, range) == 0));
+               LASSERT((lu_seq_range_compare_loc(new_range, range) == 0));
                GOTO(out, rc = -EEXIST);
        }
 
@@ -122,7 +112,7 @@ int fld_declare_index_create(const struct lu_env *env,
        /* Check for merge case, since the fld entry can only be increamental,
         * so we will only check whether it can be merged from the left. */
        if (new_range->lsr_start == range->lsr_end && range->lsr_end != 0 &&
-           range_compare_loc(new_range, range) == 0) {
+           lu_seq_range_compare_loc(new_range, range) == 0) {
                range_cpu_to_be(tmp, range);
                rc = dt_declare_delete(env, fld->lsf_obj,
                                       (struct dt_key *)&tmp->lsr_start, th);
@@ -131,10 +121,10 @@ int fld_declare_index_create(const struct lu_env *env,
                               fld->lsf_name, PRANGE(range), rc);
                        GOTO(out, rc);
                }
-               memcpy(tmp, new_range, sizeof(*new_range));
+               *tmp = *new_range;
                tmp->lsr_start = range->lsr_start;
        } else {
-               memcpy(tmp, new_range, sizeof(*new_range));
+               *tmp = *new_range;
        }
 
        range_cpu_to_be(tmp, tmp);
@@ -160,6 +150,8 @@ out:
  * changed between declare and create.
  * Because the fld entry can only be increamental, so we will only check
  * whether it can be merged from the left.
+ *
+ * Caller must hold fld->lsf_lock
  **/
 int fld_index_create(const struct lu_env *env, struct lu_server_fld *fld,
                     const struct lu_seq_range *new_range, struct thandle *th)
@@ -174,7 +166,7 @@ int fld_index_create(const struct lu_env *env, struct lu_server_fld *fld,
 
        info = lu_context_key_get(&env->le_ctx, &fld_thread_key);
 
-       LASSERT_MUTEX_LOCKED(&fld->lsf_lock);
+       LASSERT(mutex_is_locked(&fld->lsf_lock));
 
        range = &info->fti_lrange;
        memset(range, 0, sizeof(*range));
@@ -186,23 +178,22 @@ int fld_index_create(const struct lu_env *env, struct lu_server_fld *fld,
        }
 
        if (new_range->lsr_start == range->lsr_end && range->lsr_end != 0 &&
-           range_compare_loc(new_range, range) == 0) {
+           lu_seq_range_compare_loc(new_range, range) == 0) {
                range_cpu_to_be(tmp, range);
                rc = dt_delete(env, fld->lsf_obj,
-                              (struct dt_key *)&tmp->lsr_start, th,
-                               BYPASS_CAPA);
+                              (struct dt_key *)&tmp->lsr_start, th);
                if (rc != 0)
                        GOTO(out, rc);
-               memcpy(tmp, new_range, sizeof(*new_range));
+               *tmp = *new_range;
                tmp->lsr_start = range->lsr_start;
                deleted = 1;
        } else {
-               memcpy(tmp, new_range, sizeof(*new_range));
+               *tmp = *new_range;
        }
 
        range_cpu_to_be(tmp, tmp);
        rc = dt_insert(env, fld->lsf_obj, (struct dt_rec *)tmp,
-                      (struct dt_key *)&tmp->lsr_start, th, BYPASS_CAPA, 1);
+                      (struct dt_key *)&tmp->lsr_start, th, 1);
        if (rc != 0) {
                CERROR("%s: insert range "DRANGE" failed: rc = %d\n",
                       fld->lsf_name, PRANGE(new_range), rc);
@@ -236,7 +227,7 @@ out:
  * \retval  -ve         other error;
  */
 int fld_index_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     *fld_rec;
         struct fld_thread_info  *info;
@@ -249,28 +240,47 @@ int fld_index_lookup(const struct lu_env *env, struct lu_server_fld *fld,
 
        rc = fld_cache_lookup(fld->lsf_cache, seq, fld_rec);
        if (rc == 0) {
-                *range = *fld_rec;
-                if (range_within(range, seq))
-                        rc = 0;
-                else
-                        rc = -ENOENT;
+               *range = *fld_rec;
+               if (lu_seq_range_within(range, seq))
+                       rc = 0;
+               else
+                       rc = -ENOENT;
         }
 
-        CDEBUG(D_INFO, "%s: lookup seq = "LPX64" range : "DRANGE" rc = %d\n",
+       CDEBUG(D_INFO, "%s: lookup seq = %#llx range : "DRANGE" rc = %d\n",
                fld->lsf_name, seq, PRANGE(range), rc);
 
         RETURN(rc);
 }
 
+/**
+ * insert entry in fld store.
+ *
+ * \param  env    relevant lu_env
+ * \param  fld    fld store
+ * \param  range  range to be inserted
+ *
+ * \retval  0  success
+ * \retval  -ve error
+ *
+ * Caller must hold fld->lsf_lock
+ **/
+
 int fld_insert_entry(const struct lu_env *env,
                     struct lu_server_fld *fld,
                     const struct lu_seq_range *range)
 {
        struct thandle *th;
+       struct dt_device *dt = lu2dt_dev(fld->lsf_obj->do_lu.lo_dev);
        int rc;
        ENTRY;
 
-       th = dt_trans_create(env, lu2dt_dev(fld->lsf_obj->do_lu.lo_dev));
+       LASSERT(mutex_is_locked(&fld->lsf_lock));
+
+       if (dt->dd_rdonly)
+               RETURN(0);
+
+       th = dt_trans_create(env, dt);
        if (IS_ERR(th))
                RETURN(PTR_ERR(th));
 
@@ -281,8 +291,7 @@ int fld_insert_entry(const struct lu_env *env,
                GOTO(out, rc);
        }
 
-       rc = dt_trans_start_local(env, lu2dt_dev(fld->lsf_obj->do_lu.lo_dev),
-                                 th);
+       rc = dt_trans_start_local(env, dt, th);
        if (rc)
                GOTO(out, rc);
 
@@ -290,7 +299,7 @@ int fld_insert_entry(const struct lu_env *env,
        if (rc == -EEXIST)
                rc = 0;
 out:
-       dt_trans_stop(env, lu2dt_dev(fld->lsf_obj->do_lu.lo_dev), th);
+       dt_trans_stop(env, dt, th);
        RETURN(rc);
 }
 EXPORT_SYMBOL(fld_insert_entry);
@@ -304,13 +313,17 @@ static int fld_insert_special_entries(const struct lu_env *env,
        if (rc != 0)
                RETURN(rc);
 
+       rc = fld_insert_entry(env, fld, &DOT_LUSTRE_FLD_RANGE);
+       if (rc != 0)
+               RETURN(rc);
+
        rc = fld_insert_entry(env, fld, &ROOT_FLD_RANGE);
 
        RETURN(rc);
 }
 
 int fld_index_init(const struct lu_env *env, struct lu_server_fld *fld,
-                  struct dt_device *dt)
+                  struct dt_device *dt, int type)
 {
        struct dt_object        *dt_obj = NULL;
        struct lu_fid           fid;
@@ -321,6 +334,8 @@ int fld_index_init(const struct lu_env *env, struct lu_server_fld *fld,
        struct dt_it            *it;
        const struct dt_it_ops  *iops;
        int                     rc;
+       __u32                   index;
+       int range_count = 0;
        ENTRY;
 
        info = lu_context_key_get(&env->le_ctx, &fld_thread_key);
@@ -337,15 +352,27 @@ int fld_index_init(const struct lu_env *env, struct lu_server_fld *fld,
        dof.dof_type = DFT_INDEX;
        dof.u.dof_idx.di_feat = &fld_index_features;
 
-       dt_obj = dt_find_or_create(env, dt, &fid, &dof, attr);
+       dt_obj = dt_locate(env, dt, &fid);
        if (IS_ERR(dt_obj)) {
                rc = PTR_ERR(dt_obj);
-               CERROR("%s: Can't find \"%s\" obj %d\n", fld->lsf_name,
-                       fld_index_name, rc);
                dt_obj = NULL;
                GOTO(out, rc);
        }
 
+       LASSERT(dt_obj != NULL);
+       if (!dt_object_exists(dt_obj)) {
+               dt_object_put(env, dt_obj);
+               dt_obj = dt_find_or_create(env, dt, &fid, &dof, attr);
+               fld->lsf_new = 1;
+               if (IS_ERR(dt_obj)) {
+                       rc = PTR_ERR(dt_obj);
+                       CERROR("%s: Can't find \"%s\" obj %d\n", fld->lsf_name,
+                               fld_index_name, rc);
+                       dt_obj = NULL;
+                       GOTO(out, rc);
+               }
+       }
+
        fld->lsf_obj = dt_obj;
        rc = dt_obj->do_ops->do_index_try(env, dt_obj, &fld_index_features);
        if (rc != 0) {
@@ -357,40 +384,62 @@ int fld_index_init(const struct lu_env *env, struct lu_server_fld *fld,
        range = &info->fti_rec;
        /* Load fld entry to cache */
        iops = &dt_obj->do_index_ops->dio_it;
-       it = iops->init(env, dt_obj, 0, NULL);
+       it = iops->init(env, dt_obj, 0);
        if (IS_ERR(it))
                GOTO(out, rc = PTR_ERR(it));
 
        rc = iops->load(env, it, 0);
+       if (rc > 0)
+               rc = 0;
+       else if (rc == 0)
+               rc = iops->next(env, it);
+
        if (rc < 0)
                GOTO(out_it_fini, rc);
 
-       if (rc > 0) {
-               /* Load FLD entry into server cache */
-               do {
-                       rc = iops->rec(env, it, (struct dt_rec *)range, 0);
-                       if (rc != 0)
-                               GOTO(out_it_put, rc);
-                       LASSERT(range != NULL);
-                       range_be_to_cpu(range, range);
+       while (rc == 0) {
+               rc = iops->rec(env, it, (struct dt_rec *)range, 0);
+               if (rc != 0)
+                       GOTO(out_it_put, rc);
+
+               range_be_to_cpu(range, range);
+
+               /* Newly created ldiskfs IAM indexes may include a
+                * zeroed-out key and record. Ignore it here. */
+               if (range->lsr_start < range->lsr_end) {
                        rc = fld_cache_insert(fld->lsf_cache, range);
                        if (rc != 0)
                                GOTO(out_it_put, rc);
-                       rc = iops->next(env, it);
-               } while (rc == 0);
+
+                       range_count++;
+               }
+
+               rc = iops->next(env, it);
+               if (rc < 0)
+                       GOTO(out_it_fini, rc);
        }
 
-       /* Note: fld_insert_entry will detect whether these
-        * special entries already exist inside FLDB */
-       mutex_lock(&fld->lsf_lock);
-       rc = fld_insert_special_entries(env, fld);
-       mutex_unlock(&fld->lsf_lock);
-       if (rc != 0) {
-               CERROR("%s: insert special entries failed!: rc = %d\n",
-                      fld->lsf_name, rc);
+       if (range_count == 0)
+               fld->lsf_new = 1;
+
+       rc = fld_name_to_index(fld->lsf_name, &index);
+       if (rc < 0)
                GOTO(out_it_put, rc);
-       }
+       else
+               rc = 0;
 
+       if (index == 0 && type == LU_SEQ_RANGE_MDT) {
+               /* Note: fld_insert_entry will detect whether these
+                * special entries already exist inside FLDB */
+               mutex_lock(&fld->lsf_lock);
+               rc = fld_insert_special_entries(env, fld);
+               mutex_unlock(&fld->lsf_lock);
+               if (rc != 0) {
+                       CERROR("%s: insert special entries failed!: rc = %d\n",
+                              fld->lsf_name, rc);
+                       GOTO(out_it_put, rc);
+               }
+       }
 out_it_put:
        iops->put(env, it);
 out_it_fini:
@@ -399,9 +448,9 @@ out:
        if (attr != NULL)
                OBD_FREE_PTR(attr);
 
-       if (rc != 0) {
+       if (rc < 0) {
                if (dt_obj != NULL)
-                       lu_object_put(env, &dt_obj->do_lu);
+                       dt_object_put(env, dt_obj);
                fld->lsf_obj = NULL;
        }
        RETURN(rc);
@@ -412,8 +461,65 @@ void fld_index_fini(const struct lu_env *env, struct lu_server_fld *fld)
        ENTRY;
        if (fld->lsf_obj != NULL) {
                if (!IS_ERR(fld->lsf_obj))
-                       lu_object_put(env, &fld->lsf_obj->do_lu);
+                       dt_object_put(env, fld->lsf_obj);
                fld->lsf_obj = NULL;
        }
        EXIT;
 }
+
+int fld_server_read(const struct lu_env *env, struct lu_server_fld *fld,
+                   struct lu_seq_range *range, void *data, int data_len)
+{
+       struct lu_seq_range_array *lsra = data;
+       struct fld_thread_info    *info;
+       struct dt_object          *dt_obj = fld->lsf_obj;
+       struct lu_seq_range       *entry;
+       struct dt_it              *it;
+       const struct dt_it_ops    *iops;
+       int                       rc;
+
+       ENTRY;
+
+       lsra->lsra_count = 0;
+       iops = &dt_obj->do_index_ops->dio_it;
+       it = iops->init(env, dt_obj, 0);
+       if (IS_ERR(it))
+               RETURN(PTR_ERR(it));
+
+       rc = iops->load(env, it, range->lsr_end);
+       if (rc <= 0)
+               GOTO(out_it_fini, rc);
+
+       info = lu_context_key_get(&env->le_ctx, &fld_thread_key);
+       LASSERT(info != NULL);
+       entry = &info->fti_rec;
+       do {
+               rc = iops->rec(env, it, (struct dt_rec *)entry, 0);
+               if (rc != 0)
+                       GOTO(out_it_put, rc);
+
+               if (offsetof(typeof(*lsra), lsra_lsr[lsra->lsra_count + 1]) >
+                   data_len)
+                       GOTO(out, rc = -EAGAIN);
+
+               range_be_to_cpu(entry, entry);
+               if (entry->lsr_index == range->lsr_index &&
+                   entry->lsr_flags == range->lsr_flags &&
+                   entry->lsr_start > range->lsr_start) {
+                       lsra->lsra_lsr[lsra->lsra_count] = *entry;
+                       lsra->lsra_count++;
+               }
+
+               rc = iops->next(env, it);
+       } while (rc == 0);
+       if (rc > 0)
+               rc = 0;
+out:
+       range_array_cpu_to_le(lsra, lsra);
+out_it_put:
+       iops->put(env, it);
+out_it_fini:
+       iops->fini(env, it);
+
+       RETURN(rc);
+}