Martchus
/
mdadm
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

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

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
detection.

Signed-off-by: Adam Kwolek <adam.kwolek@intel.com>
Signed-off-by: NeilBrown <neilb@suse.de>
This commit is contained in:
Adam Kwolek 2011-12-15 14:34:36 +11:00 committed by NeilBrown
parent a648241517
commit 2d04d7e5c3
1 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
Grow.c
View File

@ -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,24 @@ 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() calls for
* the same array.
* It can happen when reshape_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.
* This is possibly interim until the behaviour of
* reshape_array is resolved().
*/
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,