Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 1 May 2006 15:09:03 -0700
From:      Peter Wemm <peter@wemm.org>
To:        Alan Cox <alc@cs.rice.edu>
Cc:        cvs-src@freebsd.org, src-committers@freebsd.org, Peter Jeremy <peterjeremy@optushome.com.au>, Kris Kennaway <kris@obsecurity.org>, cvs-all@freebsd.org, Nate Lawson <nate@root.org>
Subject:   Re: cvs commit: src/sys/i386/i386 pmap.c
Message-ID:  <200605011509.04240.peter@wemm.org>
In-Reply-To: <4453B1E2.5060301@cs.rice.edu>
References:  <200604281905.k3SJ59MZ008806@repoman.freebsd.org> <20060429182318.GA61575@xor.obsecurity.org> <4453B1E2.5060301@cs.rice.edu>

next in thread | previous in thread | raw e-mail | index | archive | help
On Saturday 29 April 2006 11:35, Alan Cox wrote:
> Kris Kennaway wrote:
> > On Sat, Apr 29, 2006 at 10:21:11AM -0700, Peter Wemm wrote:
> >> page of address space by unlinking the head of the ptelist. 
> >> Freeing a page is as simple as storing it on the head.  I'm
> >> running my laptop with that code right now.
> >
> > I'm running on a couple of machines too (WITNESS + INVARIANTS)
> > under heavy load; they've been stable, but I've not tried to
> > determine if the code is doing its job.
>
> Yesterday's patch had one problem.  The vm_page_alloc() call that is
> only executed under low-memory conditions omits VM_PAGE_WIRED.  This
> could cause a later panic when the page is freed.  I e-mailed Peter
> about this yesterday.  I haven't looked to see if he has fixed it
> yet.

There were two cases where I'd missed this.  I committed both before 
touching the rest of the code.

> Alan

-- 
Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com
"All of this is for nothing if we don't go to the stars" - JMS/B5



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