diff options
| author | Alan Cox <alc@FreeBSD.org> | 2002-11-28 08:44:26 +0000 |
|---|---|---|
| committer | Alan Cox <alc@FreeBSD.org> | 2002-11-28 08:44:26 +0000 |
| commit | fdff30d256e3cd6f6414902c4494b797308279aa (patch) | |
| tree | 102fd2d9dd7aed523d9839b1310d2289bd983299 /sys/kern/kern_subr.c | |
| parent | e80b7b691ec855dd12675138ec99a17b1166f9c9 (diff) | |
| download | src-fdff30d256e3cd6f6414902c4494b797308279aa.tar.gz src-fdff30d256e3cd6f6414902c4494b797308279aa.zip | |
Use pmap_remove_all() instead of pmap_remove() before freeing the page
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)
Notes
Notes:
svn path=/head/; revision=107371
Diffstat (limited to 'sys/kern/kern_subr.c')
| -rw-r--r-- | sys/kern/kern_subr.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/sys/kern/kern_subr.c b/sys/kern/kern_subr.c index 065659872d6c..2df1a130a5b2 100644 --- a/sys/kern/kern_subr.c +++ b/sys/kern/kern_subr.c @@ -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(); } |