From owner-cvs-src@FreeBSD.ORG Fri Nov 5 17:23:11 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 46D4E16A4CE; Fri, 5 Nov 2004 17:23:11 +0000 (GMT) Received: from darkness.comp.waw.pl (darkness.comp.waw.pl [195.117.238.136]) by mx1.FreeBSD.org (Postfix) with ESMTP id DCDD043D5A; Fri, 5 Nov 2004 17:23:10 +0000 (GMT) (envelope-from pjd@darkness.comp.waw.pl) Received: by darkness.comp.waw.pl (Postfix, from userid 1009) id 008E9ACDD0; Fri, 5 Nov 2004 18:23:08 +0100 (CET) Date: Fri, 5 Nov 2004 18:23:08 +0100 From: Pawel Jakub Dawidek To: Scott Long Message-ID: <20041105172308.GW8120@darkness.comp.waw.pl> References: <200411050905.iA595Gct083808@repoman.freebsd.org> <418B7C92.5000005@freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="LGr9gtkSK9ARF7Ax" Content-Disposition: inline In-Reply-To: <418B7C92.5000005@freebsd.org> User-Agent: Mutt/1.4.2i X-PGP-Key-URL: http://people.freebsd.org/~pjd/pjd.asc X-OS: FreeBSD 5.2.1-RC2 i386 cc: cvs-src@freebsd.org cc: src-committers@freebsd.org cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/geom/mirror g_mirror.c g_mirror.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 05 Nov 2004 17:23:11 -0000 --LGr9gtkSK9ARF7Ax Content-Type: text/plain; charset=iso-8859-2 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Nov 05, 2004 at 06:13:54AM -0700, Scott Long wrote: +> Pawel Jakub Dawidek wrote: +> >pjd 2004-11-05 09:05:15 UTC +> > +> > FreeBSD src repository +> > +> > Modified files: +> > sys/geom/mirror g_mirror.c g_mirror.h=20 +> > Log: +> > MFp4: +> > - Fix for good (I hope) force-stopping mirrors and some filure cases +> > (e.g. the last good component dies when synchronization is in=20 +> > progress). +> > Don't use ->nstart/->nend consumer's fields, as this could be racy, +> > because those fields are used in g_down/g_up, use ->index consumer's +> > field instead for tracking number of not finished requests. +> > =20 +> > Reported by: marcel +> > =20 +> > - After 5 seconds of idle time (this should be configurable) mark all +> > dirty providers as clean, so when mirror is not used in 5 seconds +> > and there will be power failure, no synchronization on boot is need= ed. +> > =20 +> > Idea from: sorry, I can't find who suggested this +>=20 +> Do you check that all outstanding writes are actually completed in this +> timeout period, or just that you haven't recieved any new writes in this +> period? I'm checking it now, thanks! --=20 Pawel Jakub Dawidek http://www.FreeBSD.org pjd@FreeBSD.org http://garage.freebsd.pl FreeBSD committer Am I Evil? Yes, I Am! --LGr9gtkSK9ARF7Ax Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.4 (FreeBSD) iD8DBQFBi7b8ForvXbEpPzQRAgmcAKDjJttrod9fGMKqyWMLWA6gXW5gawCbBFCa 4UYW/n/HSsNtswYnQhff/4E= =og9R -----END PGP SIGNATURE----- --LGr9gtkSK9ARF7Ax--