 @@ -1,14 +1,14 @@
-/* $NetBSD: pmap.c,v 1.42 2011/08/30 10:44:06 reinoud Exp $ */
+/* $NetBSD: pmap.c,v 1.43 2011/08/30 10:58:41 reinoud Exp $ */
 /*-
  * Copyright (c) 2011 Reinoud Zandijk <reinoud@NetBSD.org>
  * All rights reserved.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
 @@ -17,27 +17,27 @@
  * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
  * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
  * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
  * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
  * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
  * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
  * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
  * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pmap.c,v 1.42 2011/08/30 10:44:06 reinoud Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pmap.c,v 1.43 2011/08/30 10:58:41 reinoud Exp $");
 #include "opt_memsize.h"
 #include "opt_kmempages.h"
 #include <sys/types.h>
 #include <sys/param.h>
 #include <sys/mutex.h>
 #include <sys/buf.h>
 #include <sys/malloc.h>
 #include <sys/pool.h>
 #include <machine/thunk.h>
 #include <uvm/uvm.h>
 @@ -822,44 +822,70 @@ pmap_zero_page(paddr_t pa)
 void
 pmap_copy_page(paddr_t src, paddr_t dst)
+{
 aprint_debug("pmap_copy_page not implemented\n");
+}
 void
 pmap_page_protect(struct vm_page *pg, vm_prot_t prot)
+{
 aprint_debug("pmap_page_protect not implemented\n");
+}
 bool
-pmap_clear_modify(struct vm_page *pg)
+pmap_clear_modify(struct vm_page *page)
+{
-aprint_debug("pmap_clear_modify not implemented\n");
+	struct pv_entry *pv;
-	return true;
+	uintptr_t ppn;
 	bool rv;
 	ppn = atop(VM_PAGE_TO_PHYS(page));
 	rv = pmap_is_modified(page);
 	aprint_debug("pmap_clear_modify page %"PRIiPTR"\n", ppn);
 	/* if marked modified, clear it in all the pmap's referencing it */
 	if (rv) {
 		/* if its marked modified in a kernel mapping, don't clear it */
 		for (pv = &pv_table[ppn]; pv != NULL; pv = pv->pv_next)
 			if (pv->pv_pmap == pmap_kernel() &&
 			    (pv->pv_prot & VM_PROT_WRITE))
 				return rv;
 		/* clear it */
 		pv_table[ppn].pv_pflags &= ~PV_MODIFIED;
 		pmap_update_page(ppn);
+	}
 	return rv;
+}
 bool
 pmap_clear_reference(struct vm_page *pg)
+{
 aprint_debug("pmap_clear_reference not implemented\n");
 	return true;
+}
 bool
-pmap_is_modified(struct vm_page *pg)
+pmap_is_modified(struct vm_page *page)
+{
-aprint_debug("pmap_is_modified not implemented\n");
+	intptr_t ppn;
-	return false;
+	bool rv;
 	ppn = atop(VM_PAGE_TO_PHYS(page));
 	rv = (pv_table[ppn].pv_pflags & PV_MODIFIED) != 0;
 	aprint_debug("pmap_is_modified page %"PRIiPTR" : %s\n", ppn, rv?"yes":"no");
 	return rv;
+}
 bool
 pmap_is_referenced(struct vm_page *pg)
+{
 aprint_debug("pmap_is_referenced not implemented\n");
 	return false;
+}
 paddr_t
 pmap_phys_address(paddr_t cookie)
+{
 	panic("pmap_phys_address not implemented\n");