Fix 'rebuild' command (we ignore retaste event now, so don't relay on it).

1 files changed, 18 insertions, 9 deletions

diff --git a/sys/geom/mirror/g_mirror_ctl.c b/sys/geom/mirror/g_mirror_ctl.c
index 65c52bd..ee6ef93 100644
--- a/sys/geom/mirror/g_mirror_ctl.c
+++ b/sys/geom/mirror/g_mirror_ctl.c
@@ -198,11 +198,13 @@ g_mirror_ctl_configure(struct gctl_req *req, struct g_class *mp)
 static void
 g_mirror_ctl_rebuild(struct gctl_req *req, struct g_class *mp)
 {
+	struct g_mirror_metadata md;
 	struct g_mirror_softc *sc;
 	struct g_mirror_disk *disk;
+	struct g_provider *pp;
 	const char *name;
 	char param[16];
-	int *nargs;
+	int error, *nargs;
 	u_int i;
 
 	g_topology_assert();
@@ -250,16 +252,28 @@ g_mirror_ctl_rebuild(struct gctl_req *req, struct g_class *mp)
 			return;
 		}
 		/*
-		 * Do rebuild by resetting syncid and disconnecting disk.
-		 * It'll be retasted, connected to the mirror and
-		 * synchronized.
+		 * Do rebuild by resetting syncid, disconnecting the disk and
+		 * connecting it again.
 		 */
 		disk->d_sync.ds_syncid = 0;
 		if ((sc->sc_flags & G_MIRROR_DEVICE_FLAG_NOAUTOSYNC) != 0)
 			disk->d_flags |= G_MIRROR_DISK_FLAG_FORCE_SYNC;
 		g_mirror_update_metadata(disk);
+		pp = disk->d_consumer->provider;
+		error = g_mirror_read_metadata(disk->d_consumer, &md);
 		g_mirror_event_send(disk, G_MIRROR_DISK_STATE_DISCONNECTED,
 		    G_MIRROR_EVENT_WAIT);
+		if (error != 0) {
+			gctl_error(req, "Cannot read metadata from %s.",
+			    pp->name);
+			continue;
+		}
+		error = g_mirror_add_disk(sc, pp, &md);
+		if (error != 0) {
+			gctl_error(req, "Cannot reconnect component %s.",
+			    pp->name);
+			continue;
+		}
 	}
 }
 
@@ -519,11 +533,6 @@ g_mirror_ctl_deactivate(struct gctl_req *req, struct g_class *mp)
 			gctl_error(req, "No such provider: %s.", name);
 			continue;
 		}
-		/*
-		 * Do rebuild by resetting syncid and disconnecting disk.
-		 * It'll be retasted, connected to the mirror and
-		 * synchronized.
-		 */
 		disk->d_flags |= G_MIRROR_DISK_FLAG_INACTIVE;
 		disk->d_flags &= ~G_MIRROR_DISK_FLAG_FORCE_SYNC;
 		g_mirror_update_metadata(disk);