From owner-cvs-src@FreeBSD.ORG Fri Aug 27 05:23:15 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 B0B1516A4CE; Fri, 27 Aug 2004 05:23:15 +0000 (GMT) Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.86.163]) by mx1.FreeBSD.org (Postfix) with ESMTP id 1B92543D2D; Fri, 27 Aug 2004 05:23:15 +0000 (GMT) (envelope-from phk@critter.freebsd.dk) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.13.1/8.13.1) with ESMTP id i7R5N1O9075046; Fri, 27 Aug 2004 07:23:01 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Brian Fundakowski Feldman From: "Poul-Henning Kamp" In-Reply-To: Your message of "Thu, 26 Aug 2004 16:15:31 EDT." <20040826201531.GQ77326@green.homeunix.org> Date: Fri, 27 Aug 2004 07:23:01 +0200 Message-ID: <75045.1093584181@critter.freebsd.dk> Sender: phk@critter.freebsd.dk cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: Pawel Jakub Dawidek cc: cvs-all@FreeBSD.org cc: Roman Kurakin Subject: Re: cvs commit: src/sys/geom/concat g_concat.c src/sys/geom/label g_label.c src/sys/geom/mirror g_mirror.c src/sys/geom/raid3 g_raid3.c src/sys/geom/stripe g_stripe.c 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, 27 Aug 2004 05:23:15 -0000 In message <20040826201531.GQ77326@green.homeunix.org>, Brian Fundakowski Feldm an writes: >> >Why in the world would you crash the machine when you don't have to? >> >> To prevent bogus code from being committed in the first place. >> >> Presumably if people can not make their code work because it panics >> they will not commit it. > >You cannot presume testing will expose every possible state/condition. If you hit any of the KASSERTS in geom, your code is very broken and if your testing doesn't uncover that (with the help of the KASSERTS) then you have no business being a committer. >It hasn't in FreeBSD's past, but maybe you can start a revolution that >makes it simple to create very complete test suites and put them into >the tree along with the code being tested. I've been of the most vocal advocates of getting regression tests into the tree for longer than you have been a committer. I was me who added src/tools originally. Now, shut up and code. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence.