Thread (5 messages) 5 messages, 2 authors, 2011-01-31

[PATCH] imsm: Update metadata for second array

flat view
STALE5736d

From: Adam Kwolek <hidden>
Date: 2011-01-28 08:21:41
Subsystem: the rest · Maintainer: Linus Torvalds

When second array reshape is about to start external metadata should be updated
by mdmon in imsm_set_array_state().
for this purposes imsm_progress_container_reshape() is reused.

In imsm_progress_container_reshape() we have to make sure that prev_disks
shows expanded disks number. Current decision about prev_disks in 'for'
loop doesn't make us sure about this. We are getting first positive value,
but not necessarily right one.

prev_disks have to be found before we enter 'for' loop.

Signed-off-by: Adam Kwolek <redacted>
---

 super-intel.c |   15 ++++++++++++++-
 1 files changed, 14 insertions(+), 1 deletions(-)
diff --git a/super-intel.c b/super-intel.c
index d5501bd..4ae4764 100644
--- a/super-intel.c
+++ b/super-intel.c
@@ -5062,6 +5062,15 @@ static void imsm_progress_container_reshape(struct intel_super *super)
 	int prev_disks = -1;
 	int i;
 
+	/* find maximum number of disks used in any array
+	 */
+	for (i = 0; i < mpb->num_raid_devs; i++) {
+		struct imsm_dev *dev = get_imsm_dev(super, i);
+		struct imsm_map *map = get_imsm_map(dev, 0);
+		if (map->num_members > prev_disks)
+			prev_disks = map->num_members;
+	}
+
 	for (i = 0; i < mpb->num_raid_devs; i++) {
 		struct imsm_dev *dev = get_imsm_dev(super, i);
 		struct imsm_map *map = get_imsm_map(dev, 0);
@@ -5253,13 +5262,17 @@ static int imsm_set_array_state(struct active_array *a, int consistent)
 		super->updates_pending++;
 	}
 
-	/* finalize online capacity expansion/reshape */
+	/* manage online capacity expansion/reshape */
 	if ((a->curr_action != reshape) &&
 	    (a->prev_action == reshape)) {
 		struct mdinfo *mdi;
 
+		/* finalize online capacity expansion/reshape */
 		for (mdi = a->info.devs; mdi; mdi = mdi->next)
 			imsm_set_disk(a, mdi->disk.raid_disk, mdi->curr_state);
+
+		/* check next volume reshape */
+		imsm_progress_container_reshape(super);
 	}
 
 	return consistent;
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help