Whamcloud - gitweb
LU-11868 mdc: Improve xattr buffer allocations
[fs/lustre-release.git] / lustre / mdc / mdc_reint.c
index 73572b4..03e6aac 100644 (file)
@@ -131,13 +131,14 @@ int mdc_setattr(struct obd_export *exp, struct md_op_data *op_data,
        }
 
         if (op_data->op_attr.ia_valid & (ATTR_MTIME | ATTR_CTIME))
-               CDEBUG(D_INODE, "setting mtime %ld, ctime %ld\n",
-                       LTIME_S(op_data->op_attr.ia_mtime),
-                       LTIME_S(op_data->op_attr.ia_ctime));
+               CDEBUG(D_INODE, "setting mtime %lld, ctime %lld\n",
+                      (s64)op_data->op_attr.ia_mtime.tv_sec,
+                      (s64)op_data->op_attr.ia_ctime.tv_sec);
        mdc_setattr_pack(req, op_data, ea, ealen);
 
        req_capsule_set_size(&req->rq_pill, &RMF_ACL, RCL_SERVER,
-                            req->rq_import->imp_connect_data.ocd_max_easize);
+                            MIN(req->rq_import->imp_connect_data.ocd_max_easize,
+                                XATTR_SIZE_MAX));
         ptlrpc_request_set_replen(req);
 
        rc = mdc_reint(req, LUSTRE_IMP_FULL);
@@ -234,17 +235,19 @@ rebuild:
                 level = LUSTRE_IMP_RECOVER;
                 goto resend;
         } else if (rc == -EINPROGRESS) {
-                /* Retry create infinitely until succeed or get other
-                 * error code. */
-                ptlrpc_req_finished(req);
-                resends++;
-
-                CDEBUG(D_HA, "%s: resend:%d create on "DFID"/"DFID"\n",
-                       exp->exp_obd->obd_name, resends,
-                       PFID(&op_data->op_fid1), PFID(&op_data->op_fid2));
-
-                if (generation == import->imp_generation) {
-                        goto rebuild;
+               /* Retry create infinitely until succeed or get other
+                * error code or interrupted. */
+               ptlrpc_req_finished(req);
+               if (generation == import->imp_generation) {
+                       if (signal_pending(current))
+                               RETURN(-EINTR);
+
+                       resends++;
+                       CDEBUG(D_HA, "%s: resend:%d create on "DFID"/"DFID"\n",
+                              exp->exp_obd->obd_name, resends,
+                              PFID(&op_data->op_fid1),
+                              PFID(&op_data->op_fid2));
+                       goto rebuild;
                 } else {
                         CDEBUG(D_HA, "resend cross eviction\n");
                         RETURN(-EIO);
@@ -357,26 +360,27 @@ int mdc_rename(struct obd_export *exp, struct md_op_data *op_data,
                struct ptlrpc_request **request)
 {
        struct list_head cancels = LIST_HEAD_INIT(cancels);
-        struct obd_device *obd = exp->exp_obd;
-        struct ptlrpc_request *req;
-        int count = 0, rc;
-        ENTRY;
+       struct obd_device *obd = exp->exp_obd;
+       struct ptlrpc_request *req;
+       int count = 0, rc;
 
-        if ((op_data->op_flags & MF_MDC_CANCEL_FID1) &&
-            (fid_is_sane(&op_data->op_fid1)))
-                count = mdc_resource_get_unused(exp, &op_data->op_fid1,
-                                                &cancels, LCK_EX,
-                                                MDS_INODELOCK_UPDATE);
-        if ((op_data->op_flags & MF_MDC_CANCEL_FID2) &&
-            (fid_is_sane(&op_data->op_fid2)))
-                count += mdc_resource_get_unused(exp, &op_data->op_fid2,
-                                                 &cancels, LCK_EX,
-                                                 MDS_INODELOCK_UPDATE);
-        if ((op_data->op_flags & MF_MDC_CANCEL_FID3) &&
-            (fid_is_sane(&op_data->op_fid3)))
-                count += mdc_resource_get_unused(exp, &op_data->op_fid3,
-                                                 &cancels, LCK_EX,
-                                                 MDS_INODELOCK_LOOKUP);
+       ENTRY;
+
+       if ((op_data->op_flags & MF_MDC_CANCEL_FID1) &&
+           (fid_is_sane(&op_data->op_fid1)))
+               count = mdc_resource_get_unused(exp, &op_data->op_fid1,
+                                               &cancels, LCK_EX,
+                                               MDS_INODELOCK_UPDATE);
+       if ((op_data->op_flags & MF_MDC_CANCEL_FID2) &&
+           (fid_is_sane(&op_data->op_fid2)))
+               count += mdc_resource_get_unused(exp, &op_data->op_fid2,
+                                                &cancels, LCK_EX,
+                                                MDS_INODELOCK_UPDATE);
+       if ((op_data->op_flags & MF_MDC_CANCEL_FID3) &&
+           (fid_is_sane(&op_data->op_fid3)))
+               count += mdc_resource_get_unused(exp, &op_data->op_fid3,
+                                                &cancels, LCK_EX,
+                                                MDS_INODELOCK_LOOKUP);
        if ((op_data->op_flags & MF_MDC_CANCEL_FID4) &&
            (fid_is_sane(&op_data->op_fid4)))
                count += mdc_resource_get_unused(exp, &op_data->op_fid4,
@@ -391,8 +395,11 @@ int mdc_rename(struct obd_export *exp, struct md_op_data *op_data,
                RETURN(-ENOMEM);
        }
 
-        req_capsule_set_size(&req->rq_pill, &RMF_NAME, RCL_CLIENT, oldlen + 1);
-        req_capsule_set_size(&req->rq_pill, &RMF_SYMTGT, RCL_CLIENT, newlen+1);
+       req_capsule_set_size(&req->rq_pill, &RMF_NAME, RCL_CLIENT, oldlen + 1);
+       req_capsule_set_size(&req->rq_pill, &RMF_SYMTGT, RCL_CLIENT, newlen+1);
+       if (op_data->op_cli_flags & CLI_MIGRATE)
+               req_capsule_set_size(&req->rq_pill, &RMF_EADATA, RCL_CLIENT,
+                                    op_data->op_data_size);
 
        rc = mdc_prep_elc_req(exp, req, MDS_REINT, &cancels, count);
        if (rc) {
@@ -400,36 +407,24 @@ int mdc_rename(struct obd_export *exp, struct md_op_data *op_data,
                RETURN(rc);
        }
 
-       if (op_data->op_cli_flags & CLI_MIGRATE && op_data->op_data != NULL) {
-               struct md_open_data *mod = op_data->op_data;
-
-               LASSERTF(mod->mod_open_req != NULL &&
-                        mod->mod_open_req->rq_type != LI_POISON,
-                        "POISONED open %p!\n", mod->mod_open_req);
-
-               DEBUG_REQ(D_HA, mod->mod_open_req, "matched open");
-               /* We no longer want to preserve this open for replay even
-                * though the open was committed. b=3632, b=3633 */
-               spin_lock(&mod->mod_open_req->rq_lock);
-               mod->mod_open_req->rq_replay = 0;
-               spin_unlock(&mod->mod_open_req->rq_lock);
-       }
-
-        if (exp_connect_cancelset(exp) && req)
-                ldlm_cli_cancel_list(&cancels, count, req, 0);
+       if (exp_connect_cancelset(exp) && req)
+               ldlm_cli_cancel_list(&cancels, count, req, 0);
 
-        mdc_rename_pack(req, op_data, old, oldlen, new, newlen);
+       if (op_data->op_cli_flags & CLI_MIGRATE)
+               mdc_migrate_pack(req, op_data, old, oldlen);
+       else
+               mdc_rename_pack(req, op_data, old, oldlen, new, newlen);
 
        req_capsule_set_size(&req->rq_pill, &RMF_MDT_MD, RCL_SERVER,
                             obd->u.cli.cl_default_mds_easize);
        ptlrpc_request_set_replen(req);
 
        rc = mdc_reint(req, LUSTRE_IMP_FULL);
-        *request = req;
-        if (rc == -ERESTARTSYS)
-                rc = 0;
+       *request = req;
+       if (rc == -ERESTARTSYS)
+               rc = 0;
 
-        RETURN(rc);
+       RETURN(rc);
 }
 
 int mdc_file_resync(struct obd_export *exp, struct md_op_data *op_data)
@@ -468,10 +463,11 @@ int mdc_file_resync(struct obd_export *exp, struct md_op_data *op_data)
        rec->rs_cap     = op_data->op_cap;
        rec->rs_fid     = op_data->op_fid1;
        rec->rs_bias    = op_data->op_bias;
+       rec->rs_mirror_id = op_data->op_mirror_id;
 
-       lock = ldlm_handle2lock(&op_data->op_handle);
+       lock = ldlm_handle2lock(&op_data->op_lease_handle);
        if (lock != NULL) {
-               rec->rs_handle = lock->l_remote_handle;
+               rec->rs_lease_handle = lock->l_remote_handle;
                LDLM_LOCK_PUT(lock);
        }