- Add vm object locking to vm_object_deallocate(). (Still more
changes are required.) - Remove special-case macros for kmem object locking. They are no longer used.
This commit is contained in:
parent
1646a714c5
commit
1c0932407b
@ -440,14 +440,13 @@ vm_object_deallocate(vm_object_t object)
|
||||
{
|
||||
vm_object_t temp;
|
||||
|
||||
vm_object_lock(object);
|
||||
if (object != kmem_object)
|
||||
mtx_lock(&Giant);
|
||||
while (object != NULL) {
|
||||
|
||||
VM_OBJECT_LOCK(object);
|
||||
if (object->type == OBJT_VNODE) {
|
||||
VM_OBJECT_LOCK(object);
|
||||
vm_object_vndeallocate(object);
|
||||
mtx_unlock(&Giant);
|
||||
return;
|
||||
goto done;
|
||||
}
|
||||
|
||||
KASSERT(object->ref_count != 0,
|
||||
@ -461,8 +460,8 @@ vm_object_deallocate(vm_object_t object)
|
||||
*/
|
||||
object->ref_count--;
|
||||
if (object->ref_count > 1) {
|
||||
vm_object_unlock(object);
|
||||
return;
|
||||
VM_OBJECT_UNLOCK(object);
|
||||
goto done;
|
||||
} else if (object->ref_count == 1) {
|
||||
if (object->shadow_count == 0) {
|
||||
vm_object_set_flag(object, OBJ_ONEMAPPING);
|
||||
@ -487,11 +486,14 @@ vm_object_deallocate(vm_object_t object)
|
||||
robject->paging_in_progress ||
|
||||
object->paging_in_progress
|
||||
) {
|
||||
/* XXX */ VM_OBJECT_UNLOCK(object);
|
||||
vm_object_pip_sleep(robject, "objde1");
|
||||
vm_object_pip_sleep(object, "objde2");
|
||||
/* XXX */ VM_OBJECT_LOCK(object);
|
||||
}
|
||||
|
||||
VM_OBJECT_UNLOCK(object);
|
||||
if (robject->ref_count == 1) {
|
||||
/* XXX */ VM_OBJECT_LOCK(robject);
|
||||
robject->ref_count--;
|
||||
object = robject;
|
||||
goto doterm;
|
||||
@ -502,11 +504,10 @@ vm_object_deallocate(vm_object_t object)
|
||||
continue;
|
||||
}
|
||||
}
|
||||
vm_object_unlock(object);
|
||||
return;
|
||||
VM_OBJECT_UNLOCK(object);
|
||||
goto done;
|
||||
}
|
||||
doterm:
|
||||
VM_OBJECT_LOCK(object);
|
||||
temp = object->backing_object;
|
||||
if (temp != NULL) {
|
||||
VM_OBJECT_LOCK(temp);
|
||||
@ -527,7 +528,9 @@ vm_object_deallocate(vm_object_t object)
|
||||
VM_OBJECT_UNLOCK(object);
|
||||
object = temp;
|
||||
}
|
||||
vm_object_unlock(object);
|
||||
done:
|
||||
if (object != kmem_object)
|
||||
mtx_unlock(&Giant);
|
||||
}
|
||||
|
||||
/*
|
||||
|
@ -182,11 +182,6 @@ extern struct vm_object kmem_object_store;
|
||||
#define VM_OBJECT_MTX(object) (&(object)->mtx)
|
||||
#define VM_OBJECT_UNLOCK(object) mtx_unlock(&(object)->mtx)
|
||||
|
||||
#define vm_object_lock(object) \
|
||||
mtx_lock((object) == kmem_object ? &kmem_object->mtx : &Giant)
|
||||
#define vm_object_unlock(object) \
|
||||
mtx_unlock((object) == kmem_object ? &kmem_object->mtx : &Giant)
|
||||
|
||||
void vm_object_set_flag(vm_object_t object, u_short bits);
|
||||
void vm_object_clear_flag(vm_object_t object, u_short bits);
|
||||
void vm_object_pip_add(vm_object_t object, short i);
|
||||
|
Loading…
Reference in New Issue
Block a user