Date: Mon, 10 Nov 2003 09:16:18 -0800 From: Gordon Tetlow <gordont@gnf.org> To: Wes Peters <wes@softweyr.com> Cc: arch@freebsd.org Subject: Re: newfs and mount vs. half-baked disks Message-ID: <20031110171618.GD71376@roark.gnf.org> In-Reply-To: <200311060943.34284.wes@softweyr.com> References: <200311041737.20467.wes@softweyr.com> <20031105015709.GC28915@dan.emsphone.com> <20031105081516.GA38016@cirb503493.alcatel.com.au> <200311060943.34284.wes@softweyr.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--5l2qnUqb3CiDhrj9 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Nov 06, 2003 at 09:43:34AM -0800, Wes Peters wrote: >=20 > I found an unused field called "fs_state" and used that, as Kirk=20 > suggested. Here's the new patch, which changes fsck to notice the=20 > fs_state and doesn't require re-writing all of the superblocks. >=20 > This patch adds a -E (generate errors) option to fsck, causing fsck to=20 > exit at various stages or to otherwise leave the state of fs_state and=20 > fs_clean in other than pristine conditions. I will, of course, commit=20 > the -E changes separately from the fs_state changes. >=20 > Thanks in advance for reviewing. And yes, I did manage to attach the=20 > patch this time. Doh! After a cursory glance, a couple of nits. Perhaps ErrorFlag should be Eflag to be consistent with the style of the rest of the source? Also, your error reporting in fsck_ffs is fs.state !=3D 0 is less than obvious: "superblock %d is not finished" Perhaps it could be "superblock %d is in an inconsistent state, this is probably due to a premature exit of newfs" or some such message. -gordon --5l2qnUqb3CiDhrj9 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.2 (FreeBSD) iD8DBQE/r8fiRu2t9DV9ZfsRAhebAJ98YoH5Bm9ldLenqlWxfXzPriLedQCdH/4U Bam0eoJTWuYZMfT7Lj1c8Sg= =Pqwh -----END PGP SIGNATURE----- --5l2qnUqb3CiDhrj9--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20031110171618.GD71376>