On Wed, Jan 24, 2001 at 03:15:37PM -0800, Greywolf wrote:

 > Actually, the first time made more sense to me; why do you need
 > to clear a reference that doesn't exist?
 > 
 > If you're doing that, you might as well say:
 > 
 > 	clear reference;
 > 	if (!pg is referenced) {
 > 		move page to inactive list;
 > 	}

What is not clear from the pseudo-code is that the actual block looks
like this:

	if (pmap_clear_reference(p) == FALSE &&
	    inactive_shortage > 0) {
		/* deactiveate page */
		...
	}

I.e. it's done in one call to the pmap module.

BTW, adding *JUST* the page-is-active check seems to help interactive
performance quite a lot.

-- 
        -- Jason R. Thorpe <thorpej@zembu.com>