o Setting PG_MAPPED and PG_WRITEABLE on pages that are mapped and unmapped

   by pmap_qenter() and pmap_qremove() is pointless.  In fact, it probably
   leads to unnecessary pmap_page_protect() calls if one of these pages is
   paged out after unwiring.

Note: setting PG_MAPPED asserts that the page's pv list may be
non-empty.  Since checking the status of the page's pv list isn't any
harder than checking this flag, the flag should probably be eliminated.
Alternatively, PG_MAPPED could be set by pmap_enter() exclusively
rather than various places throughout the kernel.

1 files changed, 0 insertions, 2 deletions

diff --git a/sys/vm/vm_glue.c b/sys/vm/vm_glue.c
index ff8b3df..49e1dd7 100644
--- a/sys/vm/vm_glue.c
+++ b/sys/vm/vm_glue.c
@@ -236,7 +236,6 @@ vm_proc_new(struct proc *p)
 
 		vm_page_wakeup(m);
 		vm_page_flag_clear(m, PG_ZERO);
-		vm_page_flag_set(m, PG_MAPPED | PG_WRITEABLE);
 		m->valid = VM_PAGE_BITS_ALL;
 	}
 
@@ -328,7 +327,6 @@ vm_proc_swapin(struct proc *p)
 		m->valid = VM_PAGE_BITS_ALL;
 		vm_page_wire(m);
 		vm_page_wakeup(m);
-		vm_page_flag_set(m, PG_MAPPED | PG_WRITEABLE);
 	}
 	vm_page_unlock_queues();
 	up = (vm_offset_t)p->p_uarea;