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