Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 7 Feb 2011 15:47:44 +0000 (UTC)
From:      Alexander Motin <mav@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject:   svn commit: r218405 - projects/graid/head/sys/geom/raid
Message-ID:  <201102071547.p17FliMZ014181@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mav
Date: Mon Feb  7 15:47:43 2011
New Revision: 218405
URL: http://svn.freebsd.org/changeset/base/218405

Log:
  Measure rebuild_meta_update in rebuild slabs, not in clusters of slabs.
  It is more stable and predictable unit.

Modified:
  projects/graid/head/sys/geom/raid/tr_raid1.c

Modified: projects/graid/head/sys/geom/raid/tr_raid1.c
==============================================================================
--- projects/graid/head/sys/geom/raid/tr_raid1.c	Mon Feb  7 15:30:11 2011	(r218404)
+++ projects/graid/head/sys/geom/raid/tr_raid1.c	Mon Feb  7 15:47:43 2011	(r218405)
@@ -76,7 +76,7 @@ SYSCTL_UINT(_kern_geom_raid_raid1, OID_A
     &g_raid1_rebuild_cluster_idle, RAID1_REBUILD_CLUSTER_IDLE,
     "Number of slabs to do each time we trigger a rebuild cycle");
 
-#define RAID1_REBUILD_META_UPDATE 500 /* update meta data every 5 GB or so */
+#define RAID1_REBUILD_META_UPDATE 1024 /* update meta data every 1GB or so */
 static int g_raid1_rebuild_meta_update = RAID1_REBUILD_META_UPDATE;
 TUNABLE_INT("kern.geom.raid.raid1.rebuild_meta_update",
     &g_raid1_rebuild_slab);
@@ -691,13 +691,13 @@ g_raid_tr_iodone_raid1(struct g_raid_tr_
 					return;
 				}
 
+				if (--trs->trso_meta_update <= 0) {
+					g_raid_write_metadata(vol->v_softc,
+					    vol, nsd, nsd->sd_disk);
+					trs->trso_meta_update =
+					    g_raid1_rebuild_meta_update;
+				}
 				if (--trs->trso_recover_slabs <= 0) {
-					if (--trs->trso_meta_update <= 0) {
-						g_raid_write_metadata(vol->v_softc,
-						    vol, nsd, nsd->sd_disk);
-						trs->trso_meta_update =
-						    g_raid1_rebuild_meta_update;
-					}
 					trs->trso_flags &= ~TR_RAID1_F_DOING_SOME;
 					return;
 				}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201102071547.p17FliMZ014181>