Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 15 Oct 2001 11:46:16 -0500 (CDT)
From:      mark tinguely <tinguely@web.cs.ndsu.nodak.edu>
To:        dillon@earth.backplane.com, tinguely@web.cs.ndsu.nodak.edu
Cc:        freebsd-hackers@FreeBSD.ORG, Patrick.Cipiere@udcast.com, tmoestl@gmx.net
Subject:   Re: contigfree, free what?
Message-ID:  <200110151646.f9FGkGm33248@web.cs.ndsu.nodak.edu>
In-Reply-To: <200110151613.f9FGDb854996@earth.backplane.com>

next in thread | previous in thread | raw e-mail | index | archive | help

Assuming we are using Thomas' patch that already removed the vm_page_wire()
from the earlier for loop, then at the point of this VM space allocation
failure, we haven't done anything too serious to the vm_page nor to the pmap,
nor are they in any object. We should be able to simply place it back to the
colored free list, something as easy as:

*** vm_page.c	Mon Oct 15 10:26:14 2001
--- vm_page.c.new	Mon Oct 15 11:32:46 2001
***************
*** 1934,1939 ****
--- 1934,1942 ----
  			 * above available.
  			 */
  			vm_map_unlock(map);
+ 			for (i = start; i < (start + size / PAGE_SIZE); i++) {
+ 				(void)vm_add_new_page(VM_PAGE_TO_PHYS(&pga[i]));
+ 			}
  			splx(s);
  			return (NULL);
  		}


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-hackers" in the body of the message




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