On Wed, Jan 24, 2001 at 08:04:56PM -0500, Chuck Cranor wrote:
> >In my mind, this is how it SHOULD work:
> >	for (pg on inactive list) {
> >		if (free target is met)
> >			break;
> >		if (pg is referenced) {
> >			/* DO NOT CLEAR REFERENCE! */
> >			move page to active list;
> >		}
> 
> given the assumptions in UVM (that inactive pages have no mappings)
> the above if statement should never fire.   that is prob. why chucks
> deleted this statement in UBC.

yup, that's what I was thinking when I removed it, but I didn't notice
that the other check wasn't there.  

has anyone tried putting back the check that I removed and seeing
how much of a difference that makes?  I'd try it myself, but I'm
about as far away from my test machines as I can get while still
remaining on the same planet.

-Chuck