Date: Sun, 4 Jul 2004 12:26:23 -0400 From: Brian Fundakowski Feldman <green@FreeBSD.org> To: Bosko Milekic <bmilekic@FreeBSD.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/vm uma_core.c Message-ID: <20040704162623.GB997@green.homeunix.org> In-Reply-To: <20040704161413.GA82599@freefall.freebsd.org> References: <20040704161413.GA82599@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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. -- Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ <> green@FreeBSD.org \ The Power to Serve! \ Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040704162623.GB997>