d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use pmap_remove_all() instead of pmap_remove() before freeing the page

Browse Source 
in vm_pgmoveco(); the page may have more than one mapping.  Hold the page
queues lock when calling pmap_remove_all().

Approved by:	re (blanket)
This commit is contained in:
alc 2002-11-28 08:44:26 +00:00
parent adf02aa8e3
commit 76681d4bc6
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
sys/kern/kern_subr.c
View File

@ -104,12 +104,11 @@ vm_pgmoveco(mapa, srcobj, kaddr, uaddr)
return(EFAULT);
}
if ((user_pg = vm_page_lookup(uobject, upindex)) != NULL) {
vm_page_lock_queues();
if (!vm_page_sleep_if_busy(user_pg, 1, "vm_pgmoveco"))
vm_page_unlock_queues();
pmap_remove(map->pmap, uaddr, uaddr+PAGE_SIZE);
vm_page_lock_queues();
do
vm_page_lock_queues();
while (vm_page_sleep_if_busy(user_pg, 1, "vm_pgmoveco"));
vm_page_busy(user_pg);
pmap_remove_all(user_pg);
vm_page_free(user_pg);
vm_page_unlock_queues();
}