File 0114-Assemble-remove-the-protection-when-clustered-raid-d.patch of Package mdadm.10093

From 1ed0ae2aad2cbd975ba0a30963fd67e7b66c410f Mon Sep 17 00:00:00 2001
From: Guoqing Jiang <gqjiang@suse.com>
Date: Tue, 2 Apr 2019 10:52:21 +0800
Subject: [PATCH] Assemble: remove the protection when clustered raid do
 assemble
Git-commit: 80d1256e98b4995b0a753678f6c7e4a8bb369741
Patch-mainline: mdadm-4.0+
References: bsc#1127385

For HA product, RA (resource agent) assembles cluster raid
through call below cmd:

$MDADM --assemble $mddev --config=$RAIDCONF $MDADM_HOMEHOST

Sometimes node can't assemble array because all the nodes
need to contend dlm lock, which causes node fence in automatic
test.

And in fact, we don't need the protection since the assemble
cmd called by RA doesn't change superblock, so revert the
commit 76781701a487090172d32befae07671a10ea88ad ("Assemble:
	provide protection when clustered raid do assemble") to remove
unneccessary protection.

Signed-off-by: Guoqing Jiang <gqjiang@suse.com>
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
---
 Assemble.c | 21 ---------------------
 1 file changed, 21 deletions(-)

diff --git a/Assemble.c b/Assemble.c
index 5e0405b319e7..d78d5337b9d3 100644
--- a/Assemble.c
+++ b/Assemble.c
@@ -1328,9 +1328,6 @@ int Assemble(struct supertype *st, char *mddev,
 	char chosen_name[1024];
 	struct map_ent *map = NULL;
 	struct map_ent *mp;
-	int locked = 0;
-	struct mdp_superblock_1 *sb;
-	bitmap_super_t *bms;
 
 	/*
 	 * If any subdevs are listed, then any that don't
@@ -1361,12 +1358,6 @@ try_again:
 	 * set of devices failed.  Those are now marked as ->used==2 and
 	 * we ignore them and try again
 	 */
-	if (locked)
-		/*
-		 * if come back try_again is called, then need to unlock first,
-		 * and lock again since the metadate is re-read.
-		 */
-		cluster_release_dlmlock();
 	if (!st && ident->st)
 		st = ident->st;
 	if (c->verbose>0)
@@ -1384,14 +1375,6 @@ try_again:
 	if (!st || !st->sb || !content)
 		return 2;
 
-	sb = st->sb;
-	bms = (bitmap_super_t*)(((char*)sb) + 4096);
-	if (sb && bms->version == BITMAP_MAJOR_CLUSTERED) {
-		locked = cluster_get_dlmlock();
-		if (locked != 1)
-			return 1;
-	}
-
 	/* We have a full set of devices - we now need to find the
 	 * array device.
 	 * However there is a risk that we are racing with "mdadm -I"
@@ -1531,8 +1514,6 @@ try_again:
 		err = assemble_container_content(st, mdfd, content, c,
 						 chosen_name, NULL);
 		close(mdfd);
-		if (locked == 1)
-			cluster_release_dlmlock();
 		return err;
 	}
 #endif
@@ -1877,8 +1858,6 @@ out:
 		close(mdfd);
 
 	/* '2' means 'OK, but not started yet' */
-	if (locked == 1)
-		cluster_release_dlmlock();
 	if (rv == -1) {
 		free(devices);
 		return 1;
-- 
2.12.3