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

Monitor: avoid adding too many spares to container 

Tests revealed that sometimes there are still more spares taken
than needed. The reason for this is that after adding one spare
to container with degraded subarray
if between ioctl in main loop and load_container in try_spare_migration
mdmon activates the spare we see active<raid but find no spares in parent
container and so add an extra spare.

To prevent such behaviour we count active disks in the list returned
by getinfo_super_disks and compare it with subarray->active.
If the number has increased it means new spare was added and activated
so there is no need for more.

Signed-off-by: Anna Czarnowska <anna.czarnowska@intel.com>
Signed-off-by: NeilBrown <neilb@suse.de>
This commit is contained in:
Czarnowska, Anna 2011-01-17 10:06:10 +00:00 committed by NeilBrown
parent 24aebf3add
commit a1e49d6956
1 changed files with 30 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
Monitor.c
View File

@ -784,7 +784,7 @@ static dev_t choose_spare(struct state *from, struct state *to,
static dev_t container_choose_spare(struct state *from, struct state *to,
struct domainlist *domlist,
unsigned long long min_size)
unsigned long long min_size, int active)
{
/* This is similar to choose_spare, but we cannot trust devstate,
* so we need to read the metadata instead
@ -807,6 +807,33 @@ static dev_t container_choose_spare(struct state *from, struct state *to,
if (err)
return 0;
if (from == to) {
/* We must check if number of active disks has not increased
* since ioctl in main loop. mdmon may have added spare
* to subarray. If so we do not need to look for more spares
* so return non zero value */
int active_cnt = 0;
struct mdinfo *dp;
list = st->ss->getinfo_super_disks(st);
if (!list) {
st->ss->free_super(st);
return 1;
}
dp = list->devs;
while (dp) {
if (dp->disk.state & (1<<MD_DISK_SYNC) &&
!(dp->disk.state & (1<<MD_DISK_FAULTY)))
active_cnt++;
dp = dp->next;
}
sysfs_free(list);
if (active < active_cnt) {
/* Spare just activated.*/
st->ss->free_super(st);
return 1;
}
}
/* We only need one spare so full list not needed */
list = container_choose_spares(st, min_size, domlist, from->spare_group,
to->metadata->ss->name, 1);
@ -851,7 +878,7 @@ static void try_spare_migration(struct state *statelist, struct alert_info *info
* no suitable spare in container already.
* If there is we don't add more */
dev_t devid = container_choose_spare(
to, to, NULL, min_size);
to, to, NULL, min_size, st->active);
if (devid > 0)
continue;
}
@ -874,7 +901,7 @@ static void try_spare_migration(struct state *statelist, struct alert_info *info
continue;
if (from->metadata->ss->external)
devid = container_choose_spare(
from, to, domlist, min_size);
from, to, domlist, min_size, 0);
else
devid = choose_spare(from, to, domlist,
min_size);