NeilBrown: ddf: fixed 'working_disks' reported by container_content.
Martin F. Krafft
madduck at alioth.debian.org
Thu May 7 12:11:37 UTC 2009
Module: mdadm
Branch: debian/experimental
Commit: c0d50ca58423168f2b86ba674227a9bbc33e1501
URL: http://git.debian.org/?p=pkg-mdadm/mdadm.git;a=commit;h=c0d50ca58423168f2b86ba674227a9bbc33e1501
Author: NeilBrown <neilb at suse.de>
Date: Tue Apr 7 17:41:55 2009 +1000
ddf: fixed 'working_disks' reported by container_content.
The 'work_disks' number should be the number that is expected, not the
number found so far. This is needed for Incremental assembly to
start the array at the right time.
Signed-off-by: NeilBrown <neilb at suse.de>
---
super-ddf.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/super-ddf.c b/super-ddf.c
index 8ab3597..93c8f06 100644
--- a/super-ddf.c
+++ b/super-ddf.c
@@ -2918,6 +2918,8 @@ static struct mdinfo *container_content_ddf(struct supertype *st)
if (vc->conf.phys_refnum[i] == 0xFFFFFFFF)
continue;
+ this->array.working_disks++;
+
for (d = ddf->dlist; d ; d=d->next)
if (d->disk.refnum == vc->conf.phys_refnum[i])
break;
@@ -2925,8 +2927,6 @@ static struct mdinfo *container_content_ddf(struct supertype *st)
/* Haven't found that one yet, maybe there are others */
continue;
- this->array.working_disks++;
-
dev = malloc(sizeof(*dev));
memset(dev, 0, sizeof(*dev));
dev->next = this->devs;
More information about the pkg-mdadm-commits
mailing list