svn commit: r204719 - head/sys/powerpc/aim

Nathan Whitehorn nwhitehorn at FreeBSD.org
Thu Mar 4 17:24:31 UTC 2010


Author: nwhitehorn
Date: Thu Mar  4 17:24:31 2010
New Revision: 204719
URL: http://svn.freebsd.org/changeset/base/204719

Log:
  Fix an obvious lock escape and fix a typo in a comment.

Modified:
  head/sys/powerpc/aim/mmu_oea64.c

Modified: head/sys/powerpc/aim/mmu_oea64.c
==============================================================================
--- head/sys/powerpc/aim/mmu_oea64.c	Thu Mar  4 16:56:36 2010	(r204718)
+++ head/sys/powerpc/aim/mmu_oea64.c	Thu Mar  4 17:24:31 2010	(r204719)
@@ -1664,8 +1664,10 @@ moea64_page_exists_quick(mmu_t mmu, pmap
 	loops = 0;
 	LOCK_TABLE();
 	LIST_FOREACH(pvo, vm_page_to_pvoh(m), pvo_vlink) {
-		if (pvo->pvo_pmap == pmap)
+		if (pvo->pvo_pmap == pmap) {
+			UNLOCK_TABLE();
 			return (TRUE);
+		}
 		if (++loops >= 16)
 			break;
 	}
@@ -2063,7 +2065,7 @@ moea64_pvo_enter(pmap_t pm, uma_zone_t z
 		bootstrap = 1;
 	} else {
 		/*
-		 * Note: drop the table around the UMA allocation in
+		 * Note: drop the table lock around the UMA allocation in
 		 * case the UMA allocator needs to manipulate the page
 		 * table. The mapping we are working with is already
 		 * protected by the PMAP lock.


More information about the svn-src-head mailing list