From owner-freebsd-stable  Wed Jun 26 00:50:40 1996
Return-Path: owner-stable
Received: (from root@localhost)
          by freefall.freebsd.org (8.7.5/8.7.3) id AAA17012
          for stable-outgoing; Wed, 26 Jun 1996 00:50:40 -0700 (PDT)
Received: from root.com (implode.root.com [198.145.90.17])
          by freefall.freebsd.org (8.7.5/8.7.3) with ESMTP id AAA17004;
          Wed, 26 Jun 1996 00:50:37 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by root.com (8.7.5/8.6.5) with SMTP id AAA00743; Wed, 26 Jun 1996 00:50:18 -0700 (PDT)
Message-Id: <199606260750.AAA00743@root.com>
X-Authentication-Warning: implode.root.com: Host localhost [127.0.0.1] didn't use HELO protocol
To: John Hay <jhay@mikom.csir.co.za>
cc: dyson@FreeBSD.org, stable@FreeBSD.org
Subject: Re: Another try at the vm_pageout.c -stable diff 
In-reply-to: Your message of "Sat, 26 Jun 1996 09:22:01 +0200."
             <199606260722.JAA03525@zibbi.mikom.csir.co.za> 
From: David Greenman <davidg@root.com>
Reply-To: davidg@root.com
Date: Wed, 26 Jun 1996 00:50:18 -0700
Sender: owner-stable@FreeBSD.org
X-Loop: FreeBSD.org
Precedence: bulk

>>    Here is yet another diff. This is the product of a discussion I just had
>> with John about the nature of some of the performance problems people have
>> complained about.
>>    This makes a slight change to the policy for bringing pages off of the
>> active queue and should make the statistics gather *much* more effective
>> in some cases. Please try it out and get back to us. Thanks!
>> 
>> -DG
>> 
>> David Greenman
>> Core-team/Principal Architect, The FreeBSD Project
>> 
>> (This includes John's other changes, so apply this to a fresh -stable copy of
>> vm_pageout.c)
>> 
>
>I'm not sure how relevant this still is. I see that there was a few commits
>during the night (well night for me here in SA).
>
>David I have tried your patch, but the kernel dies with "panic: unwire:
>page not in pmap" as soon as the machine starts to swap. It did not say
>anything more.
...
>| f0186eb0 T _vmtotal
>| f0187060 F vm_mmap.o

   Hmmm. I just discovered that my test kernel had the "NO_SWAPPING" option in
it and thus didn't exercise this piece of code. I'm building a kernel right
now with that option removed. I also did a careful code review of that patch,
but something apparantly slipped by. I'm working on it...

-DG

David Greenman
Core-team/Principal Architect, The FreeBSD Project