From d2294d38b1426e4caa40bddf9653c59b1cc81f2f Mon Sep 17 00:00:00 2001 From: dg Date: Sun, 15 Jan 1995 07:31:34 +0000 Subject: [PATCH] Moved some splx's down a few lines in vm_page_insert and vm_page_remove to make the locking a bit more clear - this change is currently a NOP as the calls to those routines are already at splhigh(). --- sys/vm/vm_page.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c index b058a9394872..45852fde3922 100644 --- a/sys/vm/vm_page.c +++ b/sys/vm/vm_page.c @@ -34,7 +34,7 @@ * SUCH DAMAGE. * * from: @(#)vm_page.c 7.4 (Berkeley) 5/7/91 - * $Id: vm_page.c,v 1.14 1995/01/10 07:32:48 davidg Exp $ + * $Id: vm_page.c,v 1.15 1995/01/10 09:19:46 davidg Exp $ */ /* @@ -411,13 +411,13 @@ vm_page_insert(mem, object, offset) simple_lock(&bucket_lock); TAILQ_INSERT_TAIL(bucket, mem, hashq); simple_unlock(&bucket_lock); - (void) splx(s); /* * Now link into the object's list of backed pages. */ TAILQ_INSERT_TAIL(&object->memq, mem, listq); + (void) splx(s); mem->flags |= PG_TABLED; /* @@ -459,13 +459,13 @@ vm_page_remove(mem) simple_lock(&bucket_lock); TAILQ_REMOVE(bucket, mem, hashq); simple_unlock(&bucket_lock); - (void) splx(s); /* * Now remove from the object's list of backed pages. */ TAILQ_REMOVE(&mem->object->memq, mem, listq); + (void) splx(s); /* * And show that the object has one fewer resident page.