From owner-svn-src-all@freebsd.org Sat Sep 19 17:39:41 2015 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 096F5A05546 for ; Sat, 19 Sep 2015 17:39:41 +0000 (UTC) (envelope-from ian@freebsd.org) Received: from erouter6.ore.mailhop.org (erouter6.ore.mailhop.org [54.187.213.119]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 3B60913B4 for ; Sat, 19 Sep 2015 17:39:40 +0000 (UTC) (envelope-from ian@freebsd.org) Received: from ilsoft.org (unknown [73.34.117.227]) by outbound3.ore.mailhop.org (Halon Mail Gateway) with ESMTPSA; Sat, 19 Sep 2015 17:38:32 +0000 (UTC) Received: from rev (rev [172.22.42.240]) by ilsoft.org (8.14.9/8.14.9) with ESMTP id t8JHdTtK005522; Sat, 19 Sep 2015 11:39:29 -0600 (MDT) (envelope-from ian@freebsd.org) Message-ID: <1442684369.1224.179.camel@freebsd.org> Subject: Re: svn commit: r287780 - in head: share/man/man9 sys/kern sys/sys From: Ian Lepore To: George Neville-Neil Cc: Hans Petter Selasky , David Chisnall , Gleb Smirnoff , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Date: Sat, 19 Sep 2015 11:39:29 -0600 In-Reply-To: <64D8263B-1F5D-40E5-994C-479C39B69DC9@neville-neil.com> References: <201509141052.t8EAqRWf008293@repo.freebsd.org> <20150916220559.GS1023@FreeBSD.org> <55FA69BD.10507@selasky.org> <0952027A-5276-487D-99B8-74747B0EEF5D@FreeBSD.org> <55FD23C5.5010008@selasky.org> <64D8263B-1F5D-40E5-994C-479C39B69DC9@neville-neil.com> Content-Type: text/plain; charset="us-ascii" X-Mailer: Evolution 3.12.10 FreeBSD GNOME Team Port Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 19 Sep 2015 17:39:41 -0000 On Sat, 2015-09-19 at 12:37 -0400, George Neville-Neil wrote: > > On 19 Sep 2015, at 4:58, Hans Petter Selasky wrote: > > > On 09/17/15 11:07, David Chisnall wrote: > >> I would expect*anyone* making a change like this to have both the > >> design and code reviewed for sanity checking. > > > > Please add an entry to MAINTAINERS for those parts of the kernel which > > require this strict reviews. > > > > This is not about MAINTAINERS, this is about review. Several commenters > on this thread > have pointed out that this code, which is part of a core component of > the system, was committed > without review. Please back this out and then put whatever you propose > to do up in the reviews.freebsd.org > system. > > Best, > George > > I'm afraid this message can be interpetted as "reviews are now mandatory for a 'core component of the system' (whatever that means)". If so, this would be a Big Change from the last thing I heard about code reviews, which was basically: as much as some people would like it to be so, they are not mandatory. Another way to interpret this might be "While code reviews are not generally mandatory, if multiple developers request a review of some change, even after the fact, then that should (must?) be honored." Could we please get some clarification (officially, from core) about this? And yes I know that certain areas of the system, such as crypto-related code, have always been treated differently, but that has never involved a definition as meaningless and vague as "a core component of the system". -- Ian