PERFORCE change 85882 for review

Alan Cox alc at FreeBSD.org
Wed Oct 26 00:49:00 PDT 2005


http://perforce.freebsd.org/chv.cgi?CH=85882

Change 85882 by alc at alc_home on 2005/10/26 07:47:58

	Move an assertion to its proper place.

Affected files ...

.. //depot/projects/superpages/src/sys/vm/vm_reserve.c#11 edit

Differences ...

==== //depot/projects/superpages/src/sys/vm/vm_reserve.c#11 (text+ko) ====

@@ -315,7 +315,7 @@
 	m->reserv = res;
 	for (;;) {
 		pos = m - res->first_page;
-		KASSERT(res && pos >=0 && pos < (1 << res->order) &&
+		KASSERT(pos >= 0 && pos < (1 << res->order) &&
 				res->popfrom >= 0 && res->refcnt >= 0,
 				("reserve_populate: res invalid"));	
 		res->refcnt++;
@@ -341,6 +341,7 @@
 			reserve_procreate(res, m);
 		}
 		res = follow_child(res, IDX(m, res));
+		KASSERT(res != NULL, ("reserve_populate: missing reservation"));
 	}
 	update_maxavail(res);
 	preempt_move(m->reserv, TRUE);


More information about the p4-projects mailing list