X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Fmdt%2Fmdt_handler.c;h=94a675a81fe7146449b9aed269195eb1c32c641f;hb=75a417fa0065d52a31215daaaaf41c0fa9751a89;hp=ace6f1fcc93ad47cf51f7f27407d2e0d5aa39e96;hpb=e02cb40761ff8aae3df76c4210a345420b6d4ba1;p=fs%2Flustre-release.git

diff --git a/lustre/mdt/mdt_handler.c b/lustre/mdt/mdt_handler.c
index ace6f1f..94a675a 100644
--- a/lustre/mdt/mdt_handler.c
+++ b/lustre/mdt/mdt_handler.c
@@ -436,8 +436,13 @@ static int mdt_statfs(struct tgt_session_info *tsi)
 	if (!osfs)
 		GOTO(out, rc = -EPROTO);
 
-	if (mdt_is_sum_statfs_client(req->rq_export))
+	if (mdt_is_sum_statfs_client(req->rq_export) &&
+		lustre_packed_msg_size(req->rq_reqmsg) ==
+		req_capsule_fmt_size(req->rq_reqmsg->lm_magic,
+				     &RQF_MDS_STATFS_NEW, RCL_CLIENT)) {
+		req_capsule_extend(info->mti_pill, &RQF_MDS_STATFS_NEW);
 		reqbody = req_capsule_client_get(info->mti_pill, &RMF_MDT_BODY);
+	}
 
 	if (reqbody && reqbody->mbo_valid & OBD_MD_FLAGSTATFS)
 		msf = &mdt->mdt_sum_osfs;
@@ -1522,12 +1527,12 @@ static int mdt_swap_layouts(struct tgt_session_info *tsi)
 	/* permission check. Make sure the calling process having permission
 	 * to write both files. */
 	rc = mo_permission(info->mti_env, NULL, mdt_object_child(o1), NULL,
-				MAY_WRITE);
+			   MAY_WRITE);
 	if (rc < 0)
 		GOTO(put, rc);
 
 	rc = mo_permission(info->mti_env, NULL, mdt_object_child(o2), NULL,
-				MAY_WRITE);
+			   MAY_WRITE);
 	if (rc < 0)
 		GOTO(put, rc);
 
@@ -2653,6 +2658,7 @@ int mdt_blocking_ast(struct ldlm_lock *lock, struct ldlm_lock_desc *desc,
 {
 	struct obd_device *obd = ldlm_lock_to_ns(lock)->ns_obd;
 	struct mdt_device *mdt = mdt_dev(obd->obd_lu_dev);
+	struct ldlm_cb_set_arg *arg = data;
 	bool commit_async = false;
 	int rc;
 	ENTRY;
@@ -2665,17 +2671,22 @@ int mdt_blocking_ast(struct ldlm_lock *lock, struct ldlm_lock_desc *desc,
 		unlock_res_and_lock(lock);
 		RETURN(0);
 	}
-	/* There is no lock conflict if l_blocking_lock == NULL,
-	 * it indicates a blocking ast sent from ldlm_lock_decref_internal
-	 * when the last reference to a local lock was released */
-	if (lock->l_req_mode & (LCK_PW | LCK_EX) &&
-	    lock->l_blocking_lock != NULL) {
+
+	/* A blocking ast may be sent from ldlm_lock_decref_internal
+	 * when the last reference to a local lock was released and
+	 * during blocking event from ldlm_work_bl_ast_lock().
+	 * The 'data' parameter is l_ast_data in the first case and
+	 * callback arguments in the second one. Distinguish them by that.
+	 */
+	if (!data || data == lock->l_ast_data || !arg->bl_desc)
+		goto skip_cos_checks;
+
+	if (lock->l_req_mode & (LCK_PW | LCK_EX)) {
 		if (mdt_cos_is_enabled(mdt)) {
-			if (lock->l_client_cookie !=
-			    lock->l_blocking_lock->l_client_cookie)
+			if (!arg->bl_desc->bl_same_client)
 				mdt_set_lock_sync(lock);
 		} else if (mdt_slc_is_enabled(mdt) &&
-			   ldlm_is_cos_incompat(lock->l_blocking_lock)) {
+			   arg->bl_desc->bl_cos_incompat) {
 			mdt_set_lock_sync(lock);
 			/*
 			 * we may do extra commit here, but there is a small
@@ -2689,11 +2700,11 @@ int mdt_blocking_ast(struct ldlm_lock *lock, struct ldlm_lock_desc *desc,
 			 */
 			commit_async = true;
 		}
-	} else if (lock->l_req_mode == LCK_COS &&
-		   lock->l_blocking_lock != NULL) {
+	} else if (lock->l_req_mode == LCK_COS) {
 		commit_async = true;
 	}
 
+skip_cos_checks:
 	rc = ldlm_blocking_ast_nocheck(lock);
 
 	if (commit_async) {
@@ -3046,6 +3057,10 @@ mdt_object_lock_internal(struct mdt_thread_info *info, struct mdt_object *o,
 		}
 	}
 
+	/* other components like LFSCK can use lockless access
+	 * and populate cache, so we better invalidate it */
+	mo_invalidate(info->mti_env, mdt_object_child(o));
+
 	RETURN(0);
 }
 
@@ -3788,6 +3803,8 @@ static int mdt_intent_layout(enum ldlm_intent_flags it_opc,
 			if (layout_size > info->mti_mdt->mdt_max_mdsize)
 				info->mti_mdt->mdt_max_mdsize = layout_size;
 		}
+		CDEBUG(D_INFO, "%s: layout_size %d\n",
+		       mdt_obd_name(info->mti_mdt), layout_size);
 	}
 
 	/*
@@ -3859,7 +3876,7 @@ out_obj:
 out:
 	lhc->mlh_reg_lh.cookie = 0;
 
-	return rc;
+	RETURN(rc);
 }
 
 static int mdt_intent_open(enum ldlm_intent_flags it_opc,
@@ -5027,7 +5044,7 @@ static int mdt_init0(const struct lu_env *env, struct mdt_device *m,
 	obd = class_name2obd(dev);
 	LASSERT(obd != NULL);
 
-	m->mdt_max_mdsize = MAX_MD_SIZE; /* 4 stripes */
+	m->mdt_max_mdsize = MAX_MD_SIZE_OLD;
 	m->mdt_opts.mo_evict_tgt_nids = 1;
 	m->mdt_opts.mo_cos = MDT_COS_DEFAULT;
 
@@ -5746,7 +5763,7 @@ static int mdt_export_cleanup(struct obd_export *exp)
 			 * archive request into a noop if it's not actually
 			 * dirty.
 			 */
-			if (mfd->mfd_mode & MDS_FMODE_WRITE)
+			if (mfd->mfd_open_flags & MDS_FMODE_WRITE)
 				rc = mdt_ctxt_add_dirty_flag(&env, info, mfd);
 
 			/* Don't unlink orphan on failover umount, LU-184 */