Whamcloud - gitweb
LU-1866 lfsck: enhance otable-based iteration
[fs/lustre-release.git] / lustre / fid / fid_request.c
index 7bfcc57..6facae1 100644 (file)
@@ -87,34 +87,39 @@ static int seq_client_rpc(struct lu_client_seq *seq,
 
        ptlrpc_request_set_replen(req);
 
-       if (seq->lcs_type == LUSTRE_SEQ_METADATA) {
-               req->rq_request_portal = SEQ_METADATA_PORTAL;
+       in->lsr_index = seq->lcs_space.lsr_index;
+       if (seq->lcs_type == LUSTRE_SEQ_METADATA)
                in->lsr_flags = LU_SEQ_RANGE_MDT;
-       } else {
-               LASSERTF(seq->lcs_type == LUSTRE_SEQ_DATA,
-                        "unknown lcs_type %u\n", seq->lcs_type);
-               req->rq_request_portal = SEQ_DATA_PORTAL;
+       else
                in->lsr_flags = LU_SEQ_RANGE_OST;
-       }
 
        if (opc == SEQ_ALLOC_SUPER) {
-               /* Update index field of *in, it is required for
-                * FLD update on super sequence allocator node. */
-               in->lsr_index = seq->lcs_space.lsr_index;
                req->rq_request_portal = SEQ_CONTROLLER_PORTAL;
+               req->rq_reply_portal = MDC_REPLY_PORTAL;
+               /* During allocating super sequence for data object,
+                * the current thread might hold the export of MDT0(MDT0
+                * precreating objects on this OST), and it will send the
+                * request to MDT0 here, so we can not keep resending the
+                * request here, otherwise if MDT0 is failed(umounted),
+                * it can not release the export of MDT0 */
+               if (seq->lcs_type == LUSTRE_SEQ_DATA)
+                       req->rq_no_delay = req->rq_no_resend = 1;
                debug_mask = D_CONSOLE;
        } else {
+               if (seq->lcs_type == LUSTRE_SEQ_METADATA)
+                       req->rq_request_portal = SEQ_METADATA_PORTAL;
+               else
+                       req->rq_request_portal = SEQ_DATA_PORTAL;
                debug_mask = D_INFO;
-               LASSERTF(opc == SEQ_ALLOC_META,
-                        "unknown opcode %u\n, opc", opc);
        }
 
        ptlrpc_at_set_req_timeout(req);
 
-       mdc_get_rpc_lock(exp->exp_obd->u.cli.cl_rpc_lock, NULL);
+       if (seq->lcs_type == LUSTRE_SEQ_METADATA)
+               mdc_get_rpc_lock(exp->exp_obd->u.cli.cl_rpc_lock, NULL);
        rc = ptlrpc_queue_wait(req);
-       mdc_put_rpc_lock(exp->exp_obd->u.cli.cl_rpc_lock, NULL);
-
+       if (seq->lcs_type == LUSTRE_SEQ_METADATA)
+               mdc_put_rpc_lock(exp->exp_obd->u.cli.cl_rpc_lock, NULL);
        if (rc)
                GOTO(out_req, rc);
 
@@ -158,7 +163,14 @@ int seq_client_alloc_super(struct lu_client_seq *seq,
                                             env);
         } else {
 #endif
-                rc = seq_client_rpc(seq, &seq->lcs_space,
+               /* Check whether the connection to seq controller has been
+                * setup (lcs_exp != NULL) */
+               if (seq->lcs_exp == NULL) {
+                       mutex_unlock(&seq->lcs_mutex);
+                       RETURN(-EINPROGRESS);
+               }
+
+               rc = seq_client_rpc(seq, &seq->lcs_space,
                                     SEQ_ALLOC_SUPER, "super");
 #ifdef __KERNEL__
         }
@@ -180,8 +192,14 @@ static int seq_client_alloc_meta(const struct lu_env *env,
                 rc = seq_server_alloc_meta(seq->lcs_srv, &seq->lcs_space, env);
         } else {
 #endif
-                rc = seq_client_rpc(seq, &seq->lcs_space,
-                                    SEQ_ALLOC_META, "meta");
+               do {
+                       /* If meta server return -EINPROGRESS or EAGAIN,
+                        * it means meta server might not be ready to
+                        * allocate super sequence from sequence controller
+                        * (MDT0)yet */
+                       rc = seq_client_rpc(seq, &seq->lcs_space,
+                                           SEQ_ALLOC_META, "meta");
+               } while (rc == -EINPROGRESS || rc == -EAGAIN);
 #ifdef __KERNEL__
         }
 #endif
@@ -249,7 +267,9 @@ static void seq_fid_alloc_fini(struct lu_client_seq *seq)
         cfs_waitq_signal(&seq->lcs_waitq);
 }
 
-/* Allocate the whole seq to the caller*/
+/**
+ * Allocate the whole seq to the caller.
+ **/
 int seq_client_get_seq(const struct lu_env *env,
                        struct lu_client_seq *seq, seqno_t *seqnr)
 {
@@ -278,12 +298,15 @@ int seq_client_get_seq(const struct lu_env *env,
         CDEBUG(D_INFO, "%s: allocate sequence "
                "[0x%16.16"LPF64"x]\n", seq->lcs_name, *seqnr);
 
-        /*Since the caller require the whole seq,
-         *so marked this seq to be used*/
-        seq->lcs_fid.f_oid = LUSTRE_SEQ_MAX_WIDTH;
-        seq->lcs_fid.f_seq = *seqnr;
-        seq->lcs_fid.f_ver = 0;
+       /* Since the caller require the whole seq,
+        * so marked this seq to be used */
+       if (seq->lcs_type == LUSTRE_SEQ_METADATA)
+               seq->lcs_fid.f_oid = LUSTRE_METADATA_SEQ_MAX_WIDTH;
+       else
+               seq->lcs_fid.f_oid = LUSTRE_DATA_SEQ_MAX_WIDTH;
 
+       seq->lcs_fid.f_seq = *seqnr;
+       seq->lcs_fid.f_ver = 0;
         /*
          * Inform caller that sequence switch is performed to allow it
          * to setup FLD for it.
@@ -460,36 +483,37 @@ int seq_client_init(struct lu_client_seq *seq,
                     const char *prefix,
                     struct lu_server_seq *srv)
 {
-        int rc;
-        ENTRY;
+       int rc;
+       ENTRY;
 
-        LASSERT(seq != NULL);
-        LASSERT(prefix != NULL);
+       LASSERT(seq != NULL);
+       LASSERT(prefix != NULL);
+
+       seq->lcs_srv = srv;
+       seq->lcs_type = type;
 
-        seq->lcs_exp = exp;
-        seq->lcs_srv = srv;
-        seq->lcs_type = type;
        mutex_init(&seq->lcs_mutex);
-        seq->lcs_width = LUSTRE_SEQ_MAX_WIDTH;
-        cfs_waitq_init(&seq->lcs_waitq);
+       if (type == LUSTRE_SEQ_METADATA)
+               seq->lcs_width = LUSTRE_METADATA_SEQ_MAX_WIDTH;
+       else
+               seq->lcs_width = LUSTRE_DATA_SEQ_MAX_WIDTH;
 
-        /* Make sure that things are clear before work is started. */
-        seq_client_flush(seq);
+       cfs_waitq_init(&seq->lcs_waitq);
+       /* Make sure that things are clear before work is started. */
+       seq_client_flush(seq);
 
-        if (exp == NULL) {
-                LASSERT(seq->lcs_srv != NULL);
-        } else {
-                LASSERT(seq->lcs_exp != NULL);
-                seq->lcs_exp = class_export_get(seq->lcs_exp);
-        }
+       if (exp != NULL)
+               seq->lcs_exp = class_export_get(exp);
+       else if (type == LUSTRE_SEQ_METADATA)
+               LASSERT(seq->lcs_srv != NULL);
 
-        snprintf(seq->lcs_name, sizeof(seq->lcs_name),
-                 "cli-%s", prefix);
+       snprintf(seq->lcs_name, sizeof(seq->lcs_name),
+                "cli-%s", prefix);
 
-        rc = seq_client_proc_init(seq);
-        if (rc)
-                seq_client_fini(seq);
-        RETURN(rc);
+       rc = seq_client_proc_init(seq);
+       if (rc)
+               seq_client_fini(seq);
+       RETURN(rc);
 }
 EXPORT_SYMBOL(seq_client_init);