From owner-freebsd-hackers@FreeBSD.ORG Fri Jun 18 22:04:05 2004 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from green.homeunix.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 48B7A16A4CE; Fri, 18 Jun 2004 22:04:05 +0000 (GMT) Received: from green.homeunix.org (green@localhost [127.0.0.1]) by green.homeunix.org (8.12.11/8.12.11) with ESMTP id i5IM44gR027974; Fri, 18 Jun 2004 18:04:04 -0400 (EDT) (envelope-from green@green.homeunix.org) Received: (from green@localhost) by green.homeunix.org (8.12.11/8.12.11/Submit) id i5IM43qU027973; Fri, 18 Jun 2004 18:04:03 -0400 (EDT) (envelope-from green) Date: Fri, 18 Jun 2004 18:04:02 -0400 From: Brian Fundakowski Feldman To: Alan Cox Message-ID: <20040618220402.GB27588@green.homeunix.org> References: <20040615195709.GE1016@green.homeunix.org> <20040618205115.GA27588@green.homeunix.org> <20040618212127.GS6753@cs.rice.edu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20040618212127.GS6753@cs.rice.edu> User-Agent: Mutt/1.5.6i cc: alc@FreeBSD.org cc: dillon@backplane.com cc: hackers@FreeBSD.org cc: peter@FreeBSD.org Subject: Re: contigmalloc(9) rewrite X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 18 Jun 2004 22:04:05 -0000 On Fri, Jun 18, 2004 at 04:21:27PM -0500, Alan Cox wrote: > On Fri, Jun 18, 2004 at 04:51:15PM -0400, Brian Fundakowski Feldman wrote: > > On Tue, Jun 15, 2004 at 03:57:09PM -0400, Brian Fundakowski Feldman wrote: > > > The patch, which applies to 5-CURRENT, can be found here: > > > > > > The default is to use the old contigmalloc(). You can set the > > > sysctl or loader tunable "vm.old_contigmalloc" to 0 to enable it. > > > > > > For anyone that normally runs into failed allocations hot-plugging > > > hardware, please try this and see if it helps out. > > > > By the way, I have updated it further to split apart contigmalloc() > > into a separate vm_page_alloc_contig() and mapping function as per > > feedback from Alan Cox and Hiten Pandya. The operation is still the > > same except for now being able to see memory allocated with it > > in your vmstat(8) -m output. The patch is still at the same location, > > and requires sysctl vm.old_contigmalloc=0 to enable. > > > > Why don't you commit the part that makes allocation of physical memory > start from high addresses? I don't think it's very useful fixing the very broken vm.old_contigmalloc=1 anymore, but if someone else would like to do it, I certainly won't object to it. Just that change in behavior isn't useful to what is probably the majority of users for which the low and high arguments to contigmalloc() aren't used. -- Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ <> green@FreeBSD.org \ The Power to Serve! \ Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\