svn commit: r287780 - in head: share/man/man9 sys/kern sys/sys

[Ian Lepore]

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