Date: Wed, 26 Oct 2005 07:48:58 GMT From: Alan Cox <alc@FreeBSD.org> To: Perforce Change Reviews <perforce@freebsd.org> Subject: PERFORCE change 85882 for review Message-ID: <200510260748.j9Q7mwP9096380@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=85882 Change 85882 by alc@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);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200510260748.j9Q7mwP9096380>