svn commit: r191531 - head/sys/vm

Alan Cox alc at FreeBSD.org
Sun Apr 26 20:54:58 UTC 2009


Author: alc
Date: Sun Apr 26 20:54:57 2009
New Revision: 191531
URL: http://svn.freebsd.org/changeset/base/191531

Log:
  Eliminate an archaic band-aid.  The immediately preceding comment already
  explains why the band-aid is unnecessary.
  
  Suggested by:	tegge

Modified:
  head/sys/vm/vm_fault.c

Modified: head/sys/vm/vm_fault.c
==============================================================================
--- head/sys/vm/vm_fault.c	Sun Apr 26 19:16:32 2009	(r191530)
+++ head/sys/vm/vm_fault.c	Sun Apr 26 20:54:57 2009	(r191531)
@@ -916,13 +916,11 @@ vnode_locked:
 	KASSERT(fs.m->oflags & VPO_BUSY,
 		("vm_fault: page %p not busy!", fs.m));
 	/*
-	 * Sanity check: page must be completely valid or it is not fit to
+	 * Page must be completely valid or it is not fit to
 	 * map into user space.  vm_pager_get_pages() ensures this.
 	 */
-	if (fs.m->valid != VM_PAGE_BITS_ALL) {
-		vm_page_zero_invalid(fs.m, TRUE);
-		printf("Warning: page %p partially invalid on fault\n", fs.m);
-	}
+	KASSERT(fs.m->valid == VM_PAGE_BITS_ALL,
+	    ("vm_fault: page %p partially invalid", fs.m));
 	VM_OBJECT_UNLOCK(fs.object);
 
 	/*


More information about the svn-src-all mailing list