Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 19 Jul 2004 02:11:47 -0500
From:      Alan Cox <alc@cs.rice.edu>
To:        Scott Long <scottl@samsco.org>
Cc:        cvs-src@FreeBSD.org
Subject:   Re: cvs commit: src/sys/sys malloc.h src/sys/kern kern_malloc.c src/sys/vm vm_contig.c vm_page.h
Message-ID:  <20040719071147.GS18577@cs.rice.edu>
In-Reply-To: <40FB718B.7080206@samsco.org>
References:  <200407190621.i6J6LRWJ019976@repoman.freebsd.org> <40FB718B.7080206@samsco.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Jul 19, 2004 at 01:00:27AM -0600, Scott Long wrote:
> This is a pretty darn big change to something that is so vital to so 
> many drivers (not including busdma, which means that this touches nearly
> every driver).  Who reviewed this?  Was it discussed on a public mailing
> list?  What kind of testing was done for it?  If you can't answer these
> questions then I'm not terribly thrilled.
> 

I reviewed an earlier version of this and gave Brian some feedback.

It is important to note that he added the new implementation side-by-side
with the old. By default, the old implementation is still used.  So, I
wouldn't be too concerned.

Alan



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040719071147.GS18577>