LU-6158 mdt: always shrink_capsule in getxattr_all

[fs/lustre-release.git] / lustre / fld / fld_request.c

diff --git a/lustre/fld/fld_request.c b/lustre/fld/fld_request.c
index 3ef67a7..a59ab95 100644 (file)
--- a/lustre/fld/fld_request.c
+++ b/lustre/fld/fld_request.c
@@ -164,15 +164,8 @@ int fld_client_add_target(struct lu_client_fld *fld,
         LASSERT(name != NULL);
         LASSERT(tar->ft_srv != NULL || tar->ft_exp != NULL);
 
-        if (fld->lcf_flags != LUSTRE_FLD_INIT) {
-                CERROR("%s: Attempt to add target %s (idx "LPU64") "
-                       "on fly - skip it\n", fld->lcf_name, name,
-                       tar->ft_idx);
-                RETURN(0);
-        } else {
-                CDEBUG(D_INFO, "%s: Adding target %s (idx "
-                       LPU64")\n", fld->lcf_name, name, tar->ft_idx);
-        }
+       CDEBUG(D_INFO, "%s: Adding target %s (idx "LPU64")\n", fld->lcf_name,
+              name, tar->ft_idx);
 
         OBD_ALLOC_PTR(target);
         if (target == NULL)
@@ -227,7 +220,6 @@ int fld_client_del_target(struct lu_client_fld *fld, __u64 idx)
        spin_unlock(&fld->lcf_lock);
        RETURN(-ENOENT);
 }
-EXPORT_SYMBOL(fld_client_del_target);
 
 #ifdef CONFIG_PROC_FS
 static int fld_client_proc_init(struct lu_client_fld *fld)
@@ -305,10 +297,9 @@ int fld_client_init(struct lu_client_fld *fld,
                 RETURN(-EINVAL);
         }
 
-        fld->lcf_count = 0;
+       fld->lcf_count = 0;
        spin_lock_init(&fld->lcf_lock);
-        fld->lcf_hash = &fld_hash[hash];
-        fld->lcf_flags = LUSTRE_FLD_INIT;
+       fld->lcf_hash = &fld_hash[hash];
        INIT_LIST_HEAD(&fld->lcf_targets);
 
         cache_size = FLD_CLIENT_CACHE_SIZE /
@@ -434,7 +425,7 @@ again:
        }
 
        if (rc != 0) {
-               if (imp->imp_state != LUSTRE_IMP_CLOSED) {
+               if (imp->imp_state != LUSTRE_IMP_CLOSED && !imp->imp_deactive) {
                        /* Since LWP is not replayable, so it will keep
                         * trying unless umount happens, otherwise it would
                         * cause unecessary failure of the application. */
@@ -471,21 +462,21 @@ int fld_client_lookup(struct lu_client_fld *fld, u64 seq, u32 *mds,
 {
        struct lu_seq_range res = { 0 };
        struct lu_fld_target *target;
+       struct lu_fld_target *origin;
        int rc;
        ENTRY;
 
-        fld->lcf_flags |= LUSTRE_FLD_RUN;
-
-        rc = fld_cache_lookup(fld->lcf_cache, seq, &res);
-        if (rc == 0) {
-                *mds = res.lsr_index;
-                RETURN(0);
-        }
+       rc = fld_cache_lookup(fld->lcf_cache, seq, &res);
+       if (rc == 0) {
+               *mds = res.lsr_index;
+               RETURN(0);
+       }
 
         /* Can not find it in the cache */
         target = fld_client_get_target(fld, seq);
         LASSERT(target != NULL);
-
+       origin = target;
+again:
         CDEBUG(D_INFO, "%s: Lookup fld entry (seq: "LPX64") on "
                "target %s (idx "LPU64")\n", fld->lcf_name, seq,
                fld_target_name(target), target->ft_idx);
@@ -503,6 +494,22 @@ int fld_client_lookup(struct lu_client_fld *fld, u64 seq, u32 *mds,
                rc = fld_client_rpc(target->ft_exp, &res, FLD_QUERY, NULL);
        }
 
+       if (rc == -ESHUTDOWN) {
+               /* If fld lookup failed because the target has been shutdown,
+                * then try next target in the list, until trying all targets
+                * or fld lookup succeeds */
+               spin_lock(&fld->lcf_lock);
+               if (target->ft_chain.next == fld->lcf_targets.prev)
+                       target = list_entry(fld->lcf_targets.next,
+                                           struct lu_fld_target, ft_chain);
+               else
+                       target = list_entry(target->ft_chain.next,
+                                                struct lu_fld_target,
+                                                ft_chain);
+               spin_unlock(&fld->lcf_lock);
+               if (target != origin)
+                       goto again;
+       }
        if (rc == 0) {
                *mds = res.lsr_index;
                fld_cache_insert(fld->lcf_cache, &res);
@@ -516,7 +523,6 @@ void fld_client_flush(struct lu_client_fld *fld)
 {
         fld_cache_flush(fld->lcf_cache);
 }
-EXPORT_SYMBOL(fld_client_flush);
 
 
 struct proc_dir_entry *fld_type_proc_dir;
@@ -550,6 +556,8 @@ static void __exit fld_mod_exit(void)
 
 MODULE_AUTHOR("Sun Microsystems, Inc. <http://www.lustre.org/>");
 MODULE_DESCRIPTION("Lustre FLD");
+MODULE_VERSION(LUSTRE_VERSION_STRING);
 MODULE_LICENSE("GPL");
 
-cfs_module(mdd, LUSTRE_VERSION_STRING, fld_mod_init, fld_mod_exit);
+module_init(fld_mod_init);
+module_exit(fld_mod_exit);