From owner-freebsd-questions@FreeBSD.ORG Wed Sep 8 04:52:29 2004 Return-Path: Delivered-To: freebsd-questions@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 0ED5016A4CE; Wed, 8 Sep 2004 04:52:29 +0000 (GMT) Received: from pcwin002.win.tue.nl (pcwin002.win.tue.nl [131.155.71.72]) by mx1.FreeBSD.org (Postfix) with ESMTP id 8505343D48; Wed, 8 Sep 2004 04:52:28 +0000 (GMT) (envelope-from stijn@pcwin002.win.tue.nl) Received: from pcwin002.win.tue.nl (localhost [127.0.0.1]) by pcwin002.win.tue.nl (8.13.1/8.13.1) with ESMTP id i884qRg9052441; Wed, 8 Sep 2004 06:52:27 +0200 (CEST) (envelope-from stijn@pcwin002.win.tue.nl) Received: (from stijn@localhost) by pcwin002.win.tue.nl (8.13.1/8.13.1/Submit) id i884qRad052440; Wed, 8 Sep 2004 06:52:27 +0200 (CEST) (envelope-from stijn) Date: Wed, 8 Sep 2004 06:52:27 +0200 From: Stijn Hoop To: "Greg 'groggy' Lehey" Message-ID: <20040908045227.GC48026@pcwin002.win.tue.nl> Mail-Followup-To: Stijn Hoop , Greg 'groggy' Lehey , freebsd-questions@FreeBSD.org References: <20040825083123.GD17106@pcwin002.win.tue.nl> <86pt5f1qdm.fsf@borg.borderworlds.dk> <20040827093352.GB965@pcwin002.win.tue.nl> <86isb4py41.fsf@borg.borderworlds.dk> <20040829142657.GE7435@pcwin002.win.tue.nl> <20040902101700.GL65336@pcwin002.win.tue.nl> <20040908015846.GO82881@wantadilla.lemis.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="4ZLFUWh1odzi/v6L" Content-Disposition: inline In-Reply-To: <20040908015846.GO82881@wantadilla.lemis.com> User-Agent: Mutt/1.4.2.1i X-Bright-Idea: Let's abolish HTML mail! cc: freebsd-questions@FreeBSD.org Subject: Re: vinum revive does not rebuild parity (was vinum rebuildparity, when?) X-BeenThere: freebsd-questions@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: User questions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 Sep 2004 04:52:29 -0000 --4ZLFUWh1odzi/v6L Content-Type: multipart/mixed; boundary="A6N2fC+uXW/VQSAv" Content-Disposition: inline --A6N2fC+uXW/VQSAv Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Sep 08, 2004 at 11:28:46AM +0930, Greg 'groggy' Lehey wrote: > >> [...] the parity surely is not correctly recalculated during > >> the revive. >=20 > If that were the case, the parity would be incorrect at offset 0. > Yes, it is recalculated. Of course -- I hadn't thought of that. > >> Greg, can you tell me if this is correct behaviour? >=20 > Sorry for the slow response. I was at a conference last week. No, > it's not correct. No problem; this is still a volunteer project last time I checked. In a way I am glad to hear that it is not correct. > > While not having heard back yet, I had to rebuild another subdisk, > > but I decided to do it off-line this time. Turns out the parity was > > rebuilt ok.=20 >=20 > Yes, this is what I recommended. OK. > > Might there be a bug in the online rebuild code? >=20 > Looks like it. >=20 > The current version of Vinum is on its last legs. Lukas Ertl is > rewriting it, so don't expect much change in this version. For the > time being, just accept that you should umount before rebuilding a > plex. I will; it's just that somehow I was led to believe that I didn't need to do that. This has caused me some pain in the past. May I suggest applying the attached patch to /usr/src/sbin/vinum/vinum.8? At least it would prevent someone else from making the same mistakes as me. Thanks for your response, --Stijn --=20 The problem is that there are several people in design positions now who couldn't design the Next Big Thing(TM) unless it involved them taking a photocopier and someone else's design of The Next Big Thing(TM). -- 'Alkaiser' in a post on Slashdot on game originality --A6N2fC+uXW/VQSAv Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="vinum.8.patch" --- vinum.8.orig Wed Sep 8 06:47:46 2004 +++ vinum.8 Wed Sep 8 06:51:19 2004 @@ -441,6 +441,10 @@ .Ic checkparity prints a running progress report. .Pp +It is advisable to always check the parity of a RAID-4 or RAID-5 plex after +an unclean shutdown. Corrupt parity is as bad as degraded mode for such a +plex; if one of the subdisks of such a plex fails, data corruption will occur. +.Pp .It Xo .Ic concat .Op Fl f @@ -1046,6 +1050,11 @@ flag is specified, .Ic rebuildparity prints a running progress report. +.Pp +At present, a bug prevents rebuildparity from correctly completing its job +when the vinum volume is mounted and being accessed. You should only rebuild +the parity of plexes on unmounted volumes in order to guarantee correct parity +checks. .Pp .It Xo .Ic rename --A6N2fC+uXW/VQSAv-- --4ZLFUWh1odzi/v6L Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.6 (FreeBSD) iD8DBQFBPpALY3r/tLQmfWcRAo4OAKCXQw+tekDkeqPqWR/KZoUs+eBQTACfUdZV 2zqyiXtekQEJcGs8izy9BhM= =iFxw -----END PGP SIGNATURE----- --4ZLFUWh1odzi/v6L--