From owner-freebsd-current@FreeBSD.ORG Mon Jul 11 23:57:33 2011 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 9A32E1065672; Mon, 11 Jul 2011 23:57:32 +0000 (UTC) (envelope-from rmacklem@uoguelph.ca) Received: from esa-annu.mail.uoguelph.ca (esa-annu.mail.uoguelph.ca [131.104.91.36]) by mx1.freebsd.org (Postfix) with ESMTP id 3FD4E8FC13; Mon, 11 Jul 2011 23:57:31 +0000 (UTC) X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: Ap0EAP+MG06DaFvO/2dsb2JhbABThESjWbk0kHuBK4QAgQ8EkEuCC4lWhw0 X-IronPort-AV: E=Sophos;i="4.65,517,1304308800"; d="scan'208";a="126887714" Received: from erie.cs.uoguelph.ca (HELO zcs3.mail.uoguelph.ca) ([131.104.91.206]) by esa-annu-pri.mail.uoguelph.ca with ESMTP; 11 Jul 2011 19:57:31 -0400 Received: from zcs3.mail.uoguelph.ca (localhost.localdomain [127.0.0.1]) by zcs3.mail.uoguelph.ca (Postfix) with ESMTP id 42465B3F5A; Mon, 11 Jul 2011 19:57:31 -0400 (EDT) Date: Mon, 11 Jul 2011 19:57:31 -0400 (EDT) From: Rick Macklem To: mdf@FreeBSD.org Message-ID: <1441865969.450238.1310428651218.JavaMail.root@erie.cs.uoguelph.ca> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Originating-IP: [172.17.91.203] X-Mailer: Zimbra 6.0.10_GA_2692 (ZimbraWebClient - FF3.0 (Win)/6.0.10_GA_2692) Cc: Ali Mashtizadeh , FreeBSD Current , Arnaud Lacombe Subject: Re: Heavy I/O blocks FreeBSD box for several seconds X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 11 Jul 2011 23:57:33 -0000 mdf@FreeBSD.org wrote: > On Mon, Jul 11, 2011 at 4:00 PM, Ali Mashtizadeh > wrote: > > Maybe someone can setup something like reviewboard [1] for > > developers > > to use. This may also help folks who want to keep abreast of the > > current work in a particular subsystem or get involved into the > > development process more. At my company we use reviews and it seems > > to > > help the catch some bugs and help new engineers ramp up faster. > > > > [1] http://www.reviewboard.org/ >=20 > FreeBSD development is completely open; anyone can sign up for the > svn-src-* mailing list they are interested in, including > svn-src-head@. Code reviews are plenty as well; just check the list > archives for discussion of bugs, poor design choices and unintended > effects. But most reviews are silent and after-the-fact by looking at > the list mail. It's a system that seems to be working just fine for > the FreeBSD project so far. This isn't a job for most anyone; it's a > volunteer project and so anything that raises the barrier to getting > work done for the project should be looked at with skepticism. >=20 Yes, and if you see a problem with any commit to head (svn-src-head@), you can post to the list about this. Commits to head do not get MFC'd to stable/N until a minimum of 3 days (and usually a couple of weeks), specifically so that anyone can comment on it. If there are serious problems with the commit to head, it can be reverted or fixed via additional commits before it goes into stable/N. rick > Is there a specific deficit that you want to address? >=20 > Thanks, > matthew >=20 > > On Mon, Jul 11, 2011 at 2:48 PM, Arnaud Lacombe > > wrote: > >> Hi, > >> > >> On Mon, Jul 11, 2011 at 5:14 PM, Andriy Gapon > >> wrote: > >>> on 11/07/2011 23:33 Arnaud Lacombe said the following: > >>>> For the record, I would like to see enforced public review for > >>>> _every_ > >>>> patch *before* it is checked in, as a strong rule. gcc system is > >>>> particularly interesting. But it is not likely to happen in > >>>> FreeBSD > >>>> where FreeBSD committers are clearly more free than other at > >>>> checking-in un-publicly-reviewed stuff (especially _bad_ stuff). > >>>> > >>>> This would of course apply even to long-time committers, no > >>>> matter how > >>>> it hurt their ego (which I definitively do not care about). > >>> > >>> Have you just volunteered to review all of the patches that I > >>> would like to > >>> commit? And are you prepared to take responsibility for quality of > >>> your reviews? > >>> I am sure that other developers will gladly accept your offer too. > >>> > >> _No-one_ can do all the reviews, especially not me (on a purely > >> technical level). ACK must come from subsystem maintainers. Having > >> public review would allow the community review, which is now just > >> not > >> possible today. As about patches from the maintainer, they might be > >> committed without his approval, but still sent for review. If a > >> maintainer goes outside his area, he has to get approval from the > >> other subsystem maintainer. > >> > >> =C2=A0- Arnaud > >> _______________________________________________ > >> freebsd-current@freebsd.org mailing list > >> http://lists.freebsd.org/mailman/listinfo/freebsd-current > >> To unsubscribe, send any mail to > >> "freebsd-current-unsubscribe@freebsd.org" > >> > > _______________________________________________ > > freebsd-current@freebsd.org mailing list > > http://lists.freebsd.org/mailman/listinfo/freebsd-current > > To unsubscribe, send any mail to > > "freebsd-current-unsubscribe@freebsd.org" > > > _______________________________________________ > freebsd-current@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-current > To unsubscribe, send any mail to > "freebsd-current-unsubscribe@freebsd.org"