From owner-cvs-all@FreeBSD.ORG Sun Jul 4 17:10:59 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2466C16A4CE; Sun, 4 Jul 2004 17:10:59 +0000 (GMT) Received: from harmony.village.org (rover.village.org [168.103.84.182]) by mx1.FreeBSD.org (Postfix) with ESMTP id 97D1A43D1F; Sun, 4 Jul 2004 17:10:58 +0000 (GMT) (envelope-from imp@bsdimp.com) Received: from localhost (warner@rover2.village.org [10.0.0.1]) by harmony.village.org (8.12.11/8.12.11) with ESMTP id i64H9NjS032331; Sun, 4 Jul 2004 11:09:23 -0600 (MDT) (envelope-from imp@bsdimp.com) Date: Sun, 04 Jul 2004 11:09:25 -0600 (MDT) Message-Id: <20040704.110925.83993844.imp@bsdimp.com> To: green@FreeBSD.ORG From: "M. Warner Losh" In-Reply-To: <20040704162623.GB997@green.homeunix.org> References: <20040704161413.GA82599@freefall.freebsd.org> <20040704162623.GB997@green.homeunix.org> X-Mailer: Mew version 3.3 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit cc: cvs-src@FreeBSD.ORG cc: src-committers@FreeBSD.ORG cc: bmilekic@FreeBSD.ORG cc: cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/vm uma_core.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 04 Jul 2004 17:10:59 -0000 In message: <20040704162623.GB997@green.homeunix.org> Brian Fundakowski Feldman writes: : On Sun, Jul 04, 2004 at 04:14:13PM +0000, Bosko Milekic wrote: : > : > This is unfair. I told you both in public and in private that : > I didn't think this approach of segragating this behavior to mbuf : > allocations was the right approach and even suggested what the right : > approach should have been. You asked me to review a change that : > further segragated this behavior to mbuf allocations but then : > committed the change 5 minutes later without giving me an : > opportunity to reply. : > : > What I've done is made the sysctl-introduction change myself and : > committed it instead of this. : > : > In the future I hope that you'll learn to be more patient and less : > righteous when ordering others not to commit something before : > going through the SO. : : You pointed out that it was incorrect because any of the mbuf zones : could be allocated from with locks held and you wanted to prevent : that, so I made the change _I_ had done before less bogus. : : You just introduced a gaping security hole that is turned on by : default. Back this out immediately. SHUT THE FUCK UP. YOU HAVE NO GROUNDS TO MAKE SUCH DEMANDS. Yelling at other committers is viewed dimly by core. If you don't stop it, core will take action to correct the problem, despite pending elections. Proof by assertion isn't acceptible and your actions here disturb me greatly. It is the security officer's responsibiltiy to make such requests, not yours. I'm sorry to shout, but Brian, your hands are far from clean in this debate. Please learn how to work with your fellow committer rather than stomp on them. Thanks. Warner