Whamcloud - gitweb
LU-1303 fld: verify support for range lookups
[fs/lustre-release.git] / lustre / fld / fld_index.c
index 997e781..e4c4169 100644 (file)
@@ -1,6 +1,4 @@
-/* -*- mode: c; c-basic-offset: 8; indent-tabs-mode: nil; -*-
- * vim:expandtab:shiftwidth=8:tabstop=8:
- *
+/*
  * GPL HEADER START
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
@@ -28,6 +26,8 @@
 /*
  * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
+ *
+ * Copyright (c) 2011, 2012, Whamcloud, Inc.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -38,9 +38,7 @@
  * Author: WangDi <wangdi@clusterfs.com>
  * Author: Yury Umanets <umka@clusterfs.com>
  */
-#ifndef EXPORT_SYMTAB
-# define EXPORT_SYMTAB
-#endif
+
 #define DEBUG_SUBSYSTEM S_FLD
 
 #ifdef __KERNEL__
@@ -74,7 +72,7 @@ static const struct lu_seq_range IGIF_FLD_RANGE = {
 };
 
 const struct dt_index_features fld_index_features = {
-        .dif_flags       = DT_IND_UPDATE,
+       .dif_flags       = DT_IND_UPDATE | DT_IND_RANGE,
         .dif_keysize_min = sizeof(seqno_t),
         .dif_keysize_max = sizeof(seqno_t),
         .dif_recsize_min = sizeof(struct lu_seq_range),
@@ -84,8 +82,7 @@ const struct dt_index_features fld_index_features = {
 
 extern struct lu_context_key fld_thread_key;
 
-static struct dt_key *fld_key(const struct lu_env *env,
-                              const seqno_t seq)
+static struct dt_key *fld_key(const struct lu_env *env, const seqno_t seq)
 {
         struct fld_thread_info *info;
         ENTRY;
@@ -112,19 +109,24 @@ static struct dt_rec *fld_rec(const struct lu_env *env,
         RETURN((void *)rec);
 }
 
-struct thandle* fld_trans_start(struct lu_server_fld *fld,
-                                const struct lu_env *env, int credit)
+struct thandle *fld_trans_create(struct lu_server_fld *fld,
+                                const struct lu_env *env)
 {
-        struct fld_thread_info *info;
         struct dt_device *dt_dev;
-        struct txn_param *p;
 
         dt_dev = lu2dt_dev(fld->lsf_obj->do_lu.lo_dev);
-        info = lu_context_key_get(&env->le_ctx, &fld_thread_key);
-        p = &info->fti_txn_param;
-        txn_param_init(p, credit);
 
-        return dt_dev->dd_ops->dt_trans_start(env, dt_dev, p);
+        return dt_dev->dd_ops->dt_trans_create(env, dt_dev);
+}
+
+int fld_trans_start(struct lu_server_fld *fld,
+                                const struct lu_env *env, struct thandle *th)
+{
+        struct dt_device *dt_dev;
+
+        dt_dev = lu2dt_dev(fld->lsf_obj->do_lu.lo_dev);
+
+        return dt_dev->dd_ops->dt_trans_start(env, dt_dev, th);
 }
 
 void fld_trans_stop(struct lu_server_fld *fld,
@@ -136,6 +138,31 @@ void fld_trans_stop(struct lu_server_fld *fld,
         dt_dev->dd_ops->dt_trans_stop(env, th);
 }
 
+int fld_declare_index_create(struct lu_server_fld *fld,
+                             const struct lu_env *env,
+                             const struct lu_seq_range *range,
+                             struct thandle *th)
+{
+        struct dt_object *dt_obj = fld->lsf_obj;
+        seqno_t start;
+        int rc;
+
+        ENTRY;
+
+       if (fld->lsf_no_range_lookup) {
+               /* Stub for underlying FS which can't lookup ranges */
+               return 0;
+       }
+
+        start = range->lsr_start;
+        LASSERT(range_is_sane(range));
+
+        rc = dt_obj->do_index_ops->dio_declare_insert(env, dt_obj,
+                                                      fld_rec(env, range),
+                                                      fld_key(env, start), th);
+        RETURN(rc);
+}
+
 /**
  * insert range in fld store.
  *
@@ -158,6 +185,17 @@ int fld_index_create(struct lu_server_fld *fld,
 
         ENTRY;
 
+       if (fld->lsf_no_range_lookup) {
+               /* Stub for underlying FS which can't lookup ranges */
+               if (range->lsr_index != 0) {
+                       CERROR("%s: FLD backend does not support range"
+                              "lookups, so DNE and FIDs-on-OST are not"
+                              "supported in this configuration\n",
+                              fld->lsf_name);
+                       return -EINVAL;
+               }
+       }
+
         start = range->lsr_start;
         LASSERT(range_is_sane(range));
 
@@ -192,9 +230,8 @@ int fld_index_delete(struct lu_server_fld *fld,
 
         ENTRY;
 
-        rc = dt_obj->do_index_ops->dio_delete(env, dt_obj,
-                                              fld_key(env, seq), th,
-                                              BYPASS_CAPA);
+        rc = dt_obj->do_index_ops->dio_delete(env, dt_obj, fld_key(env, seq),
+                                              th, BYPASS_CAPA);
 
         CDEBUG(D_INFO, "%s: delete given range : "DRANGE" rc = %d\n",
                fld->lsf_name, PRANGE(range), rc);
@@ -227,6 +264,17 @@ int fld_index_lookup(struct lu_server_fld *fld,
 
         ENTRY;
 
+       if (fld->lsf_no_range_lookup) {
+               /* Stub for underlying FS which can't lookup ranges */
+               range->lsr_start = 0;
+               range->lsr_end = ~0;
+               range->lsr_index = 0;
+               range->lsr_flags = LU_SEQ_RANGE_MDT;
+
+               range_cpu_to_be(range, range);
+               return 0;
+       }
+
         info = lu_context_key_get(&env->le_ctx, &fld_thread_key);
         fld_rec = &info->fti_rec;
 
@@ -254,11 +302,22 @@ static int fld_insert_igif_fld(struct lu_server_fld *fld,
 {
         struct thandle *th;
         int rc;
-
         ENTRY;
-        th = fld_trans_start(fld, env, FLD_TXN_INDEX_INSERT_CREDITS);
+
+        /* FLD_TXN_INDEX_INSERT_CREDITS */
+        th = fld_trans_create(fld, env);
         if (IS_ERR(th))
                 RETURN(PTR_ERR(th));
+        rc = fld_declare_index_create(fld, env, &IGIF_FLD_RANGE, th);
+        if (rc) {
+                fld_trans_stop(fld, env, th);
+                RETURN(rc);
+        }
+        rc = fld_trans_start(fld, env, th);
+        if (rc) {
+                fld_trans_stop(fld, env, th);
+                RETURN(rc);
+        }
 
         rc = fld_index_create(fld, env, &IGIF_FLD_RANGE, th);
         fld_trans_stop(fld, env, th);
@@ -290,9 +349,21 @@ int fld_index_init(struct lu_server_fld *fld,
                                 lu_object_put(env, &dt_obj->do_lu);
                                 fld->lsf_obj = NULL;
                         }
-                } else
-                        CERROR("%s: File \"%s\" is not an index!\n",
-                               fld->lsf_name, fld_index_name);
+               } else if (rc == -ERANGE) {
+                       CWARN("%s: File \"%s\" doesn't support range lookup, "
+                             "using stub. DNE and FIDs on OST will not work "
+                             "with this backend\n",
+                             fld->lsf_name, fld_index_name);
+
+                       LASSERT(dt_obj->do_index_ops == NULL);
+                       fld->lsf_no_range_lookup = 1;
+                       rc = 0;
+               } else {
+                       CERROR("%s: File \"%s\" is not index, rc %d!\n",
+                              fld->lsf_name, fld_index_name, rc);
+                       lu_object_put(env, &fld->lsf_obj->do_lu);
+                       fld->lsf_obj = NULL;
+               }
 
 
         } else {