PERFORCE change 147750 for review
Mayur Shardul
mayur at FreeBSD.org
Mon Aug 18 18:38:46 UTC 2008
http://perforce.freebsd.org/chv.cgi?CH=147750
Change 147750 by mayur at mayur_freebsd_vm on 2008/08/18 18:38:25
Added more checks
Affected files ...
.. //depot/projects/soc2008/mayur_vmalgo/kern/src/sys/vm/vm_reserv.c#3 edit
Differences ...
==== //depot/projects/soc2008/mayur_vmalgo/kern/src/sys/vm/vm_reserv.c#3 (text+ko) ====
@@ -314,10 +314,11 @@
msucc = NULL;
//mpred = object->root;
mpred = radix_tree_lookup_le(pindex, &object->rtree);
- if(mpred != NULL){
+ if(mpred != NULL && mpred->object == object){
KASSERT(mpred->pindex != pindex,
("vm_reserv_alloc_page: pindex already allocated"));
rv = vm_reserv_from_page(mpred);
+ KASSERT( rv != NULL, ("vm_reserv_alloc_page: null rv"));
if (rv->object == object && vm_reserv_has_pindex(rv, pindex)) {
m = &rv->pages[VM_RESERV_INDEX(object, pindex)];
//Handle vm_page_rename(m, new_object, ...).
@@ -326,12 +327,16 @@
vm_reserv_populate(rv);
return (m);
}
+ }else{
+ mpred = NULL;
}
+
msucc = radix_tree_lookup_ge(pindex, &object->rtree);
- if(msucc != NULL){
+ if(msucc != NULL && msucc->object == object){
KASSERT(msucc->pindex != pindex,
("vm_reserv_alloc_page: pindex already allocated"));
rv = vm_reserv_from_page(msucc);
+ KASSERT( rv != NULL, ("vm_reserv_alloc_page: null rv"));
if (rv->object == object && vm_reserv_has_pindex(rv, pindex)) {
m = &rv->pages[VM_RESERV_INDEX(object, pindex)];
//Handle vm_page_rename(m, new_object, ...).
@@ -340,6 +345,8 @@
vm_reserv_populate(rv);
return (m);
}
+ }else{
+ msucc = NULL;
}
/*
while (mpred != NULL) {
More information about the p4-projects
mailing list