Whamcloud - gitweb
LU-14090 mgs: no local logs flag
[fs/lustre-release.git] / lustre / mgc / mgc_request.c
index 6dd4cc1..ecce505 100644 (file)
@@ -39,6 +39,7 @@
 
 #include <linux/module.h>
 #include <linux/kthread.h>
+#include <linux/random.h>
 
 #include <dt_object.h>
 #include <lprocfs_status.h>
@@ -117,7 +118,7 @@ int mgc_logname2resid(char *logname, struct ldlm_res_id *res_id, int type)
 EXPORT_SYMBOL(mgc_logname2resid);
 
 /********************** config llog list **********************/
-static struct list_head config_llog_list = LIST_HEAD_INIT(config_llog_list);
+static LIST_HEAD(config_llog_list);
 static DEFINE_SPINLOCK(config_list_lock);      /* protects config_llog_list */
 
 /* Take a reference to a config log */
@@ -125,7 +126,7 @@ static int config_log_get(struct config_llog_data *cld)
 {
        ENTRY;
        atomic_inc(&cld->cld_refcount);
-       CDEBUG(D_INFO, "log %s refs %d\n", cld->cld_logname,
+       CDEBUG(D_INFO, "log %s (%p) refs %d\n", cld->cld_logname, cld,
                atomic_read(&cld->cld_refcount));
        RETURN(0);
 }
@@ -139,7 +140,7 @@ static void config_log_put(struct config_llog_data *cld)
        if (unlikely(!cld))
                RETURN_EXIT;
 
-       CDEBUG(D_INFO, "log %s refs %d\n", cld->cld_logname,
+       CDEBUG(D_INFO, "log %s(%p) refs %d\n", cld->cld_logname, cld,
                atomic_read(&cld->cld_refcount));
        LASSERT(atomic_read(&cld->cld_refcount) > 0);
 
@@ -155,8 +156,10 @@ static void config_log_put(struct config_llog_data *cld)
                config_log_put(cld->cld_params);
                config_log_put(cld->cld_nodemap);
                config_log_put(cld->cld_sptlrpc);
-               if (cld_is_sptlrpc(cld))
+               if (cld_is_sptlrpc(cld)) {
+                       cld->cld_stopping = 1;
                        sptlrpc_conf_log_stop(cld->cld_logname);
+               }
 
                class_export_put(cld->cld_mgcexp);
                OBD_FREE(cld, sizeof(*cld) + strlen(cld->cld_logname) + 1);
@@ -170,18 +173,18 @@ static
 struct config_llog_data *config_log_find(char *logname,
                                          struct config_llog_instance *cfg)
 {
-        struct config_llog_data *cld;
-        struct config_llog_data *found = NULL;
-        void *                   instance;
-        ENTRY;
+       struct config_llog_data *cld;
+       struct config_llog_data *found = NULL;
+       unsigned long cfg_instance;
 
-        LASSERT(logname != NULL);
+       ENTRY;
+       LASSERT(logname != NULL);
 
-        instance = cfg ? cfg->cfg_instance : NULL;
+       cfg_instance = cfg ? cfg->cfg_instance : 0;
        spin_lock(&config_list_lock);
        list_for_each_entry(cld, &config_llog_list, cld_list_chain) {
-               /* check if instance equals */
-               if (instance != cld->cld_cfg.cfg_instance)
+               /* check if cfg_instance is the one we want */
+               if (cfg_instance != cld->cld_cfg.cfg_instance)
                        continue;
 
                /* instance may be NULL, should check name */
@@ -207,8 +210,8 @@ struct config_llog_data *do_config_log_add(struct obd_device *obd,
 
        ENTRY;
 
-       CDEBUG(D_MGC, "do adding config log %s:%p\n", logname,
-              cfg ? cfg->cfg_instance : NULL);
+       CDEBUG(D_MGC, "do adding config log %s-%016lx\n", logname,
+              cfg ? cfg->cfg_instance : 0);
 
        OBD_ALLOC(cld, sizeof(*cld) + strlen(logname) + 1);
        if (!cld)
@@ -235,10 +238,8 @@ struct config_llog_data *do_config_log_add(struct obd_device *obd,
        /* Keep the mgc around until we are done */
        cld->cld_mgcexp = class_export_get(obd->obd_self_export);
 
-       if (cld_is_sptlrpc(cld)) {
+       if (cld_is_sptlrpc(cld))
                sptlrpc_conf_log_start(logname);
-               cld->cld_cfg.cfg_obdname = obd->obd_name;
-       }
 
        spin_lock(&config_list_lock);
        list_add(&cld->cld_list_chain, &config_llog_list);
@@ -255,50 +256,49 @@ struct config_llog_data *do_config_log_add(struct obd_device *obd,
 }
 
 static struct config_llog_data *config_recover_log_add(struct obd_device *obd,
-        char *fsname,
-        struct config_llog_instance *cfg,
-        struct super_block *sb)
+                                       char *fsname,
+                                       struct config_llog_instance *cfg,
+                                       struct super_block *sb)
 {
-        struct config_llog_instance lcfg = *cfg;
-        struct lustre_sb_info *lsi = s2lsi(sb);
-        struct config_llog_data *cld;
-        char logname[32];
+       struct config_llog_instance lcfg = *cfg;
+       struct lustre_sb_info *lsi = s2lsi(sb);
+       struct config_llog_data *cld;
+       char logname[32];
 
        if (IS_OST(lsi))
-                return NULL;
+               return NULL;
 
        /* for osp-on-ost, see lustre_start_osp() */
        if (IS_MDT(lsi) && lcfg.cfg_instance)
                return NULL;
 
-        /* we have to use different llog for clients and mdts for cmd
-         * where only clients are notified if one of cmd server restarts */
-        LASSERT(strlen(fsname) < sizeof(logname) / 2);
-        strcpy(logname, fsname);
+       /* We have to use different llog for clients and MDTs for DNE,
+        * where only clients are notified if one of DNE server restarts.
+        */
+       LASSERT(strlen(fsname) < sizeof(logname) / 2);
+       strncpy(logname, fsname, sizeof(logname));
        if (IS_SERVER(lsi)) { /* mdt */
-                LASSERT(lcfg.cfg_instance == NULL);
-                lcfg.cfg_instance = sb;
-                strcat(logname, "-mdtir");
-        } else {
-                LASSERT(lcfg.cfg_instance != NULL);
-                strcat(logname, "-cliir");
-        }
+               LASSERT(lcfg.cfg_instance == 0);
+               lcfg.cfg_instance = ll_get_cfg_instance(sb);
+               strncat(logname, "-mdtir", sizeof(logname));
+       } else {
+               LASSERT(lcfg.cfg_instance != 0);
+               strncat(logname, "-cliir", sizeof(logname));
+       }
 
-        cld = do_config_log_add(obd, logname, CONFIG_T_RECOVER, &lcfg, sb);
-        return cld;
+       cld = do_config_log_add(obd, logname, CONFIG_T_RECOVER, &lcfg, sb);
+       return cld;
 }
 
 static struct config_llog_data *config_log_find_or_add(struct obd_device *obd,
                                char *logname, struct super_block *sb, int type,
                                struct config_llog_instance *cfg)
 {
-       struct config_llog_instance     lcfg = *cfg;
-       struct config_llog_data         *cld;
-
-       lcfg.cfg_instance = sb != NULL ? (void *)sb : (void *)obd;
+       struct config_llog_instance lcfg = *cfg;
+       struct config_llog_data *cld;
 
-       if (type == CONFIG_T_SPTLRPC)
-               lcfg.cfg_instance = NULL;
+       /* Note class_config_llog_handler() depends on getting "obd" back */
+       lcfg.cfg_instance = sb ? ll_get_cfg_instance(sb) : (unsigned long)obd;
 
        cld = config_log_find(logname, &lcfg);
        if (unlikely(cld != NULL))
@@ -328,7 +328,8 @@ config_log_add(struct obd_device *obd, char *logname,
        bool locked = false;
        ENTRY;
 
-       CDEBUG(D_MGC, "adding config log %s:%p\n", logname, cfg->cfg_instance);
+       CDEBUG(D_MGC, "add config log %s-%016lx\n", logname,
+              cfg->cfg_instance);
 
        /*
         * for each regular log, the depended sptlrpc log name is
@@ -450,13 +451,24 @@ out_sptlrpc:
 
 DEFINE_MUTEX(llog_process_lock);
 
+static inline void config_mark_cld_stop_nolock(struct config_llog_data *cld)
+{
+       ENTRY;
+
+       spin_lock(&config_list_lock);
+       cld->cld_stopping = 1;
+       spin_unlock(&config_list_lock);
+
+       CDEBUG(D_INFO, "lockh %#llx\n", cld->cld_lockh.cookie);
+       if (!ldlm_lock_addref_try(&cld->cld_lockh, LCK_CR))
+               ldlm_lock_decref_and_cancel(&cld->cld_lockh, LCK_CR);
+}
+
 static inline void config_mark_cld_stop(struct config_llog_data *cld)
 {
        if (cld) {
                mutex_lock(&cld->cld_lock);
-               spin_lock(&config_list_lock);
-               cld->cld_stopping = 1;
-               spin_unlock(&config_list_lock);
+               config_mark_cld_stop_nolock(cld);
                mutex_unlock(&cld->cld_lock);
        }
 }
@@ -494,10 +506,6 @@ static int config_log_end(char *logname, struct config_llog_instance *cfg)
                RETURN(rc);
        }
 
-       spin_lock(&config_list_lock);
-       cld->cld_stopping = 1;
-       spin_unlock(&config_list_lock);
-
        cld_recover = cld->cld_recover;
        cld->cld_recover = NULL;
        cld_params = cld->cld_params;
@@ -508,26 +516,23 @@ static int config_log_end(char *logname, struct config_llog_instance *cfg)
        cld->cld_barrier = NULL;
        cld_sptlrpc = cld->cld_sptlrpc;
        cld->cld_sptlrpc = NULL;
+
+       config_mark_cld_stop_nolock(cld);
        mutex_unlock(&cld->cld_lock);
 
        config_mark_cld_stop(cld_recover);
        config_log_put(cld_recover);
-
        config_mark_cld_stop(cld_params);
        config_log_put(cld_params);
-
+       config_mark_cld_stop(cld_barrier);
+       config_log_put(cld_barrier);
+       /* don't explicitly set cld_stopping on sptlrpc lock here, as other
+        * targets may be active, it will be done in config_log_put if necessary
+        */
+       config_log_put(cld_sptlrpc);
        /* don't set cld_stopping on nm lock as other targets may be active */
        config_log_put(cld_nodemap);
 
-       if (cld_barrier) {
-               mutex_lock(&cld_barrier->cld_lock);
-               cld_barrier->cld_stopping = 1;
-               mutex_unlock(&cld_barrier->cld_lock);
-               config_log_put(cld_barrier);
-       }
-
-       config_log_put(cld_sptlrpc);
-
        /* drop the ref from the find */
        config_log_put(cld);
        /* drop the start ref */
@@ -544,15 +549,20 @@ int lprocfs_mgc_rd_ir_state(struct seq_file *m, void *data)
        struct obd_import       *imp;
        struct obd_connect_data *ocd;
        struct config_llog_data *cld;
+       int rc = 0;
 
        ENTRY;
        LASSERT(obd);
-       LPROCFS_CLIMP_CHECK(obd);
-       imp = obd->u.cli.cl_import;
-       ocd = &imp->imp_connect_data;
+       with_imp_locked(obd, imp, rc) {
+               ocd = &imp->imp_connect_data;
+
+               seq_printf(m, "imperative_recovery: %s\n",
+                          OCD_HAS_FLAG(ocd, IMP_RECOV) ?
+                          "ENABLED" : "DISABLED");
+       }
+       if (rc)
+               RETURN(rc);
 
-       seq_printf(m, "imperative_recovery: %s\n",
-                  OCD_HAS_FLAG(ocd, IMP_RECOV) ? "ENABLED" : "DISABLED");
        seq_printf(m, "client_state:\n");
 
        spin_lock(&config_list_lock);
@@ -565,7 +575,6 @@ int lprocfs_mgc_rd_ir_state(struct seq_file *m, void *data)
        }
        spin_unlock(&config_list_lock);
 
-       LPROCFS_CLIMP_EXIT(obd);
        RETURN(0);
 }
 
@@ -613,7 +622,7 @@ static void do_requeue(struct config_llog_data *cld)
  * in order to not flood the MGS.
  */
 #define MGC_TIMEOUT_MIN_SECONDS   5
-#define MGC_TIMEOUT_RAND_CENTISEC 0x1ff /* ~500 */
+#define MGC_TIMEOUT_RAND_CENTISEC 500
 
 static int mgc_requeue_thread(void *data)
 {
@@ -627,9 +636,8 @@ static int mgc_requeue_thread(void *data)
        spin_lock(&config_list_lock);
        rq_state |= RQ_RUNNING;
        while (!(rq_state & RQ_STOP)) {
-               struct l_wait_info lwi;
                struct config_llog_data *cld, *cld_prev;
-               int rand = cfs_rand() & MGC_TIMEOUT_RAND_CENTISEC;
+               int rand = prandom_u32_max(MGC_TIMEOUT_RAND_CENTISEC);
                int to;
 
                /* Any new or requeued lostlocks will change the state */
@@ -642,24 +650,24 @@ static int mgc_requeue_thread(void *data)
                }
 
                /* Always wait a few seconds to allow the server who
-                  caused the lock revocation to finish its setup, plus some
-                  random so everyone doesn't try to reconnect at once. */
-               to = msecs_to_jiffies(MGC_TIMEOUT_MIN_SECONDS * MSEC_PER_SEC);
+                * caused the lock revocation to finish its setup, plus some
+                * random so everyone doesn't try to reconnect at once.
+                */
+               to = cfs_time_seconds(MGC_TIMEOUT_MIN_SECONDS * 100 + rand);
                /* rand is centi-seconds */
-               to += msecs_to_jiffies(rand * MSEC_PER_SEC / 100);
-               lwi = LWI_TIMEOUT(to, NULL, NULL);
-               l_wait_event(rq_waitq, rq_state & (RQ_STOP | RQ_PRECLEANUP),
-                            &lwi);
-
-                /*
-                 * iterate & processing through the list. for each cld, process
-                 * its depending sptlrpc cld firstly (if any) and then itself.
-                 *
-                 * it's guaranteed any item in the list must have
-                 * reference > 0; and if cld_lostlock is set, at
-                 * least one reference is taken by the previous enqueue.
-                 */
-                cld_prev = NULL;
+               wait_event_idle_timeout(rq_waitq,
+                                       rq_state & (RQ_STOP | RQ_PRECLEANUP),
+                                       to/100);
+
+               /*
+                * iterate & processing through the list. for each cld, process
+                * its depending sptlrpc cld firstly (if any) and then itself.
+                *
+                * it's guaranteed any item in the list must have
+                * reference > 0; and if cld_lostlock is set, at
+                * least one reference is taken by the previous enqueue.
+                */
+               cld_prev = NULL;
 
                spin_lock(&config_list_lock);
                rq_state &= ~RQ_PRECLEANUP;
@@ -689,9 +697,7 @@ static int mgc_requeue_thread(void *data)
                config_log_put(cld_prev);
 
                /* Wait a bit to see if anyone else needs a requeue */
-               lwi = (struct l_wait_info) { 0 };
-               l_wait_event(rq_waitq, rq_state & (RQ_NOW | RQ_STOP),
-                            &lwi);
+               wait_event_idle(rq_waitq, rq_state & (RQ_NOW | RQ_STOP));
                spin_lock(&config_list_lock);
        }
 
@@ -717,9 +723,14 @@ static void mgc_requeue_add(struct config_llog_data *cld)
                cld->cld_stopping, rq_state);
        LASSERT(atomic_read(&cld->cld_refcount) > 0);
 
+       /* lets cancel an existent lock to mark cld as "lostlock" */
+       CDEBUG(D_INFO, "lockh %#llx\n", cld->cld_lockh.cookie);
+       if (!ldlm_lock_addref_try(&cld->cld_lockh, LCK_CR))
+               ldlm_lock_decref_and_cancel(&cld->cld_lockh, LCK_CR);
+
        mutex_lock(&cld->cld_lock);
        spin_lock(&config_list_lock);
-       if (!(rq_state & RQ_STOP) && !cld->cld_stopping && !cld->cld_lostlock) {
+       if (!(rq_state & RQ_STOP) && !cld->cld_stopping) {
                cld->cld_lostlock = 1;
                rq_state |= RQ_NOW;
                wakeup = true;
@@ -934,7 +945,7 @@ static int mgc_cleanup(struct obd_device *obd)
 
         /* COMPAT_146 - old config logs may have added profiles we don't
            know about */
-        if (obd->obd_type->typ_refcnt <= 1)
+       if (atomic_read(&obd->obd_type->typ_refcnt) <= 1)
                 /* Only for the last mgc */
                 class_del_profiles();
 
@@ -1031,6 +1042,7 @@ static int mgc_blocking_ast(struct ldlm_lock *lock, struct ldlm_lock_desc *desc,
                LASSERT(atomic_read(&cld->cld_refcount) > 0);
 
                lock->l_ast_data = NULL;
+               cld->cld_lockh.cookie = 0;
                /* Are we done with this log? */
                if (cld->cld_stopping) {
                        CDEBUG(D_MGC, "log %s: stopping, won't requeue\n",
@@ -1068,45 +1080,6 @@ static int mgc_blocking_ast(struct ldlm_lock *lock, struct ldlm_lock_desc *desc,
 #define  MGC_TARGET_REG_LIMIT_MAX RECONNECT_DELAY_MAX
 #define  MGC_SEND_PARAM_LIMIT 10
 
-#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 13, 53, 0)
-/* Send parameter to MGS*/
-static int mgc_set_mgs_param(struct obd_export *exp,
-                             struct mgs_send_param *msp)
-{
-        struct ptlrpc_request *req;
-        struct mgs_send_param *req_msp, *rep_msp;
-        int rc;
-        ENTRY;
-
-        req = ptlrpc_request_alloc_pack(class_exp2cliimp(exp),
-                                        &RQF_MGS_SET_INFO, LUSTRE_MGS_VERSION,
-                                        MGS_SET_INFO);
-        if (!req)
-                RETURN(-ENOMEM);
-
-        req_msp = req_capsule_client_get(&req->rq_pill, &RMF_MGS_SEND_PARAM);
-        if (!req_msp) {
-                ptlrpc_req_finished(req);
-                RETURN(-ENOMEM);
-        }
-
-        memcpy(req_msp, msp, sizeof(*req_msp));
-        ptlrpc_request_set_replen(req);
-
-        /* Limit how long we will wait for the enqueue to complete */
-        req->rq_delay_limit = MGC_SEND_PARAM_LIMIT;
-        rc = ptlrpc_queue_wait(req);
-        if (!rc) {
-                rep_msp = req_capsule_server_get(&req->rq_pill, &RMF_MGS_SEND_PARAM);
-                memcpy(msp, rep_msp, sizeof(*rep_msp));
-        }
-
-        ptlrpc_req_finished(req);
-
-        RETURN(rc);
-}
-#endif
-
 /* Take a config lock so we can get cancel notifications */
 static int mgc_enqueue(struct obd_export *exp, enum ldlm_type type,
                       union ldlm_policy_data *policy, enum ldlm_mode mode,
@@ -1127,6 +1100,9 @@ static int mgc_enqueue(struct obd_export *exp, enum ldlm_type type,
        int rc;
        ENTRY;
 
+       if (!exp)
+               RETURN(-EBADR);
+
        CDEBUG(D_MGC, "Enqueue for %s (res %#llx)\n", cld->cld_logname,
                cld->cld_resid.name[0]);
 
@@ -1281,15 +1257,6 @@ static int mgc_set_info_async(const struct lu_env *env, struct obd_export *exp,
                rc = mgc_fs_cleanup(env, exp->exp_obd);
                RETURN(rc);
        }
-#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 13, 53, 0)
-        if (KEY_IS(KEY_SET_INFO)) {
-                struct mgs_send_param *msp;
-
-                msp = (struct mgs_send_param *)val;
-                rc =  mgc_set_mgs_param(exp, msp);
-                RETURN(rc);
-        }
-#endif
         if (KEY_IS(KEY_MGSSEC)) {
                 struct client_obd     *cli = &exp->exp_obd->u.cli;
                 struct sptlrpc_flavor  flvr;
@@ -1407,34 +1374,39 @@ static int mgc_apply_recover_logs(struct obd_device *mgc,
                                  __u64 max_version,
                                  void *data, int datalen, bool mne_swab)
 {
-        struct config_llog_instance *cfg = &cld->cld_cfg;
-        struct lustre_sb_info       *lsi = s2lsi(cfg->cfg_sb);
-        struct mgs_nidtbl_entry *entry;
-        struct lustre_cfg       *lcfg;
-        struct lustre_cfg_bufs   bufs;
-        u64   prev_version = 0;
-        char *inst;
-        char *buf;
-        int   bufsz;
-        int   pos;
-        int   rc  = 0;
-        int   off = 0;
-        ENTRY;
+       struct config_llog_instance *cfg = &cld->cld_cfg;
+       struct lustre_sb_info *lsi = s2lsi(cfg->cfg_sb);
+       struct mgs_nidtbl_entry *entry;
+       struct lustre_cfg *lcfg;
+       struct lustre_cfg_bufs bufs;
+       u64 prev_version = 0;
+       char *inst;
+       char *buf;
+       int bufsz;
+       int pos = 0;
+       int rc  = 0;
+       int off = 0;
+       unsigned long dynamic_nids;
+
+       ENTRY;
+       LASSERT(cfg->cfg_instance != 0);
+       LASSERT(ll_get_cfg_instance(cfg->cfg_sb) == cfg->cfg_instance);
 
-        LASSERT(cfg->cfg_instance != NULL);
-        LASSERT(cfg->cfg_sb == cfg->cfg_instance);
+       /* get dynamic nids setting */
+       dynamic_nids = mgc->obd_dynamic_nids;
 
        OBD_ALLOC(inst, PAGE_SIZE);
        if (inst == NULL)
                RETURN(-ENOMEM);
 
        if (!IS_SERVER(lsi)) {
-               pos = snprintf(inst, PAGE_SIZE, "%p", cfg->cfg_instance);
+               pos = snprintf(inst, PAGE_SIZE, "%016lx", cfg->cfg_instance);
                if (pos >= PAGE_SIZE) {
                        OBD_FREE(inst, PAGE_SIZE);
                        return -E2BIG;
                }
-        } else {
+#ifdef HAVE_SERVER_SUPPORT
+       } else {
                LASSERT(IS_MDT(lsi));
                rc = server_name2svname(lsi->lsi_svname, inst, NULL,
                                        PAGE_SIZE);
@@ -1443,38 +1415,40 @@ static int mgc_apply_recover_logs(struct obd_device *mgc,
                        RETURN(-EINVAL);
                }
                pos = strlen(inst);
-        }
+#endif /* HAVE_SERVER_SUPPORT */
+       }
 
-        ++pos;
-        buf   = inst + pos;
+       ++pos;
+       buf   = inst + pos;
        bufsz = PAGE_SIZE - pos;
 
-        while (datalen > 0) {
-                int   entry_len = sizeof(*entry);
-               int   is_ost, i;
-                struct obd_device *obd;
-                char *obdname;
-                char *cname;
-                char *params;
-                char *uuid;
-
-                rc = -EINVAL;
-                if (datalen < sizeof(*entry))
-                        break;
-
-                entry = (typeof(entry))(data + off);
-
-                /* sanity check */
-                if (entry->mne_nid_type != 0) /* only support type 0 for ipv4 */
-                        break;
-                if (entry->mne_nid_count == 0) /* at least one nid entry */
-                        break;
-                if (entry->mne_nid_size != sizeof(lnet_nid_t))
-                        break;
-
-                entry_len += entry->mne_nid_count * entry->mne_nid_size;
-                if (datalen < entry_len) /* must have entry_len at least */
-                        break;
+       while (datalen > 0) {
+               int   entry_len = sizeof(*entry);
+               int   is_ost;
+               struct obd_device *obd;
+               struct obd_import *imp;
+               char *obdname;
+               char *cname;
+               char *params;
+               char *uuid;
+
+               rc = -EINVAL;
+               if (datalen < sizeof(*entry))
+                       break;
+
+               entry = (typeof(entry))(data + off);
+
+               /* sanity check */
+               if (entry->mne_nid_type != 0) /* only support type 0 for ipv4 */
+                       break;
+               if (entry->mne_nid_count == 0) /* at least one nid entry */
+                       break;
+               if (entry->mne_nid_size != sizeof(lnet_nid_t))
+                       break;
+
+               entry_len += entry->mne_nid_count * entry->mne_nid_size;
+               if (datalen < entry_len) /* must have entry_len at least */
+                       break;
 
                /* Keep this swab for normal mixed endian handling. LU-1644 */
                if (mne_swab)
@@ -1487,102 +1461,130 @@ static int mgc_apply_recover_logs(struct obd_device *mgc,
                if (entry->mne_length < entry_len)
                        break;
 
-                off     += entry->mne_length;
-                datalen -= entry->mne_length;
-                if (datalen < 0)
-                        break;
+               off     += entry->mne_length;
+               datalen -= entry->mne_length;
+               if (datalen < 0)
+                       break;
 
-                if (entry->mne_version > max_version) {
-                        CERROR("entry index(%lld) is over max_index(%lld)\n",
-                               entry->mne_version, max_version);
-                        break;
-                }
+               if (entry->mne_version > max_version) {
+                       CERROR("entry index(%lld) is over max_index(%lld)\n",
+                              entry->mne_version, max_version);
+                       break;
+               }
 
-                if (prev_version >= entry->mne_version) {
-                        CERROR("index unsorted, prev %lld, now %lld\n",
-                               prev_version, entry->mne_version);
-                        break;
-                }
-                prev_version = entry->mne_version;
+               if (prev_version >= entry->mne_version) {
+                       CERROR("index unsorted, prev %lld, now %lld\n",
+                              prev_version, entry->mne_version);
+                       break;
+               }
+               prev_version = entry->mne_version;
 
-                /*
-                 * Write a string with format "nid::instance" to
-                 * lustre/<osc|mdc>/<target>-<osc|mdc>-<instance>/import.
-                 */
+               /*
+                * Write a string with format "nid::instance" to
+                * lustre/<osc|mdc>/<target>-<osc|mdc>-<instance>/import.
+                */
 
-                is_ost = entry->mne_type == LDD_F_SV_TYPE_OST;
-                memset(buf, 0, bufsz);
-                obdname = buf;
-                pos = 0;
-
-                /* lustre-OST0001-osc-<instance #> */
-                strcpy(obdname, cld->cld_logname);
-                cname = strrchr(obdname, '-');
-                if (cname == NULL) {
-                        CERROR("mgc %s: invalid logname %s\n",
-                               mgc->obd_name, obdname);
-                        break;
-                }
+               is_ost = entry->mne_type == LDD_F_SV_TYPE_OST;
+               memset(buf, 0, bufsz);
+               obdname = buf;
+               pos = 0;
+
+               /* lustre-OST0001-osc-<instance #> */
+               strcpy(obdname, cld->cld_logname);
+               cname = strrchr(obdname, '-');
+               if (cname == NULL) {
+                       CERROR("mgc %s: invalid logname %s\n",
+                              mgc->obd_name, obdname);
+                       break;
+               }
 
-                pos = cname - obdname;
-                obdname[pos] = 0;
-                pos += sprintf(obdname + pos, "-%s%04x",
-                                  is_ost ? "OST" : "MDT", entry->mne_index);
+               pos = cname - obdname;
+               obdname[pos] = 0;
+               pos += sprintf(obdname + pos, "-%s%04x",
+                              is_ost ? "OST" : "MDT", entry->mne_index);
 
-                cname = is_ost ? "osc" : "mdc",
-                pos += sprintf(obdname + pos, "-%s-%s", cname, inst);
-                lustre_cfg_bufs_reset(&bufs, obdname);
+               cname = is_ost ? "osc" : "mdc",
+                       pos += sprintf(obdname + pos, "-%s-%s", cname, inst);
+               lustre_cfg_bufs_reset(&bufs, obdname);
 
-                /* find the obd by obdname */
-                obd = class_name2obd(obdname);
-                if (obd == NULL) {
-                        CDEBUG(D_INFO, "mgc %s: cannot find obdname %s\n",
-                               mgc->obd_name, obdname);
+               /* find the obd by obdname */
+               obd = class_name2obd(obdname);
+               if (obd == NULL) {
+                       CDEBUG(D_INFO, "mgc %s: cannot find obdname %s\n",
+                              mgc->obd_name, obdname);
                        rc = 0;
-                        /* this is a safe race, when the ost is starting up...*/
-                        continue;
-                }
-
-                /* osc.import = "connection=<Conn UUID>::<target instance>" */
-                ++pos;
-                params = buf + pos;
-                pos += sprintf(params, "%s.import=%s", cname, "connection=");
-                uuid = buf + pos;
+                       /* this is a safe race, when the ost is starting up...*/
+                       continue;
+               }
 
-               down_read(&obd->u.cli.cl_sem);
-               if (obd->u.cli.cl_import == NULL) {
+               /* osc.import = "connection=<Conn UUID>::<target instance>" */
+               ++pos;
+               params = buf + pos;
+               pos += sprintf(params, "%s.import=%s", cname, "connection=");
+               uuid = buf + pos;
+
+               with_imp_locked(obd, imp, rc) {
+                       /* iterate all nids to find one */
+                       /* find uuid by nid */
+                       /* create import entries if they don't exist */
+                       rc = client_import_add_nids_to_conn(
+                               imp, entry->u.nids, entry->mne_nid_count,
+                               (struct obd_uuid *)uuid);
+
+                       if (rc == -ENOENT && dynamic_nids) {
+                               /* create a new connection for this import */
+                               char *primary_nid =
+                                       libcfs_nid2str(entry->u.nids[0]);
+                               int prim_nid_len = strlen(primary_nid) + 1;
+                               struct obd_uuid server_uuid;
+
+                               if (prim_nid_len > UUID_MAX)
+                                       goto fail;
+                               strncpy(server_uuid.uuid, primary_nid,
+                                       prim_nid_len);
+
+                               CDEBUG(D_INFO, "Adding a connection for %s\n",
+                                      primary_nid);
+
+                               rc = client_import_dyn_add_conn(
+                                       imp, &server_uuid, entry->u.nids[0], 1);
+                               if (rc < 0) {
+                                       CERROR("%s: Failed to add new connection with NID '%s' to import: rc = %d\n",
+                                              obd->obd_name, primary_nid, rc);
+                                       goto fail;
+                               }
+                               rc = client_import_add_nids_to_conn(
+                                       imp, entry->u.nids,
+                                       entry->mne_nid_count,
+                                       (struct obd_uuid *)uuid);
+                               if (rc < 0) {
+                                       CERROR("%s: failed to lookup UUID: rc = %d\n",
+                                              obd->obd_name, rc);
+                                       goto fail;
+                               }
+                       }
+fail:;
+               }
+               if (rc == -ENODEV) {
                        /* client does not connect to the OST yet */
-                       up_read(&obd->u.cli.cl_sem);
                        rc = 0;
                        continue;
                }
 
-               /* iterate all nids to find one */
-               /* find uuid by nid */
-               rc = -ENOENT;
-               for (i = 0; i < entry->mne_nid_count; i++) {
-                       rc = client_import_find_conn(obd->u.cli.cl_import,
-                                                    entry->u.nids[i],
-                                                    (struct obd_uuid *)uuid);
-                       if (rc == 0)
-                               break;
+               if (rc < 0 && rc != -ENOSPC) {
+                       CERROR("mgc: cannot find UUID by nid '%s': rc = %d\n",
+                              libcfs_nid2str(entry->u.nids[0]), rc);
+                       break;
                }
 
-               up_read(&obd->u.cli.cl_sem);
-                if (rc < 0) {
-                        CERROR("mgc: cannot find uuid by nid %s\n",
-                               libcfs_nid2str(entry->u.nids[0]));
-                        break;
-                }
-
-                CDEBUG(D_INFO, "Find uuid %s by nid %s\n",
-                       uuid, libcfs_nid2str(entry->u.nids[0]));
+               CDEBUG(D_INFO, "Found UUID '%s' by NID '%s'\n",
+                      uuid, libcfs_nid2str(entry->u.nids[0]));
 
-                pos += strlen(uuid);
-                pos += sprintf(buf + pos, "::%u", entry->mne_instance);
-                LASSERT(pos < bufsz);
+               pos += strlen(uuid);
+               pos += sprintf(buf + pos, "::%u", entry->mne_instance);
+               LASSERT(pos < bufsz);
 
-                lustre_cfg_bufs_set_string(&bufs, 1, params);
+               lustre_cfg_bufs_set_string(&bufs, 1, params);
 
                OBD_ALLOC(lcfg, lustre_cfg_len(bufs.lcfg_bufcount,
                                               bufs.lcfg_buflen));
@@ -1593,17 +1595,17 @@ static int mgc_apply_recover_logs(struct obd_device *mgc,
                lustre_cfg_init(lcfg, LCFG_PARAM, &bufs);
 
                CDEBUG(D_INFO, "ir apply logs %lld/%lld for %s -> %s\n",
-                       prev_version, max_version, obdname, params);
+                      prev_version, max_version, obdname, params);
 
-                rc = class_process_config(lcfg);
+               rc = class_process_config(lcfg);
                OBD_FREE(lcfg, lustre_cfg_len(lcfg->lcfg_bufcount,
                                              lcfg->lcfg_buflens));
-                if (rc)
-                        CDEBUG(D_INFO, "process config for %s error %d\n",
-                               obdname, rc);
+               if (rc)
+                       CDEBUG(D_INFO, "process config for %s error %d\n",
+                              obdname, rc);
 
-                /* continue, even one with error */
-        }
+               /* continue, even one with error */
+       }
 
        OBD_FREE(inst, PAGE_SIZE);
         RETURN(rc);
@@ -1639,29 +1641,28 @@ static int mgc_process_recover_nodemap_log(struct obd_device *obd,
        mgc_conn = class_exp2cliimp(cld->cld_mgcexp)->imp_connection;
 
        /* don't need to get local config */
-       if (cld_is_nodemap(cld) &&
-           (LNET_NETTYP(LNET_NIDNET(mgc_conn->c_peer.nid)) == LOLND))
+       if (cld_is_nodemap(cld) && LNetIsPeerLocal(mgc_conn->c_peer.nid))
                GOTO(out, rc = 0);
 
-        /* allocate buffer for bulk transfer.
-         * if this is the first time for this mgs to read logs,
-         * CONFIG_READ_NRPAGES_INIT will be used since it will read all logs
-         * once; otherwise, it only reads increment of logs, this should be
-         * small and CONFIG_READ_NRPAGES will be used.
-         */
-        nrpages = CONFIG_READ_NRPAGES;
+       /* allocate buffer for bulk transfer.
+        * if this is the first time for this mgs to read logs,
+        * CONFIG_READ_NRPAGES_INIT will be used since it will read all logs
+        * once; otherwise, it only reads increment of logs, this should be
+        * small and CONFIG_READ_NRPAGES will be used.
+        */
+       nrpages = CONFIG_READ_NRPAGES;
        if (cfg->cfg_last_idx == 0 || cld_is_nodemap(cld))
-                nrpages = CONFIG_READ_NRPAGES_INIT;
+               nrpages = CONFIG_READ_NRPAGES_INIT;
 
-        OBD_ALLOC(pages, sizeof(*pages) * nrpages);
-        if (pages == NULL)
-                GOTO(out, rc = -ENOMEM);
+       OBD_ALLOC_PTR_ARRAY(pages, nrpages);
+       if (pages == NULL)
+               GOTO(out, rc = -ENOMEM);
 
-        for (i = 0; i < nrpages; i++) {
+       for (i = 0; i < nrpages; i++) {
                pages[i] = alloc_page(GFP_KERNEL);
-                if (pages[i] == NULL)
-                        GOTO(out, rc = -ENOMEM);
-        }
+               if (pages[i] == NULL)
+                       GOTO(out, rc = -ENOMEM);
+       }
 
 again:
 #ifdef HAVE_SERVER_SUPPORT
@@ -1703,7 +1704,7 @@ again:
 
        /* allocate bulk transfer descriptor */
        desc = ptlrpc_prep_bulk_imp(req, nrpages, 1,
-                                   PTLRPC_BULK_PUT_SINK | PTLRPC_BULK_BUF_KIOV,
+                                   PTLRPC_BULK_PUT_SINK,
                                    MGS_BULK_PORTAL,
                                    &ptlrpc_bulk_kiov_pin_ops);
        if (desc == NULL)
@@ -1760,13 +1761,7 @@ again:
                GOTO(out, rc);
        }
 
-       mne_swab = !!ptlrpc_rep_need_swab(req);
-#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(3, 0, 53, 0)
-       /* This import flag means the server did an extra swab of IR MNE
-        * records (fixed in LU-1252), reverse it here if needed. LU-1644 */
-       if (unlikely(req->rq_import->imp_need_mne_swab))
-               mne_swab = !mne_swab;
-#endif
+       mne_swab = ptlrpc_rep_need_swab(req);
 
        /* When a nodemap config is received, we build a new nodemap config,
         * with new nodemap structs. We keep track of the most recently added
@@ -1829,7 +1824,7 @@ out:
                                break;
                        __free_page(pages[i]);
                }
-               OBD_FREE(pages, sizeof(*pages) * nrpages);
+               OBD_FREE_PTR_ARRAY(pages, nrpages);
        }
        return rc;
 }
@@ -1929,21 +1924,32 @@ static int mgc_process_cfg_log(struct obd_device *mgc,
            cli->cl_mgc_configs_dir != NULL &&
            lu2dt_dev(cli->cl_mgc_configs_dir->do_lu.lo_dev) ==
            lsi->lsi_dt_dev) {
-               if (!local_only && !lsi->lsi_dt_dev->dd_rdonly)
+               if (!local_only && !lsi->lsi_dt_dev->dd_rdonly) {
                        /* Only try to copy log if we have the lock. */
+                       CDEBUG(D_INFO, "%s: copy local log %s\n",
+                              mgc->obd_name, cld->cld_logname);
+
                        rc = mgc_llog_local_copy(env, mgc, ctxt, lctxt,
                                                 cld->cld_logname);
+                       if (!rc)
+                               lsi->lsi_flags &= ~LDD_F_NO_LOCAL_LOGS;
+               }
                if (local_only || rc) {
+                       if (unlikely(lsi->lsi_flags & LDD_F_NO_LOCAL_LOGS)
+                           || rc) {
+                               CWARN("%s: local log %s are not valid and/or remote logs are not accessbile rc = %d\n",
+                                     mgc->obd_name, cld->cld_logname, rc);
+                               GOTO(out_pop, rc = -EIO);
+                       }
+
                        if (strcmp(cld->cld_logname, PARAMS_FILENAME) != 0 &&
                            llog_is_empty(env, lctxt, cld->cld_logname)) {
-                               LCONSOLE_ERROR_MSG(0x13a, "Failed to get MGS "
-                                                  "log %s and no local copy."
-                                                  "\n", cld->cld_logname);
+                               LCONSOLE_ERROR_MSG(0x13a, "Failed to get MGS log %s and no local copy.\n",
+                                                  cld->cld_logname);
                                GOTO(out_pop, rc = -ENOENT);
                        }
-                       CDEBUG(D_MGC, "Failed to get MGS log %s, using local "
-                              "copy for now, will try to update later.\n",
-                              cld->cld_logname);
+                       CDEBUG(D_MGC, "%s: Failed to get MGS log %s, using local copy for now, will try to update later.\n",
+                              mgc->obd_name, cld->cld_logname);
                        rc = 0;
                }
                /* Now, whether we copied or not, start using the local llog.
@@ -1977,9 +1983,7 @@ static int mgc_process_cfg_log(struct obd_device *mgc,
                                             &cld->cld_cfg);
 
        /*
-        * update settings on existing OBDs. doing it inside
-        * of llog_process_lock so no device is attaching/detaching
-        * in parallel.
+        * update settings on existing OBDs.
         * the logname must be <fsname>-sptlrpc
         */
        if (rc == 0 && cld_is_sptlrpc(cld))
@@ -2058,12 +2062,12 @@ restart:
        mutex_lock(&cld->cld_lock);
        if (cld->cld_stopping) {
                mutex_unlock(&cld->cld_lock);
-                RETURN(0);
-        }
+               RETURN(0);
+       }
 
-        OBD_FAIL_TIMEOUT(OBD_FAIL_MGC_PAUSE_PROCESS_LOG, 20);
+       OBD_FAIL_TIMEOUT(OBD_FAIL_MGC_PAUSE_PROCESS_LOG, 20);
 
-       CDEBUG(D_MGC, "Process log %s:%p from %d\n", cld->cld_logname,
+       CDEBUG(D_MGC, "Process log %s-%016lx from %d\n", cld->cld_logname,
               cld->cld_cfg.cfg_instance, cld->cld_cfg.cfg_last_idx + 1);
 
        /* Get the cfg lock on the llog */
@@ -2075,15 +2079,17 @@ restart:
                /* Get the cld, it will be released in mgc_blocking_ast. */
                config_log_get(cld);
                rc = ldlm_lock_set_data(&lockh, (void *)cld);
+               LASSERT(!lustre_handle_is_used(&cld->cld_lockh));
                LASSERT(rc == 0);
+               cld->cld_lockh = lockh;
        } else {
                CDEBUG(D_MGC, "Can't get cfg lock: %d\n", rcl);
+               cld->cld_lockh.cookie = 0;
 
                if (rcl == -ESHUTDOWN &&
                    atomic_read(&mgc->u.cli.cl_mgc_refcount) > 0 && !retry) {
                        struct obd_import *imp;
-                       struct l_wait_info lwi;
-                       int secs = cfs_time_seconds(obd_timeout);
+                       long timeout = cfs_time_seconds(obd_timeout);
 
                        mutex_unlock(&cld->cld_lock);
                        imp = class_exp2cliimp(mgc->u.cli.cl_mgc_mgsexp);
@@ -2096,15 +2102,20 @@ restart:
                         * FULL or closed */
                        ptlrpc_pinger_force(imp);
 
-                       lwi = LWI_TIMEOUT(secs, NULL, NULL);
-                       l_wait_event(imp->imp_recovery_waitq,
-                                    !mgc_import_in_recovery(imp), &lwi);
+                       wait_event_idle_timeout(imp->imp_recovery_waitq,
+                                               !mgc_import_in_recovery(imp),
+                                               timeout);
 
                        if (imp->imp_state == LUSTRE_IMP_FULL) {
                                retry = true;
                                goto restart;
                        } else {
                                mutex_lock(&cld->cld_lock);
+                               /* unlock/lock mutex, so check stopping again */
+                               if (cld->cld_stopping) {
+                                       mutex_unlock(&cld->cld_lock);
+                                       RETURN(0);
+                               }
                                spin_lock(&config_list_lock);
                                cld->cld_lostlock = 1;
                                spin_unlock(&config_list_lock);
@@ -2124,16 +2135,6 @@ restart:
                else if (cld_is_nodemap(cld))
                        rc = rcl;
 
-               if (cld_is_recover(cld) && rc) {
-                       if (!rcl) {
-                               CERROR("%s: recover log %s failed, not fatal: rc = %d\n",
-                                      mgc->obd_name, cld->cld_logname, rc);
-                               spin_lock(&config_list_lock);
-                               cld->cld_lostlock = 1;
-                               spin_unlock(&config_list_lock);
-                       }
-                       rc = 0; /* this is not a fatal error for recover log */
-               }
        } else if (!cld_is_barrier(cld)) {
                rc = mgc_process_cfg_log(mgc, cld, rcl != 0);
        }
@@ -2141,17 +2142,20 @@ restart:
        CDEBUG(D_MGC, "%s: configuration from log '%s' %sed (%d).\n",
               mgc->obd_name, cld->cld_logname, rc ? "fail" : "succeed", rc);
 
-       mutex_unlock(&cld->cld_lock);
-
        /* Now drop the lock so MGS can revoke it */
        if (!rcl) {
                rcl = mgc_cancel(mgc->u.cli.cl_mgc_mgsexp, LCK_CR, &lockh);
                if (rcl)
                        CERROR("Can't drop cfg lock: %d\n", rcl);
        }
+       mutex_unlock(&cld->cld_lock);
 
        /* requeue nodemap lock immediately if transfer was interrupted */
-       if (cld_is_nodemap(cld) && rc == -EAGAIN) {
+       if ((cld_is_nodemap(cld) && rc == -EAGAIN) ||
+           (cld_is_recover(cld) && rc)) {
+               if (cld_is_recover(cld))
+                       CWARN("%s: IR log %s failed, not fatal: rc = %d\n",
+                             mgc->obd_name, cld->cld_logname, rc);
                mgc_requeue_add(cld);
                rc = 0;
        }
@@ -2214,11 +2218,6 @@ static int mgc_process_config(struct obd_device *obd, size_t len, void *buf)
                        break;
                }
 
-               /* COMPAT_146 */
-               /* FIXME only set this for old logs!  Right now this forces
-                  us to always skip the "inside markers" check */
-               cld->cld_cfg.cfg_flags |= CFG_F_COMPAT146;
-
                rc = mgc_process_log(obd, cld);
                if (rc == 0 && cld->cld_recover != NULL) {
                        if (OCD_HAS_FLAG(&obd->u.cli.cl_import->
@@ -2272,7 +2271,7 @@ out:
         RETURN(rc);
 }
 
-static struct obd_ops mgc_obd_ops = {
+static const struct obd_ops mgc_obd_ops = {
         .o_owner        = THIS_MODULE,
         .o_setup        = mgc_setup,
         .o_precleanup   = mgc_precleanup,
@@ -2289,7 +2288,7 @@ static struct obd_ops mgc_obd_ops = {
 
 static int __init mgc_init(void)
 {
-       return class_register_type(&mgc_obd_ops, NULL, false, NULL,
+       return class_register_type(&mgc_obd_ops, NULL, false,
                                   LUSTRE_MGC_NAME, NULL);
 }