Date: Mon, 19 Jul 2004 01:16:51 -0600 From: Scott Long <scottl@samsco.org> To: Alan Cox <alc@cs.rice.edu> Cc: cvs-src@FreeBSD.org Subject: Re: cvs commit: src/sys/sys malloc.h src/sys/kern kern_malloc.csrc/sys/vm vm_contig.c vm_page.h Message-ID: <40FB7563.4080600@samsco.org> In-Reply-To: <20040719071147.GS18577@cs.rice.edu> References: <200407190621.i6J6LRWJ019976@repoman.freebsd.org> <40FB718B.7080206@samsco.org> <20040719071147.GS18577@cs.rice.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
Alan Cox wrote: > 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 Ok, I guess I missed the mention of the vm.old_contigmalloc tunable on my first reading. I'd still like to know what it has been tested on and what the future testing and maintenance requirements are. This work looks to be very interesting and beneficial, so please don't take these emails to mean that I don't like it. I'm just a bit nervous these days. Scott
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?40FB7563.4080600>