[PATCH] FIX: Do not allow for multiple reshape_array() execution during reshape_container() call

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

It can happen during reshape restart that reshape_array() can exit without
error (e.g. Grow.c:1915) and reshape is not moved to next array.
reshape_array() is called again for the same device.
Do not allow for such execution and check if last reshaped array is not
the current one.
This patch can be treat not as solution, but it allows for such errors

Signed-off-by: Adam Kwolek <adam.kwolek@xxxxxxxxx>

 Grow.c |   17 +++++++++++++++++
 1 files changed, 17 insertions(+), 0 deletions(-)

diff --git a/Grow.c b/Grow.c
index 184a973..66863f1 100644
--- a/Grow.c
+++ b/Grow.c
@@ -2462,6 +2462,7 @@ int reshape_container(char *container, char *devname,
 	struct mdinfo *cc = NULL;
 	int rv = restart;
+	int last_devnum = -1;
 	/* component_size is not meaningful for a container,
 	 * so pass '-1' meaning 'no change'
@@ -2546,6 +2547,22 @@ int reshape_container(char *container, char *devname,
 		if (!adev)
 			adev = content->text_version;
+		if (last_devnum == mdstat->devnum) {
+			/* Do not allow for multiple reshape_array() call for
+			 * the same array.
+			 * It can happen when resahpe_array() returns without
+			 * error, when reshape is not finished (wrong reshape
+			 * starting/continuation conditions). Mdmon doesn't
+			 * switch to next array in container and reentry
+			 * conditions for the same array occur.
+			 */
+			printf(Name ": Multiple reshape execution detected for "
+			       "device  %s.", adev);
+			close(fd);
+			break;
+		}
+		last_devnum = mdstat->devnum;
 		sysfs_init(content, fd, mdstat->devnum);
 		rv = reshape_array(container, fd, adev, st,

To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[ATA RAID]     [Linux SCSI Target Infrastructure]     [Managing RAID on Linux]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device-Mapper]     [Kernel]     [Linux Books]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Photos]     [Yosemite Photos]     [Yosemite News]     [AMD 64]     [Linux Networking]

Add to Google Powered by Linux